Remove WebContentsImpl::OnDidRedirectProvisionalLoad.

content/browser/web_contents/navigation_controller_impl_unittest.cc

Index: content/browser/web_contents/navigation_controller_impl_unittest.cc
diff --git a/content/browser/web_contents/navigation_controller_impl_unittest.cc b/content/browser/web_contents/navigation_controller_impl_unittest.cc
index f3b95609a6dad83ddb53b37a6995c470b20833d1..53db738fe67180e1d63ddbb2246cdad18eb17ffc 100644
--- a/content/browser/web_contents/navigation_controller_impl_unittest.cc
+++ b/content/browser/web_contents/navigation_controller_impl_unittest.cc
@@ -516,18 +516,10 @@ TEST_F(NavigationControllerTest, LoadURL_RedirectAbortCancelsPending) {
   EXPECT_EQ(-1, controller.GetLastCommittedEntryIndex());
   EXPECT_EQ(1, delegate->navigation_state_change_count());
 
-  // Now the navigation redirects.
+  // Now the navigation redirects.  The NavigationController no longer
+  // receives a notification of the redirect, so nothing happens here.
+  // See https://chromiumcodereview.appspot.com/10316020
   const GURL kRedirectURL("http://bee");
-  test_rvh()->OnMessageReceived(
-      ViewHostMsg_DidRedirectProvisionalLoad(0,  // routing_id
-                                             -1,  // pending page_id
-                                             GURL(),  // opener
-                                             kNewURL,  // old url
-                                             kRedirectURL));  // new url
-
-  // We don't want to change the NavigationEntry's url, in case it cancels.
-  // Prevents regression of http://crbug.com/77786.
-  EXPECT_EQ(kNewURL, controller.GetPendingEntry()->GetURL());
 
   // It may abort before committing, if it's a download or due to a stop or
   // a new navigation from the user.