Removed -[CRWWebDelegate webDidStartLoadingURL:updateHistory:].

ios/chrome/browser/tabs/tab.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 "ios/chrome/browser/tabs/tab.h"
 
 #import <CoreLocation/CoreLocation.h>
 #import <UIKit/UIKit.h>
 
 #include <utility>
@@ skipping 1385 matching lines @@
 
 - (void)webState:(web::WebState*)webState
     didFinishNavigation:(web::NavigationContext*)navigation {
   if (navigation->IsSamePage()) {
     auto* faviconDriver = favicon::WebFaviconDriver::FromWebState(webState);
     if (faviconDriver) {
       // Fetch the favicon for the new URL.
       faviconDriver->FetchFavicon(navigation->GetUrl());
     }
   }
+
+  if (!navigation->IsErrorPage()) {
+    [self addCurrentEntryToHistoryDB];
+    [self countMainFrameLoad];
+  }
+
   [parentTabModel_ notifyTabChanged:self];
 }
 
 // Records the state (scroll position, form values, whatever can be
 // harvested) from the current page into the current session entry.
 - (void)recordStateInHistory {
   // Link-loading prerender tab may not have correct zoom value during the load.
   if (!self.isLinkLoadingPrerenderTab)
     [self.webController recordStateInHistory];
 }
@@ skipping 309 matching lines @@
   // notification now that it's in the right place.
   NSString* URLSpec = base::SysUTF8ToNSString(lastCommittedURL.spec());
   if (URLSpec.length) {
     [[NSNotificationCenter defaultCenter]
         postNotificationName:kTabUrlStartedLoadingNotificationForCrashReporting
                       object:self
                     userInfo:@{kTabUrlKey : URLSpec}];
   }
 }
 
-// Called when the page URL has changed.
-- (void)webDidStartLoadingURL:(const GURL&)currentUrl
-          shouldUpdateHistory:(BOOL)updateHistory {
-  if (updateHistory) {
-    [self addCurrentEntryToHistoryDB];
-    [self countMainFrameLoad];
-  }
-}
-
 - (void)webState:(web::WebState*)webState
     didLoadPageWithSuccess:(BOOL)loadSuccess {
   DCHECK(self.webController.loadPhase == web::PAGE_LOADED);
 
   // Cancel prerendering if response is "application/octet-stream". It can be a
   // video file which should not be played from preload tab (crbug.com/436813).
   if (isPrerenderTab_ &&
       self.webState->GetContentsMimeType() == "application/octet-stream") {
     [delegate_ discardPrerender];
   }
@@ skipping 483 matching lines @@
 
 - (TabModel*)parentTabModel {
   return parentTabModel_;
 }
 
 - (FormInputAccessoryViewController*)inputAccessoryViewController {
   return inputAccessoryViewController_.get();
 }
 
 @end