[ios] Removed webDidFinishWithURL:loadSuccess: from CRWWebDelegate.

ios/chrome/browser/tabs/tab_unittest.mm

 // Copyright 2012 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.
 
 #import <UIKit/UIKit.h>
 
 #include <memory>
 
 #include "base/callback.h"
 #include "base/ios/block_types.h"
@@ skipping 225 matching lines @@
         rendererInitiated:YES];
     [tab_ webDidAddPendingURL];
     [[[(id)mock_web_controller_ expect]
         andReturnValue:OCMOCK_VALUE(kPageLoading)] loadPhase];
     [[tab_ navigationManager]->GetSessionController() commitPendingEntry];
     [[tab_ webController] webStateImpl]->OnNavigationCommitted(redirectUrl);
     [tab_ webDidStartLoadingURL:redirectUrl shouldUpdateHistory:YES];
     [tab_ webController:mock_web_controller_ titleDidChange:title];
     [[[(id)mock_web_controller_ expect]
         andReturnValue:OCMOCK_VALUE(kPageLoaded)] loadPhase];
-    [[tab_ webController] webStateImpl]->OnPageLoaded(redirectUrl, true);
-    [tab_ webDidFinishWithURL:redirectUrl loadSuccess:YES];
+    [tab_ webStateImpl]->OnPageLoaded(redirectUrl, true);
   }
 
   void BrowseToNewTab() {
     const GURL url(kNewTabUrl);
     // TODO(crbug.com/661992): This will not work with a mock CRWWebController.
     // The only test that uses it is currently disabled.
     web::NavigationManager::WebLoadParams params(url);
     params.transition_type = ui::PAGE_TRANSITION_TYPED;
     [[tab_ webController] loadWithParams:params];
     [[[(id)mock_web_controller_ expect]
         andReturnValue:OCMOCK_VALUE(kPageLoading)] loadPhase];
     [tab_ webDidStartLoadingURL:url shouldUpdateHistory:YES];
     [[[(id)mock_web_controller_ expect]
         andReturnValue:OCMOCK_VALUE(kPageLoaded)] loadPhase];
-    [tab_ webDidFinishWithURL:url loadSuccess:YES];
+    [tab_ webStateImpl]->OnPageLoaded(url, true);
     [tab_ webController:mock_web_controller_ titleDidChange:kNewTabTitle];
   }
 
   void QueryAllHistory(history::QueryResults* results) {
     base::CancelableTaskTracker tracker;
     base::RunLoop run_loop;
     scoped_refptr<HistoryQueryResultsObserver> observer(
         new HistoryQueryResultsObserver(&run_loop));
     history_service_->QueryHistory(
         base::string16(), history::QueryOptions(),
@@ skipping 277 matching lines @@
   EXPECT_FALSE([metadata shouldAutoOpenLinks]);
 }
 
 class TestRequestGroupID : public BlockCleanupTest {
  public:
   void SetUp() override { BlockCleanupTest::SetUp(); }
   void TearDown() override { BlockCleanupTest::TearDown(); }
 };
 
 }  // namespace