Fix a bug where redirect chain gets lost on process swap.

chrome/browser/extensions/app_process_apitest.cc

Index: chrome/browser/extensions/app_process_apitest.cc
diff --git a/chrome/browser/extensions/app_process_apitest.cc b/chrome/browser/extensions/app_process_apitest.cc
index 8c897362636f616456be87f2723cb9f2aeb7c605..18fadc76429bfe70591ad77c992084e235d808d1 100644
--- a/chrome/browser/extensions/app_process_apitest.cc
+++ b/chrome/browser/extensions/app_process_apitest.cc
@@ -348,9 +348,21 @@ IN_PROC_BROWSER_TEST_F(AppApiTest, MAYBE_AppProcessRedirectBack) {
   browser()->NewTab();
   ui_test_utils::NavigateToURL(browser(), base_url.Resolve("path1/empty.html"));
   browser()->NewTab();
-  // Wait until the second tab finishes its redirect train (2 hops).
+  // Wait until the second tab finishes its redirect train (3 hops).
+  // 1. We navigate to redirect.html
+  // 2. Renderer navigates and finishes, counting as a load stop.
+  // 3. Renderer issues the meta refresh to navigate to server-redirect.
+  // 4. Renderer is now in a "provisional load", waiting for navigation to
+  //    complete.
+  // 5. Browser sees a redirect response from server-redirect to empty.html, and
+  //    transfers that to a new navigation, using RequestTransferURL.
+  // 6. We navigate to empty.html.
+  // 7. Renderer is still in a provisional load to server-redirect, so that is
+  //    cancelled, and counts as a load stop
+  // 8. Renderer navigates to empty.html, and finishes loading, counting as the
+  //    third load stop
   ui_test_utils::NavigateToURLBlockUntilNavigationsComplete(
-      browser(), base_url.Resolve("path1/redirect.html"), 2);
+      browser(), base_url.Resolve("path1/redirect.html"), 3);
 
   // 3 tabs, including the initial about:blank. The last 2 should be the same
   // process.
@@ -358,8 +370,7 @@ IN_PROC_BROWSER_TEST_F(AppApiTest, MAYBE_AppProcessRedirectBack) {
   EXPECT_EQ("/files/extensions/api_test/app_process/path1/empty.html",
             browser()->GetTabContentsAt(2)->controller().
                 GetLastCommittedEntry()->url().path());
-  RenderViewHost* host = browser()->GetTabContentsAt(1)->render_view_host();
-  EXPECT_EQ(host->process(),
+  EXPECT_EQ(browser()->GetTabContentsAt(1)->render_view_host()->process(),
             browser()->GetTabContentsAt(2)->render_view_host()->process());
 }