| Index: chrome/browser/prerender/prerender_unittest.cc
|
| diff --git a/chrome/browser/prerender/prerender_unittest.cc b/chrome/browser/prerender/prerender_unittest.cc
|
| index 12cc4a8a1bfeb73bf15fcbba655bb431b1f6fda2..8ffb439667638213a7e500b324157cd80370132b 100644
|
| --- a/chrome/browser/prerender/prerender_unittest.cc
|
| +++ b/chrome/browser/prerender/prerender_unittest.cc
|
| @@ -18,7 +18,6 @@
|
| #include "chrome/browser/prerender/prerender_manager.h"
|
| #include "chrome/browser/prerender/prerender_origin.h"
|
| #include "chrome/common/chrome_switches.h"
|
| -#include "chrome/common/prerender_types.h"
|
| #include "chrome/test/base/testing_browser_process.h"
|
| #include "chrome/test/base/testing_profile.h"
|
| #include "content/public/browser/render_view_host.h"
|
| @@ -84,8 +83,6 @@
|
| int DummyPrerenderContents::g_next_route_id_ = 0;
|
|
|
| const gfx::Size kSize(640, 480);
|
| -
|
| -const uint32 kDefaultRelTypes = PrerenderRelTypePrerender;
|
|
|
| } // namespace
|
|
|
| @@ -379,9 +376,10 @@
|
| // true iff the prerender has been added to the PrerenderManager by the
|
| // PrerenderLinkManager and the PrerenderManager returned a handle.
|
| bool AddSimplePrerender(const GURL& url) {
|
| - prerender_link_manager()->OnAddPrerender(
|
| - kDefaultChildId, GetNextPrerenderID(), url, kDefaultRelTypes,
|
| - content::Referrer(), kSize, kDefaultRenderViewRouteId);
|
| + prerender_link_manager()->OnAddPrerender(kDefaultChildId,
|
| + GetNextPrerenderID(),
|
| + url, content::Referrer(),
|
| + kSize, kDefaultRenderViewRouteId);
|
| return LauncherHasRunningPrerender(kDefaultChildId, last_prerender_id());
|
| }
|
|
|
| @@ -721,7 +719,7 @@
|
| ORIGIN_GWS_PRERENDER,
|
| FINAL_STATUS_USED);
|
| prerender_link_manager()->OnAddPrerender(
|
| - child_id, GetNextPrerenderID(), pending_url, kDefaultRelTypes,
|
| + child_id, GetNextPrerenderID(), pending_url,
|
| Referrer(url, blink::WebReferrerPolicyDefault),
|
| kSize, route_id);
|
| EXPECT_FALSE(LauncherHasRunningPrerender(child_id, last_prerender_id()));
|
| @@ -762,7 +760,7 @@
|
| ORIGIN_GWS_PRERENDER,
|
| FINAL_STATUS_UNSUPPORTED_SCHEME);
|
| prerender_link_manager()->OnAddPrerender(
|
| - child_id, GetNextPrerenderID(), pending_url, kDefaultRelTypes,
|
| + child_id, GetNextPrerenderID(), pending_url,
|
| Referrer(url, blink::WebReferrerPolicyDefault),
|
| kSize, route_id);
|
| EXPECT_FALSE(LauncherHasRunningPrerender(child_id, last_prerender_id()));
|
| @@ -794,7 +792,7 @@
|
|
|
| // Schedule a pending prerender launched from the prerender.
|
| prerender_link_manager()->OnAddPrerender(
|
| - child_id, GetNextPrerenderID(), pending_url, kDefaultRelTypes,
|
| + child_id, GetNextPrerenderID(), pending_url,
|
| Referrer(url, blink::WebReferrerPolicyDefault),
|
| kSize, route_id);
|
| EXPECT_FALSE(LauncherHasRunningPrerender(child_id, last_prerender_id()));
|
| @@ -833,38 +831,9 @@
|
| prerender_manager()->CreateNextPrerenderContents(
|
| url,
|
| FINAL_STATUS_MANAGER_SHUTDOWN);
|
| - prerender_link_manager()->OnAddPrerender(
|
| - 100, GetNextPrerenderID(), url, kDefaultRelTypes, Referrer(), kSize, 200);
|
| + prerender_link_manager()->OnAddPrerender(100, GetNextPrerenderID(), url,
|
| + Referrer(), kSize, 200);
|
| EXPECT_FALSE(LauncherHasRunningPrerender(100, last_prerender_id()));
|
| -}
|
| -
|
| -// Tests that prerendering doesn't launch rel=next prerenders without the field
|
| -// trial.
|
| -TEST_F(PrerenderTest, NoRelNextByDefault) {
|
| - GURL url("http://www.google.com/");
|
| - prerender_manager()->CreateNextPrerenderContents(
|
| - url, FINAL_STATUS_MANAGER_SHUTDOWN);
|
| - DummyPrerenderContents* null = NULL;
|
| -
|
| - prerender_link_manager()->OnAddPrerender(
|
| - kDefaultChildId, GetNextPrerenderID(), url, PrerenderRelTypeNext,
|
| - Referrer(), kSize, kDefaultRenderViewRouteId);
|
| - EXPECT_EQ(null, prerender_manager()->FindEntry(url));
|
| -}
|
| -
|
| -// Tests that prerendering does launch rel=next prerenders with the field trial.
|
| -TEST_F(PrerenderTest, RelNextByFieldTrial) {
|
| - ASSERT_TRUE(base::FieldTrialList::CreateFieldTrial("PrerenderRelNextTrial",
|
| - "Yes"));
|
| - GURL url("http://www.google.com/");
|
| - DummyPrerenderContents* prerender_contents =
|
| - prerender_manager()->CreateNextPrerenderContents(
|
| - url, FINAL_STATUS_MANAGER_SHUTDOWN);
|
| -
|
| - prerender_link_manager()->OnAddPrerender(
|
| - kDefaultChildId, GetNextPrerenderID(), url, PrerenderRelTypeNext,
|
| - Referrer(), kSize, kDefaultRenderViewRouteId);
|
| - EXPECT_EQ(prerender_contents, prerender_manager()->FindAndUseEntry(url));
|
| }
|
|
|
| // Tests that prerendering is cancelled when we launch a second prerender of
|
| @@ -1422,9 +1391,10 @@
|
| ASSERT_TRUE(prerender_contents->GetRouteId(&route_id));
|
|
|
| GURL pending_url("http://www.neverlaunched.com");
|
| - prerender_link_manager()->OnAddPrerender(
|
| - child_id, GetNextPrerenderID(), pending_url, kDefaultRelTypes,
|
| - content::Referrer(), kSize, route_id);
|
| + prerender_link_manager()->OnAddPrerender(child_id,
|
| + GetNextPrerenderID(),
|
| + pending_url, content::Referrer(),
|
| + kSize, route_id);
|
| const int second_prerender_id = last_prerender_id();
|
|
|
| EXPECT_FALSE(IsEmptyPrerenderLinkManager());
|
|
|