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_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 325 matching lines @@
 
     EXPECT_TRUE(ntp_rvh->IsSwappedOut());
     return ntp_rvh;
   }
 
   NavigationRequest* NavigationRequestForRenderFrameManager(
       RenderFrameHostManager* manager) const {
     return manager->navigation_request_for_testing();
   }
 
+  RenderFrameHostImpl* SpeculativeRenderFrameHostForRenderFrameManager(

[nasko, 2014/07/23 14:15:46]

nit: this and the method above are returning things out, so prefixing with Get*
will make it clearer.

[clamy, 2014/07/23 14:56:10]

Done.

+      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 1468 matching lines @@
   }
 }
 
 // Browser-side navigation: Test that a proper NavigationRequest is created by
 // BeginNavigation.
 TEST_F(RenderFrameHostManagerTest, BrowserSideNavigationBeginNavigation) {
   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.

[nasko, 2014/07/23 14:15:46]

Why can't we do it now?

[clamy, 2014/07/23 14:56:10]

Because we don't have a proper commit of the navigation yet.

[nasko, 2014/07/24 09:55:47]

Ok

   // 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(
+      kUrl2, Referrer(), PAGE_TRANSITION_LINK, std::string());
   subframe_rfh->SendBeginNavigationWithURL(kUrl2);
   NavigationRequest* subframe_request =
       NavigationRequestForRenderFrameManager(
           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(
         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
+// 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 =
+      NavigationRequestForRenderFrameManager(
+        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.
+  NavigationCommitInfo commit_info;
+  commit_info.navigation_url = kUrl;
+  main_test_rfh()->frame_tree_node()->render_manager()->CommitNavigation(
+      commit_info);
+  main_request =NavigationRequestForRenderFrameManager(

[nasko, 2014/07/23 14:15:46]

nit: space after =

[clamy, 2014/07/23 14:56:10]

Done.

+      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 the same url.

[nasko, 2014/07/23 14:15:46]

It should be reused if we commit in the same SiteInstance, regardless of what
the URL is. For example, going to docs.google.com, which bounces to
accounts.google.com for auth should end up committing in the google.com
renderer.

[clamy, 2014/07/23 14:56:10]

I now test that it works as long as we commit a url in the same site instance.

+TEST_F(RenderFrameHostManagerTest,
+       BrowserSideNavigationSpeculativeRendererSameURL) {
+  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 =
+      NavigationRequestForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  ASSERT_TRUE(main_request);
+  RenderFrameHostImpl* speculative_rfh =
+      SpeculativeRenderFrameHostForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  EXPECT_TRUE(speculative_rfh != NULL);
+  EXPECT_NE(main_test_rfh(), speculative_rfh);
+
+  // Commit the same url.
+  NavigationCommitInfo commit_info;
+  commit_info.navigation_url = kUrl2;
+  main_test_rfh()->frame_tree_node()->render_manager()->CommitNavigation(
+      commit_info);
+  main_request = NavigationRequestForRenderFrameManager(
+        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 =
+      NavigationRequestForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  ASSERT_TRUE(main_request);
+  RenderFrameHostImpl* speculative_rfh =
+      SpeculativeRenderFrameHostForRenderFrameManager(
+        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.
+  NavigationCommitInfo commit_info;
+  commit_info.navigation_url = kUrl3;
+  main_test_rfh()->frame_tree_node()->render_manager()->CommitNavigation(
+      commit_info);
+  main_request = NavigationRequestForRenderFrameManager(
+        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);

[nasko, 2014/07/23 14:15:46]

Let's also assert that there is no longer speculative RFH associated with the
RFHM.

[clamy, 2014/07/23 14:56:10]

Done.

+}
+
+// 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 =
+      NavigationRequestForRenderFrameManager(
+        main_test_rfh()->frame_tree_node()->render_manager());
+  ASSERT_TRUE(main_request);
+  RenderFrameHostImpl* speculative_rfh =
+      SpeculativeRenderFrameHostForRenderFrameManager(
+        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.
+  NavigationCommitInfo commit_info;
+  commit_info.navigation_url = kUrl1;
+  main_test_rfh()->frame_tree_node()->render_manager()->CommitNavigation(
+      commit_info);
+  main_request = NavigationRequestForRenderFrameManager(
+        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);

[nasko, 2014/07/23 14:15:46]

Same as above, it is not testing that the speculative RFH is indeed discarded.

[clamy, 2014/07/23 14:56:10]

Done.

+}
+
 }  // namespace content