Navigation transitions: Place transition page in same process as destination page.

content/browser/transition_browsertest.cc

 // Copyright 2014 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/bind.h"
 #include "content/browser/loader/cross_site_resource_handler.h"
 #include "content/browser/loader/resource_dispatcher_host_impl.h"
 #include "content/browser/loader/resource_request_info_impl.h"
 #include "content/browser/transition_request_manager.h"
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/browser/web_contents_observer.h"
+#include "content/public/test/browser_test_utils.h"
 #include "content/public/test/content_browser_test.h"
 #include "content/public/test/content_browser_test_utils.h"
 #include "content/public/test/test_utils.h"
 #include "content/shell/browser/shell.h"
 #include "content/shell/browser/shell_resource_dispatcher_host_delegate.h"
 #include "net/test/embedded_test_server/embedded_test_server.h"
 #include "net/url_request/url_request.h"
 
 namespace content {
 
@@ skipping 87 matching lines @@
       new TransitionBrowserTestObserver(shell()->web_contents()));
 
   ResourceDispatcherHost::Get()->SetDelegate(observer.get());
   observer->set_pending_transition_request(true);
 
   NavigateToURL(shell(), embedded_test_server()->GetURL("/title1.html"));
 
   ASSERT_TRUE(observer->did_defer_response());
 }
 
+// This tests that the renderer is reused between the outgoing and transition.
+IN_PROC_BROWSER_TEST_F(TransitionBrowserTest,
+                       TransitionNavigationSharesRenderer) {
+  ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
+
+  NavigateToURL(shell(), embedded_test_server()->GetURL("/title1.html"));
+
+  int outgoing_process_id =
+      shell()->web_contents()->GetRenderProcessHost()->GetID();
+
+  WebContents::CreateParams create_params(
+      shell()->web_contents()->GetBrowserContext(),
+      shell()->web_contents()->GetSiteInstance());
+  scoped_ptr<WebContents> transition_web_contents(
+      WebContents::Create(create_params));
+
+  NavigationController::LoadURLParams params(GURL(url::kAboutBlankURL));
+  transition_web_contents->GetController().LoadURLWithParams(params);
+  transition_web_contents->Focus();
+
+  WaitForLoadStop(transition_web_contents.get());
+
+  int transition_process_id =
+      transition_web_contents->GetRenderProcessHost()->GetID();
+
+  ASSERT_EQ(outgoing_process_id, transition_process_id);

[Charlie Reis, 2014/07/25 17:31:22]

nit: Use EXPECT_* for things we're testing (so that the test continues to run
and might print multiple EXPECT failures) and save ASSERT_* for test
preconditions (e.g., the embedded_test_server init above).

[shatch, 2014/07/25 20:57:06]

Done.

+}
+
 }  // namespace content