PlzNavigate: implement RequestNavigation in the no live renderer case

content/browser/frame_host/render_frame_host_manager_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/strings/utf_string_conversions.h"
+#include "base/time/time.h"
 #include "content/browser/frame_host/cross_site_transferring_request.h"
+#include "content/browser/frame_host/navigation_before_commit_info.h"
 #include "content/browser/frame_host/navigation_controller_impl.h"
 #include "content/browser/frame_host/navigation_entry_impl.h"
 #include "content/browser/frame_host/navigation_request.h"
 #include "content/browser/frame_host/navigator.h"
+#include "content/browser/frame_host/navigator_impl.h"
 #include "content/browser/frame_host/render_frame_host_manager.h"
 #include "content/browser/site_instance_impl.h"
 #include "content/browser/webui/web_ui_controller_factory_registry.h"
 #include "content/common/view_messages.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_widget_host_iterator.h"
 #include "content/public/browser/web_contents_delegate.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "content/public/browser/web_ui_controller.h"
 #include "content/public/common/bindings_policy.h"
+#include "content/public/common/content_switches.h"
 #include "content/public/common/javascript_message_type.h"
 #include "content/public/common/page_transition_types.h"
 #include "content/public/common/url_constants.h"
 #include "content/public/common/url_utils.h"
 #include "content/public/test/mock_render_process_host.h"
 #include "content/public/test/test_notification_tracker.h"
 #include "content/test/test_content_browser_client.h"
 #include "content/test/test_content_client.h"
 #include "content/test/test_render_view_host.h"
 #include "content/test/test_web_contents.h"
@@ skipping 320 matching lines @@
     // BeforeUnload finishes.
     ntp_rvh->SendBeforeUnloadACK(true);
 
     dest_rvh->SendNavigate(101, kDestUrl);
     ntp_rvh->OnSwappedOut(false);
 
     EXPECT_TRUE(ntp_rvh->IsSwappedOut());
     return ntp_rvh;
   }
 
-  NavigationRequest* NavigationRequestForRenderFrameManager(
+  NavigationRequest* GetNavigationRequestForRenderFrameManager(
       RenderFrameHostManager* manager) const {
     return manager->navigation_request_for_testing();
   }
 
+  RenderFrameHostImpl* GetSpeculativeRenderFrameHostForRenderFrameManager(
+      RenderFrameHostManager* manager) {
+    return manager->speculative_render_frame_host_.get();
+  }
+
+  void EnableBrowserSideNavigation() {
+    CommandLine::ForCurrentProcess()->AppendSwitch(
+        switches::kEnableBrowserSideNavigation);
+  }
  private:
   RenderFrameHostManagerTestWebUIControllerFactory factory_;
   scoped_ptr<FrameLifetimeConsistencyChecker> lifetime_checker_;
 };
 
 // Tests that when you navigate from a chrome:// url to another page, and
 // then do that same thing in another tab, that the two resulting pages have
 // different SiteInstances, BrowsingInstances, and RenderProcessHosts. This is
 // a regression test for bug 9364.
 TEST_F(RenderFrameHostManagerTest, NewTabPageProcesses) {
@@ skipping 1462 matching lines @@
     static_cast<SiteInstanceImpl*>(
         pending_rfh->GetSiteInstance())->increment_active_view_count();
 
     main_test_rfh()->OnMessageReceived(
         FrameHostMsg_BeforeUnload_ACK(0, false, now, now));
     EXPECT_FALSE(contents()->cross_navigation_pending());
     EXPECT_FALSE(rvh_deleted_observer.deleted());
   }
 }
 
 // Browser-side navigation: Test that a proper NavigationRequest is created by

[Charlie Reis, 2014/07/30 23:37:33]

// PlzNavigate:

[clamy, 2014/08/05 11:52:44]

