Implemented WebStateObserver::DidFinishNavigation(NavigationHandle*).

ios/web/public/web_state/ui/crw_web_delegate.h

 // 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.
 
 #ifndef IOS_WEB_PUBLIC_WEB_STATE_UI_CRW_WEB_DELEGATE_H_
 #define IOS_WEB_PUBLIC_WEB_STATE_UI_CRW_WEB_DELEGATE_H_
 
 #include <stdint.h>
 #import <UIKit/UIKit.h>
 #include <vector>
@@ skipping 29 matching lines @@
 // updating the navigation manager's pending entry.
 // Phase will be LOAD_REQUESTED.
 - (void)webWillAddPendingURL:(const GURL&)url
                   transition:(ui::PageTransition)transition;
 // Called when webWillStartLoadingURL was called, but something went wrong, and
 // webDidStartLoadingURL will now never be called.
 - (void)webCancelStartLoadingRequest;
 // Called when the page URL has changed. Phase will be PAGE_LOADING. Can be
 // followed by webDidFinishWithURL or webWillStartLoadingURL.
 // |updateHistory| is YES if the URL should be added to the history DB.
-// TODO(stuartmorgan): Remove or rename the history param; the history DB
-// isn't a web concept, so this shoud be expressed differently.
+// TODO(crbug.com/692331): Remove this method and use |DidFinishNavigation|.
 - (void)webDidStartLoadingURL:(const GURL&)url
           shouldUpdateHistory:(BOOL)updateHistory;
 // Called when the page load was cancelled by page activity (before a success /
 // failure state is known). Phase will be PAGE_LOADED.
 - (void)webLoadCancelled:(const GURL&)url;
 // Called when a page updates its history stack using pushState or replaceState.
-// TODO(stuartmorgan): Generalize this to cover any change of URL without page
-// document change.
+// TODO(crbug.com/692331): Remove this method and use |DidFinishNavigation|.
 - (void)webDidUpdateHistoryStateWithPageURL:(const GURL&)pageUrl;
 // Called when a placeholder image should be displayed instead of the WebView.
 - (void)webController:(CRWWebController*)webController
     retrievePlaceholderOverlayImage:(void (^)(UIImage*))block;
 
 // ---------------------------------------------------------------------
 // TODO(rohitrao): Eliminate as many of the following delegate methods as
 // possible.  They only exist because the Tab and CRWWebController logic was
 // very intertwined. We should streamline the logic to jump between classes
 // less, then remove any delegate method that becomes unneccessary as a result.
 
 // Called when the page is reloaded.
 - (void)webWillReload;
 // Called when a page is loaded using loadWithParams.  In
 // |webWillInitiateLoadWithParams|, the |params| argument is non-const so that
 // the delegate can make changes if necessary.
 // TODO(rohitrao): This is not a great API.  Clean it up.
 - (void)webWillInitiateLoadWithParams:
     (web::NavigationManager::WebLoadParams&)params;
 - (void)webDidUpdateSessionForLoadWithParams:
             (const web::NavigationManager::WebLoadParams&)params
                         wasInitialNavigation:(BOOL)initialNavigation;
 // Called from finishHistoryNavigationFromEntry.
+// TODO(crbug.com/692331): Remove this method and use |DidFinishNavigation|.
 - (void)webWillFinishHistoryNavigationFromEntry:(CRWSessionEntry*)fromEntry;
 // ---------------------------------------------------------------------
 
 // Called when |webController| wants to open a new window. |URL| is the URL of
 // the new window; |openerURL| is the URL of the page which requested a window
 // to be open; |initiatedByUser| is YES if action was caused by the user.
 // |webController| will not open a window if this method returns nil. This
 // method can not return |webController|.
 - (CRWWebController*)webController:(CRWWebController*)webController
          createWebControllerForURL:(const GURL&)URL
@@ skipping 56 matching lines @@
 // Called when a PassKit file is downloaded. |data| should be the data from a
 // PassKit file, but this is not guaranteed, and the delegate is responsible for
 // error handling non PassKit data using -[PKPass initWithData:error:]. If the
 // download does not successfully complete, |data| will be nil.
 - (void)webController:(CRWWebController*)webController
     didLoadPassKitObject:(NSData*)data;
 
 @end
 
 #endif  // IOS_WEB_PUBLIC_WEB_STATE_UI_CRW_WEB_DELEGATE_H_