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

Unified Diff: content/browser/frame_host/render_frame_host_manager_unittest.cc

Issue 701953006: PlzNavigate: Speculatively spawns a renderer process for navigations. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase and one more round of CR work. Created 6 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/frame_host/render_frame_host_manager_unittest.cc
diff --git a/content/browser/frame_host/render_frame_host_manager_unittest.cc b/content/browser/frame_host/render_frame_host_manager_unittest.cc
index bbe0aec7afc58e0a33cb6177bb8535694dd486a7..4c8fddaadcd3ff6aa57be50158da954c96590059 100644
--- a/content/browser/frame_host/render_frame_host_manager_unittest.cc
+++ b/content/browser/frame_host/render_frame_host_manager_unittest.cc
@@ -901,7 +901,7 @@ TEST_F(RenderFrameHostManagerTest, Navigate) {
EXPECT_FALSE(manager->pending_frame_host());
// Commit.
- manager->DidNavigateFrame(host, true);
+ manager->DidNavigateFrame(host, true, false);
// Commit to SiteInstance should be delayed until RenderFrame commit.
EXPECT_TRUE(host == manager->current_frame_host());
ASSERT_TRUE(host);
@@ -922,7 +922,7 @@ TEST_F(RenderFrameHostManagerTest, Navigate) {
EXPECT_FALSE(manager->pending_frame_host());
// Commit.
- manager->DidNavigateFrame(host, true);
+ manager->DidNavigateFrame(host, true, false);
EXPECT_TRUE(host == manager->current_frame_host());
ASSERT_TRUE(host);
EXPECT_TRUE(host->GetSiteInstance()->HasSite());
@@ -943,7 +943,7 @@ TEST_F(RenderFrameHostManagerTest, Navigate) {
notifications.Reset();
// Commit.
- manager->DidNavigateFrame(manager->pending_frame_host(), true);
+ manager->DidNavigateFrame(manager->pending_frame_host(), true, false);
EXPECT_TRUE(host == manager->current_frame_host());
ASSERT_TRUE(host);
EXPECT_TRUE(host->GetSiteInstance()->HasSite());
@@ -993,7 +993,7 @@ TEST_F(RenderFrameHostManagerTest, WebUI) {
EXPECT_TRUE(manager->web_ui());
// Commit.
- manager->DidNavigateFrame(host, true);
+ manager->DidNavigateFrame(host, true, false);
EXPECT_TRUE(
host->render_view_host()->GetEnabledBindings() & BINDINGS_POLICY_WEB_UI);
}
@@ -1031,7 +1031,7 @@ TEST_F(RenderFrameHostManagerTest, WebUIInNewTab) {
host1->render_view_host()->GetEnabledBindings() & BINDINGS_POLICY_WEB_UI);
// Commit and ensure we still have bindings.
- manager1->DidNavigateFrame(host1, true);
+ manager1->DidNavigateFrame(host1, true, false);
SiteInstance* webui_instance = host1->GetSiteInstance();
EXPECT_EQ(host1, manager1->current_frame_host());
EXPECT_TRUE(
@@ -1060,7 +1060,7 @@ TEST_F(RenderFrameHostManagerTest, WebUIInNewTab) {
EXPECT_TRUE(
host2->render_view_host()->GetEnabledBindings() & BINDINGS_POLICY_WEB_UI);
- manager2->DidNavigateFrame(host2, true);
+ manager2->DidNavigateFrame(host2, true, false);
}
// Tests that we don't end up in an inconsistent state if a page does a back and
@@ -1523,7 +1523,7 @@ TEST_F(RenderFrameHostManagerTest, NoSwapOnGuestNavigations) {
EXPECT_EQ(manager->current_frame_host()->GetSiteInstance(), instance);
// Commit.
- manager->DidNavigateFrame(host, true);
+ manager->DidNavigateFrame(host, true, false);
// Commit to SiteInstance should be delayed until RenderFrame commit.
EXPECT_EQ(host, manager->current_frame_host());
ASSERT_TRUE(host);
@@ -1544,7 +1544,7 @@ TEST_F(RenderFrameHostManagerTest, NoSwapOnGuestNavigations) {
EXPECT_FALSE(manager->pending_frame_host());
// Commit.
- manager->DidNavigateFrame(host, true);
+ manager->DidNavigateFrame(host, true, false);
EXPECT_EQ(host, manager->current_frame_host());
ASSERT_TRUE(host);
EXPECT_EQ(host->GetSiteInstance(), instance);
@@ -1584,7 +1584,7 @@ TEST_F(RenderFrameHostManagerTest, NavigateWithEarlyClose) {
notifications.Reset();
// Commit.
- manager->DidNavigateFrame(host, true);
+ manager->DidNavigateFrame(host, true, false);
// Commit to SiteInstance should be delayed until RenderFrame commits.
EXPECT_EQ(host, manager->current_frame_host());
@@ -1856,7 +1856,7 @@ TEST_F(RenderFrameHostManagerTest, DetachPendingChild) {
EXPECT_FALSE(manager->pending_frame_host());
// Commit.
- manager->DidNavigateFrame(host, true);
+ manager->DidNavigateFrame(host, true, false);
// Commit to SiteInstance should be delayed until RenderFrame commit.
EXPECT_TRUE(host == manager->current_frame_host());
ASSERT_TRUE(host);

Powered by Google App Engine
This is Rietveld 408576698