Done.

 // BeginNavigation.
 TEST_F(RenderFrameHostManagerTest, BrowserSideNavigationBeginNavigation) {

[Charlie Reis, 2014/07/30 23:37:33]

As before, let's settle on something consistent to make it easier for others to
follow.  PlzNavigateBeginNavigation?

(here and in the tests below)

[clamy, 2014/08/05 11:52:44]

This was the initial name, but I was asked by jam@ to change it to something
that did not feature the project name, such as BrowserSideNavigation (which is
now the prefix for all the related test).

   const GURL kUrl1("http://www.google.com/");
   const GURL kUrl2("http://www.chromium.org/");
   const GURL kUrl3("http://www.gmail.com/");
 
+  // TODO(clamy): we should be enabling browser side navigations here
+  // eventually.

[Charlie Reis, 2014/07/30 23:37:33]

Mention why we have to do it later for now, since it's not clear at the moment.

[clamy, 2014/08/05 11:52:44]

Done.

   // Navigate to the first page.
   contents()->NavigateAndCommit(kUrl1);
   TestRenderViewHost* rvh1 = test_rvh();
   EXPECT_EQ(RenderViewHostImpl::STATE_DEFAULT, rvh1->rvh_state());
 
+  EnableBrowserSideNavigation();
   // Add a subframe.
   TestRenderFrameHost* subframe_rfh = static_cast<TestRenderFrameHost*>(
       contents()->GetFrameTree()->AddFrame(
           contents()->GetFrameTree()->root(), 14, "Child"));
 
   // Simulate a BeginNavigation IPC on the subframe.
+  subframe_rfh->frame_tree_node()->navigator()->GetController()->LoadURL(

[Charlie Reis, 2014/07/30 23:37:33]

The main frame and subframe share the same NavigationController.  Why do we need
this here and contents()->GetController().LoadURL below?

[clamy, 2014/08/05 11:52:44]

This is a hack to get a pending_navigation_entry in the controller, which we
need in BeginNavigation. It should eventually go away.

[Charlie Reis, 2014/08/05 23:59:51]

Ok.  I was just wondering why we had different ways of getting to it, so it
looks good now.

+      kUrl2, Referrer(), PAGE_TRANSITION_LINK, std::string());
   subframe_rfh->SendBeginNavigationWithURL(kUrl2);
   NavigationRequest* subframe_request =
-      NavigationRequestForRenderFrameManager(
+      GetNavigationRequestForRenderFrameManager(
           subframe_rfh->frame_tree_node()->render_manager());
   ASSERT_TRUE(subframe_request);
   EXPECT_EQ(kUrl2, subframe_request->info_for_testing().navigation_params.url);
   // First party for cookies url should be that of the main frame.
   EXPECT_EQ(
       kUrl1, subframe_request->info_for_testing().first_party_for_cookies);
   EXPECT_FALSE(subframe_request->info_for_testing().is_main_frame);
   EXPECT_TRUE(subframe_request->info_for_testing().parent_is_main_frame);
 
   // Simulate a BeginNavigation IPC on the main frame.
+  contents()->GetController().LoadURL(
+      kUrl3, Referrer(), PAGE_TRANSITION_LINK, std::string());
   main_test_rfh()->SendBeginNavigationWithURL(kUrl3);
   NavigationRequest* main_request =
-      NavigationRequestForRenderFrameManager(
+      GetNavigationRequestForRenderFrameManager(
         main_test_rfh()->frame_tree_node()->render_manager());
   ASSERT_TRUE(main_request);
   EXPECT_EQ(kUrl3, main_request->info_for_testing().navigation_params.url);
   EXPECT_EQ(kUrl3, main_request->info_for_testing().first_party_for_cookies);
   EXPECT_TRUE(main_request->info_for_testing().is_main_frame);
   EXPECT_FALSE(main_request->info_for_testing().parent_is_main_frame);
 }
 
+// Browser side navigation: Test that RequestNavigation creates a

[Charlie Reis, 2014/07/30 23:37:33]

// PlzNavigate:
(here and in all the tests below)

[clamy, 2014/08/05 11:52:44]

Done.

+// NavigationRequest and that the RenderFrameHost is initialized to handle the
+// request when it commits. Also ensures that the RenderFrameHost is not
+// modified when the navigation commits.
+TEST_F(RenderFrameHostManagerTest,
+       BrowserSideNavigationRequestNavigationNoLiveRenderer) {
+  const GURL kUrl("http://www.google.com/");
+
+  EnableBrowserSideNavigation();
+  EXPECT_FALSE(main_test_rfh()->render_view_host()->IsRenderViewLive());
+  contents()->GetController().LoadURL(
+      kUrl, Referrer(), PAGE_TRANSITION_LINK, std::string());
+  NavigationRequest* main_request =
+      GetNavigationRequestForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  // A NavigationRequest should have been generated.
+  EXPECT_TRUE(main_request != NULL);
+  // The RFH should have been initialized to handle the navigation when it
+  // commits.
+  EXPECT_TRUE(main_test_rfh()->render_view_host()->IsRenderViewLive());
+  RenderFrameHostImpl* rfh = main_test_rfh();
+
+  // Now commit the same url.
+  NavigationBeforeCommitInfo commit_info;
+  commit_info.navigation_url = kUrl;
+  main_test_rfh()->frame_tree_node()->render_manager()->CommitNavigation(
+      commit_info);
+  main_request = GetNavigationRequestForRenderFrameManager(
+      main_test_rfh()->frame_tree_node()->render_manager());
+
+  // The main rfh should not have been changed and we should not have a
+  // navigation request anymore.
+  EXPECT_EQ(rfh, main_test_rfh());
+  EXPECT_TRUE(main_request == NULL);
+}
+
+// Browser-side navigation: Test that a speculative renderer is spawned when
+// navigating to a different site instance than the one of the main render frame
+// host. Test that it is used if the navigation commits with a url pertaining to
+// the same site instance.
+TEST_F(RenderFrameHostManagerTest,
+       BrowserSideNavigationSpeculativeRendererSameURL) {
+  const GURL kUrl1("http://www.chromium.org/");
+  const GURL kUrl2("http://www.google.com/");
+  const GURL kUrl3("http://docs.google.com/");
+
+  // TODO(clamy): we should be enabling browser side navigations here
+  // eventually.
+  // Navigate to the first page.
+  contents()->NavigateAndCommit(kUrl1);
+  TestRenderViewHost* rvh1 = test_rvh();
+  EXPECT_EQ(RenderViewHostImpl::STATE_DEFAULT, rvh1->rvh_state());
+
+  EnableBrowserSideNavigation();
+  // Navigate to a different site.
+  contents()->GetController().LoadURL(
+      kUrl2, Referrer(), PAGE_TRANSITION_LINK, std::string());
+  main_test_rfh()->SendBeginNavigationWithURL(kUrl2);
+  NavigationRequest* main_request =
+      GetNavigationRequestForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  ASSERT_TRUE(main_request);
+  RenderFrameHostImpl* speculative_rfh =
+      GetSpeculativeRenderFrameHostForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  EXPECT_TRUE(speculative_rfh != NULL);
+  EXPECT_NE(main_test_rfh(), speculative_rfh);
+
+  // Commit a url of the same site instance.
+  NavigationBeforeCommitInfo commit_info;
+  commit_info.navigation_url = kUrl3;
+  main_test_rfh()->frame_tree_node()->render_manager()->CommitNavigation(
+      commit_info);
+  main_request = GetNavigationRequestForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  EXPECT_EQ(main_test_rfh(), speculative_rfh);
+  EXPECT_TRUE(main_request == NULL);
+}
+
+// Browser-side navigation: Test that a speculative renderer is spawned when
+// navigating to a different site instance than the one of the main render frame
+// host. Test that it is discarded if the navigation commits with a different
+// url.
+TEST_F(RenderFrameHostManagerTest,
+       BrowserSideNavigationSpeculativeRendererDifferentURL) {
+  const GURL kUrl1("http://www.google.com/");
+  const GURL kUrl2("http://www.chromium.org/");
+  const GURL kUrl3("http://www.gmail.com/");
+
+  // TODO(clamy): we should be enabling browser side navigations here
+  // eventually.
+  // Navigate to the first page.
+  contents()->NavigateAndCommit(kUrl1);
+  TestRenderViewHost* rvh1 = test_rvh();
+  EXPECT_EQ(RenderViewHostImpl::STATE_DEFAULT, rvh1->rvh_state());
+
+  EnableBrowserSideNavigation();
+  // Navigate to a different site.
+  contents()->GetController().LoadURL(
+      kUrl2, Referrer(), PAGE_TRANSITION_LINK, std::string());
+  main_test_rfh()->SendBeginNavigationWithURL(kUrl2);
+  NavigationRequest* main_request =
+      GetNavigationRequestForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  ASSERT_TRUE(main_request);
+  RenderFrameHostImpl* speculative_rfh =
+      GetSpeculativeRenderFrameHostForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  RenderFrameHostImpl* current_rfh = main_test_rfh();
+  EXPECT_TRUE(speculative_rfh != NULL);
+  EXPECT_NE(main_test_rfh(), speculative_rfh);
+
+  // Commit a different url.
+  NavigationBeforeCommitInfo commit_info;
+  commit_info.navigation_url = kUrl3;
+  main_test_rfh()->frame_tree_node()->render_manager()->CommitNavigation(
+      commit_info);
+  main_request = GetNavigationRequestForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  speculative_rfh = GetSpeculativeRenderFrameHostForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  EXPECT_NE(main_test_rfh(), speculative_rfh);
+  EXPECT_NE(main_test_rfh(), current_rfh);
+  EXPECT_TRUE(main_request == NULL);
+  EXPECT_TRUE(speculative_rfh == NULL);
+}
+
+// Browser-side navigation: Test that a speculative renderer is spawned when
+// navigating to a different site instance than the one of the main render frame
+// host. Test that it is discarded and the old render frame host is used if the
+// navigation commits with the url from the previous render frame host.
+TEST_F(RenderFrameHostManagerTest,
+       BrowserSideNavigationSpeculativeRendererOldURL) {
+  const GURL kUrl1("http://www.google.com/");
+  const GURL kUrl2("http://www.chromium.org/");
+
+  // TODO(clamy): we should be enabling browser side navigations here
+  // eventually.
+  // Navigate to the first page.
+  contents()->NavigateAndCommit(kUrl1);
+  TestRenderViewHost* rvh1 = test_rvh();
+  EXPECT_EQ(RenderViewHostImpl::STATE_DEFAULT, rvh1->rvh_state());
+
+  EnableBrowserSideNavigation();
+  // Navigate to a different site.
+  contents()->GetController().LoadURL(
+      kUrl2, Referrer(), PAGE_TRANSITION_LINK, std::string());
+  main_test_rfh()->SendBeginNavigationWithURL(kUrl2);
+  NavigationRequest* main_request =
+      GetNavigationRequestForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  ASSERT_TRUE(main_request);
+  RenderFrameHostImpl* speculative_rfh =
+      GetSpeculativeRenderFrameHostForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  RenderFrameHostImpl* current_rfh = main_test_rfh();
+  EXPECT_TRUE(speculative_rfh != NULL);
+  EXPECT_NE(main_test_rfh(), speculative_rfh);
+
+  // Commit a different url.
+  NavigationBeforeCommitInfo commit_info;
+  commit_info.navigation_url = kUrl1;
+  main_test_rfh()->frame_tree_node()->render_manager()->CommitNavigation(
+      commit_info);
+  main_request = GetNavigationRequestForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  speculative_rfh = GetSpeculativeRenderFrameHostForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  EXPECT_NE(main_test_rfh(), speculative_rfh);
+  EXPECT_EQ(main_test_rfh(), current_rfh);
+  EXPECT_TRUE(main_request == NULL);
+  EXPECT_TRUE(speculative_rfh == NULL);
+}
+
 }  // namespace content