Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(36)

Side by Side Diff: chrome/renderer/prerender/prerender_dispatcher_unittest.cc

Issue 10424002: Adding unit tests for PrerenderDispatcher. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleanup header and add a couple of extra tests Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/renderer/prerender/prerender_dispatcher.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/renderer/prerender/prerender_dispatcher.h"
6
7 #include "base/compiler_specific.h"
8 #include "base/logging.h"
9 #include "googleurl/src/gurl.h"
10 #include "testing/gtest/include/gtest/gtest.h"
11
12 namespace prerender {
13
14 class PrerenderDispatcherTest : public testing::Test {
15 public:
16 PrerenderDispatcherTest() {}
17
18 bool is_prerender_url(const GURL& url) const {
19 return prerender_dispatcher_.IsPrerenderURL(url);
20 }
21
22 const PrerenderDispatcher::PrerenderMap& urls() const {
23 return prerender_dispatcher_.prerender_urls_;
24 }
25
26 void AddURL(const GURL& url) { prerender_dispatcher_.OnAddPrerenderURL(url); }
27 void RemoveURL(const GURL& url) {
28 prerender_dispatcher_.OnRemovePrerenderURL(url);
29 }
30
31 int GetCountForURL(const GURL& url) const {
gavinp 2012/05/24 18:17:30 Good catch. That was a style violation, this is n
32 PrerenderDispatcher::PrerenderMap::const_iterator entry = urls().find(url);
33 if (entry == urls().end())
34 return 0;
35 EXPECT_GT(entry->second, 0);
36 return entry->second;
37 }
38
39 private:
40 PrerenderDispatcher prerender_dispatcher_;
41 DISALLOW_COPY_AND_ASSIGN(PrerenderDispatcherTest);
42 };
43
44 TEST_F(PrerenderDispatcherTest, PrerenderDispatcherEmpty) {
45 EXPECT_EQ(0U, urls().size());
46 }
47
48 TEST_F(PrerenderDispatcherTest, PrerenderDispatcherSingleAdd) {
49 GURL foo_url = GURL("http://foo.com");
50 EXPECT_FALSE(is_prerender_url(foo_url));
51 AddURL(foo_url);
52 EXPECT_TRUE(is_prerender_url(foo_url));
53 EXPECT_EQ(1, GetCountForURL(foo_url));
54 }
55
56 TEST_F(PrerenderDispatcherTest, PrerenderDispatcherMultipleAdd) {
57 GURL foo_url = GURL("http://foo.com");
58 GURL bar_url = GURL("http://bar.com");
59
60 EXPECT_FALSE(is_prerender_url(foo_url));
61 EXPECT_FALSE(is_prerender_url(bar_url));
62 AddURL(foo_url);
63 EXPECT_TRUE(is_prerender_url(foo_url));
64 EXPECT_FALSE(is_prerender_url(bar_url));
65
66 AddURL(foo_url);
67 EXPECT_TRUE(is_prerender_url(foo_url));
68 EXPECT_FALSE(is_prerender_url(bar_url));
69 EXPECT_EQ(2, GetCountForURL(foo_url));
70
71 AddURL(bar_url);
72 EXPECT_TRUE(is_prerender_url(foo_url));
73 EXPECT_TRUE(is_prerender_url(bar_url));
74 EXPECT_EQ(2, GetCountForURL(foo_url));
75 EXPECT_EQ(1, GetCountForURL(bar_url));
76 }
77
78 TEST_F(PrerenderDispatcherTest, PrerenderDispatcherSingleRemove) {
79 GURL foo_url = GURL("http://foo.com");
80 EXPECT_FALSE(is_prerender_url(foo_url));
81 AddURL(foo_url);
82 EXPECT_TRUE(is_prerender_url(foo_url));
83 RemoveURL(foo_url);
84 EXPECT_FALSE(is_prerender_url(foo_url));
85 EXPECT_EQ(0, GetCountForURL(foo_url));
86 }
87
88 TEST_F(PrerenderDispatcherTest, PrerenderDispatcherMultipleRemove) {
89 GURL foo_url = GURL("http://foo.com");
90 EXPECT_FALSE(is_prerender_url(foo_url));
91 AddURL(foo_url);
92 EXPECT_TRUE(is_prerender_url(foo_url));
93 AddURL(foo_url);
94 EXPECT_TRUE(is_prerender_url(foo_url));
95 EXPECT_EQ(2, GetCountForURL(foo_url));
96
97 RemoveURL(foo_url);
98 EXPECT_TRUE(is_prerender_url(foo_url));
99 EXPECT_EQ(1, GetCountForURL(foo_url));
100
101 RemoveURL(foo_url);
102 EXPECT_FALSE(is_prerender_url(foo_url));
103 EXPECT_EQ(0, GetCountForURL(foo_url));
104 }
105
106 TEST_F(PrerenderDispatcherTest, PrerenderDispatcherRemoveWithoutAdd) {
107 GURL foo_url = GURL("http://foo.com");
108 EXPECT_FALSE(is_prerender_url(foo_url));
109 RemoveURL(foo_url);
110 EXPECT_FALSE(is_prerender_url(foo_url));
111 EXPECT_EQ(0, GetCountForURL(foo_url));
112 }
113
114 TEST_F(PrerenderDispatcherTest, PrerenderDispatcherRemoveTooMany) {
115 GURL foo_url = GURL("http://foo.com");
116 EXPECT_FALSE(is_prerender_url(foo_url));
117 AddURL(foo_url);
118 EXPECT_TRUE(is_prerender_url(foo_url));
119 RemoveURL(foo_url);
120 EXPECT_FALSE(is_prerender_url(foo_url));
121 RemoveURL(foo_url);
122 EXPECT_FALSE(is_prerender_url(foo_url));
123 EXPECT_EQ(0, GetCountForURL(foo_url));
124 }
125
126 } // end namespace prerender
OLDNEW
« no previous file with comments | « chrome/renderer/prerender/prerender_dispatcher.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698