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

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

Issue 11459003: Add Start/Stop event signalling on Prerenders. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: match up to webkit side of patch... Created 8 years 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 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 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/renderer/prerender/prerender_dispatcher.h" 5 #include "chrome/renderer/prerender/prerender_dispatcher.h"
6 6
7 #include <map> 7 #include <map>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 18 matching lines...) Expand all
29 // WebKit signalling, we can expect PrerenderBrowserTest to provide adequate 29 // WebKit signalling, we can expect PrerenderBrowserTest to provide adequate
30 // coverage of this. 30 // coverage of this.
31 class PrerenderDispatcherTest : public testing::Test { 31 class PrerenderDispatcherTest : public testing::Test {
32 public: 32 public:
33 PrerenderDispatcherTest() {} 33 PrerenderDispatcherTest() {}
34 34
35 bool IsPrerenderURL(const GURL& url) const { 35 bool IsPrerenderURL(const GURL& url) const {
36 return prerender_dispatcher_.IsPrerenderURL(url); 36 return prerender_dispatcher_.IsPrerenderURL(url);
37 } 37 }
38 38
39 const std::map<int, GURL>& prerenders() const { 39 const std::map<int, WebPrerender>& prerenders() const {
40 return prerender_dispatcher_.prerenders_; 40 return prerender_dispatcher_.prerenders_;
41 } 41 }
42 42
43 int StartPrerender(const GURL& url) { 43 int StartPrerender(const GURL& url) {
44 std::pair<std::map<int, GURL>::iterator, bool> insert_result = 44 std::pair<std::map<int, WebPrerender>::iterator, bool> insert_result =
45 prerender_dispatcher_.prerenders_.insert( 45 prerender_dispatcher_.prerenders_.insert(
46 std::make_pair(g_next_prerender_id, url)); 46 std::make_pair(g_next_prerender_id, WebPrerender()));
47 CHECK(insert_result.second); 47 CHECK(insert_result.second);
48 48
49 prerender_dispatcher_.OnPrerenderStart(g_next_prerender_id); 49 prerender_dispatcher_.OnPrerenderStart(g_next_prerender_id);
50 prerender_dispatcher_.OnPrerenderAddAlias(g_next_prerender_id, url);
50 return g_next_prerender_id++; 51 return g_next_prerender_id++;
51 } 52 }
52 53
53 void AddAliasToPrerender(int prerender_id, const GURL& url) { 54 void AddAliasToPrerender(int prerender_id, const GURL& url) {
54 prerender_dispatcher_.OnPrerenderAddAlias(prerender_id, url); 55 prerender_dispatcher_.OnPrerenderAddAlias(prerender_id, url);
55 } 56 }
56 57
57 void StopPrerender(int prerender_id) { 58 void StopPrerender(int prerender_id) {
58 prerender_dispatcher_.OnPrerenderStop(prerender_id); 59 prerender_dispatcher_.OnPrerenderStop(prerender_id);
59 } 60 }
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
140 int foo_id = StartPrerender(foo_url); 141 int foo_id = StartPrerender(foo_url);
141 EXPECT_TRUE(IsPrerenderURL(foo_url)); 142 EXPECT_TRUE(IsPrerenderURL(foo_url));
142 StopPrerender(foo_id); 143 StopPrerender(foo_id);
143 EXPECT_FALSE(IsPrerenderURL(foo_url)); 144 EXPECT_FALSE(IsPrerenderURL(foo_url));
144 StopPrerender(foo_id); 145 StopPrerender(foo_id);
145 EXPECT_FALSE(IsPrerenderURL(foo_url)); 146 EXPECT_FALSE(IsPrerenderURL(foo_url));
146 EXPECT_EQ(0, GetCountForURL(foo_url)); 147 EXPECT_EQ(0, GetCountForURL(foo_url));
147 } 148 }
148 149
149 } // end namespace prerender 150 } // end namespace prerender
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698