Added -[Tab webState:didCommitNavigationWithDetails:].

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 1766 matching lines @@
       postNotificationName:
           kTabClosingCurrentDocumentNotificationForCrashReporting
                     object:self];
 }
 
 - (void)webCancelStartLoadingRequest {
   DCHECK(self.webController.loadPhase == web::PAGE_LOADED);
   [parentTabModel_ notifyTabChanged:self];
 }
 
-// Called when the page URL has changed.
-- (void)webDidStartLoadingURL:(const GURL&)currentUrl
-          shouldUpdateHistory:(BOOL)updateHistory {
-  DCHECK(self.webController.loadPhase == web::PAGE_LOADING);
+- (void)webState:(web::WebState*)webState
+    didCommitNavigationWithDetails:(const web::LoadCommittedDetails&)details {
   DCHECK([self navigationManager]);
-  // |webWillStartLoading:| is not called for native page loads.
+  // |webWillAddPendingURL:transition:| is not called for native page loads.
   // TODO(crbug.com/381201): Move this call there once that bug is fixed so that
   // |disableFullScreen| is called only from one place.
   [fullScreenController_ disableFullScreen];
   [findInPageController_ disableFindInPageWithCompletionHandler:nil];
-  [autoReloadBridge_ loadStartedForURL:currentUrl];
+  [autoReloadBridge_ loadStartedForURL:webState->GetLastCommittedURL()];
 
   if (isUserNavigationEvent_) {
     [[NSNotificationCenter defaultCenter]
         postNotificationName:kTabModelUserNavigatedNotification
                       object:self];
   }
   if (parentTabModel_) {
     [[NSNotificationCenter defaultCenter]
         postNotificationName:kTabModelTabWillStartLoadingNotification
                       object:parentTabModel_
-                    userInfo:[NSDictionary
-                                 dictionaryWithObject:self
-                                               forKey:kTabModelTabKey]];
+                    userInfo:@{kTabModelTabKey : self}];
   }
   favicon::FaviconDriver* faviconDriver =
-      favicon::WebFaviconDriver::FromWebState(self.webState);
+      favicon::WebFaviconDriver::FromWebState(webState);
   if (faviconDriver) {
-    faviconDriver->FetchFavicon(currentUrl);
+    faviconDriver->FetchFavicon(webState->GetLastCommittedURL());
   }
   [parentTabModel_ notifyTabChanged:self];
   if (parentTabModel_) {
     [[NSNotificationCenter defaultCenter]
         postNotificationName:kTabModelTabDidStartLoadingNotification
                       object:parentTabModel_
-                    userInfo:[NSDictionary
-                                 dictionaryWithObject:self
-                                               forKey:kTabModelTabKey]];
+                    userInfo:@{kTabModelTabKey : self}];
   }
+  [parentTabModel_ navigationCommittedInTab:self];
+}
 
-  [parentTabModel_ navigationCommittedInTab:self];
+// Called when the page URL has changed.
+- (void)webDidStartLoadingURL:(const GURL&)currentUrl
+          shouldUpdateHistory:(BOOL)updateHistory {
   if (updateHistory) {
     [self addCurrentEntryToHistoryDB];
     [self countMainFrameLoad];
   }
 
   // Sending a notification about the url change for crash reporting.
   // TODO(crbug.com/661675): Consider using the navigation entry committed
   // notification now that it's in the right place.
   NSString* urlString = base::SysUTF8ToNSString(currentUrl.spec());
   if ([urlString length]) {
@@ skipping 537 matching lines @@
 
 - (TabModel*)parentTabModel {
   return parentTabModel_;
 }
 
 - (FormInputAccessoryViewController*)inputAccessoryViewController {
   return inputAccessoryViewController_.get();
 }
 
 @end