Remove AboutToNavigateRenderFrame, issue custom notification for DevTools.

chrome/browser/ui/sync/one_click_signin_sync_observer_unittest.cc

Index: chrome/browser/ui/sync/one_click_signin_sync_observer_unittest.cc
diff --git a/chrome/browser/ui/sync/one_click_signin_sync_observer_unittest.cc b/chrome/browser/ui/sync/one_click_signin_sync_observer_unittest.cc
index 2cab4fc233e39d10c1ce95a51aab708cd76d8aa9..e19e7c37006e7cf440e174cfa9e9bba345ffba2b 100644
--- a/chrome/browser/ui/sync/one_click_signin_sync_observer_unittest.cc
+++ b/chrome/browser/ui/sync/one_click_signin_sync_observer_unittest.cc
@@ -39,8 +39,8 @@ class MockWebContentsObserver : public content::WebContentsObserver {
   // to the continue URL. Navigations in unit_tests never complete, but a
   // navigation start is a sufficient signal for the purposes of this test.
   // Listening for this call also has the advantage of being synchronous.
-  MOCK_METHOD2(AboutToNavigateRenderFrame, void(content::RenderFrameHost*,
-                                                content::RenderFrameHost*));
+  MOCK_METHOD2(DidStartNavigationToPendingEntry,
+               void(const GURL&, content::NavigationController::ReloadType));
 };
 
 class OneClickTestProfileSyncService : public TestProfileSyncService {
@@ -172,7 +172,7 @@ TEST_F(OneClickSigninSyncObserverTest, NoSyncService_RedirectsImmediately) {
               profile(), BuildNullService));
 
   // The observer should immediately redirect to the continue URL.
-  EXPECT_CALL(*web_contents_observer_, AboutToNavigateRenderFrame(_, _));
+  EXPECT_CALL(*web_contents_observer_, DidStartNavigationToPendingEntry(_, _));
   CreateSyncObserver(kContinueUrl);
   EXPECT_EQ(GURL(kContinueUrl), web_contents()->GetVisibleURL());
 
@@ -185,7 +185,7 @@ TEST_F(OneClickSigninSyncObserverTest, NoSyncService_RedirectsImmediately) {
 // firing, the observer cleans up its memory without loading the continue URL.
 TEST_F(OneClickSigninSyncObserverTest, WebContentsDestroyed) {
   EXPECT_CALL(*web_contents_observer_,
-              AboutToNavigateRenderFrame(_, _)).Times(0);
+              DidStartNavigationToPendingEntry(_, _)).Times(0);
   CreateSyncObserver(kContinueUrl);
   SetContents(NULL);
 }
@@ -198,7 +198,7 @@ TEST_F(OneClickSigninSyncObserverTest,
   sync_service_->set_first_setup_in_progress(false);
   sync_service_->set_sync_active(true);
 
-  EXPECT_CALL(*web_contents_observer_, AboutToNavigateRenderFrame(_, _));
+  EXPECT_CALL(*web_contents_observer_, DidStartNavigationToPendingEntry(_, _));
   sync_service_->NotifyObservers();
   EXPECT_EQ(GURL(kContinueUrl), web_contents()->GetVisibleURL());
 }
@@ -212,7 +212,7 @@ TEST_F(OneClickSigninSyncObserverTest,
   sync_service_->set_sync_active(false);
 
   EXPECT_CALL(*web_contents_observer_,
-              AboutToNavigateRenderFrame(_, _)).Times(0);
+              DidStartNavigationToPendingEntry(_, _)).Times(0);
   sync_service_->NotifyObservers();
   EXPECT_NE(GURL(kContinueUrl), web_contents()->GetVisibleURL());
 }
@@ -226,7 +226,7 @@ TEST_F(OneClickSigninSyncObserverTest,
   sync_service_->set_sync_active(false);
 
   EXPECT_CALL(*web_contents_observer_,
-              AboutToNavigateRenderFrame(_, _)).Times(0);
+              DidStartNavigationToPendingEntry(_, _)).Times(0);
   sync_service_->NotifyObservers();
   EXPECT_NE(GURL(kContinueUrl), web_contents()->GetVisibleURL());
 
@@ -245,7 +245,7 @@ TEST_F(OneClickSigninSyncObserverTest,
   sync_service_->set_sync_active(true);
 
   EXPECT_CALL(*web_contents_observer_,
-              AboutToNavigateRenderFrame(_, _)).Times(0);
+              DidStartNavigationToPendingEntry(_, _)).Times(0);
   sync_service_->NotifyObservers();
   EXPECT_NE(GURL(kContinueUrl), web_contents()->GetVisibleURL());
 }