Implemented WebStateObserver::DidFinishNavigation(NavigationHandle*).

ios/web/public/web_state/web_state_observer.h

 // Copyright 2014 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_WEB_STATE_OBSERVER_H_
 #define IOS_WEB_PUBLIC_WEB_STATE_WEB_STATE_OBSERVER_H_
 
 #include <stddef.h>
 
 #include <string>
 #include <vector>
 
 #include "base/macros.h"
 
 class GURL;
 
 namespace web {
 
 struct Credential;
 struct FaviconURL;
+class NavigationContext;
 struct LoadCommittedDetails;
 class WebState;
 class TestWebState;
 class WebStateImpl;
 
 enum class PageLoadCompletionStatus : bool { SUCCESS = 0, FAILURE = 1 };
 
 // An observer API implemented by classes which are interested in various page
 // load events from WebState.
 class WebStateObserver {
@@ skipping 10 matching lines @@
   // This method is invoked when a navigation item has changed.
   virtual void NavigationItemChanged() {}
 
   // This method is invoked when a new non-pending navigation item is created.
   // This corresponds to one NavigationManager item being created
   // (in the case of new navigations) or renavigated to (for back/forward
   // navigations).
   virtual void NavigationItemCommitted(
       const LoadCommittedDetails& load_details) {}
 
+  // Called when a navigation finished in the WebState. This happens when a
+  // navigation is committed, aborted or replaced by a new one. To know if the
+  // navigation has resulted in an error page, use
+  // NavigationHandle::IsErrorPage.

[rohitrao (ping after 24h), 2017/02/24 01:22:48]

What is "NavigationHandle"?  I don't see any other references to it in this
file.

Oh, is this the original name of NavigationContext?  Then this comment just
needs to be updated?

[Eugene But (OOO till 7-30), 2017/02/24 01:49:41]

Good catch. Done.

+  //
+  // If this is called because the navigation committed, then the document load
+  // will still be ongoing in the WebState returned by |navigation_handle|.

[rohitrao (ping after 24h), 2017/02/24 01:22:48]

And here.

[Eugene But (OOO till 7-30), 2017/02/24 01:49:41]

Done.

+  // Use the document loads events such as DidStopLoading
+  // and related methods to listen for continued events from this
+  // WebState.
+  //
+  // Note that this is fired by same-page navigations, such as fragment
+  // navigations or pushState/replaceState, which will not result in a document
+  // change. To filter these out, use NavigationHandle::IsSamePage.

[rohitrao (ping after 24h), 2017/02/24 01:22:48]

And here.

[Eugene But (OOO till 7-30), 2017/02/24 01:49:41]

Done.

+  //
+  // Note that |navigation_handle| will be destroyed at the end of this call,
+  // so do not keep a reference to it afterward.
+  virtual void DidFinishNavigation(NavigationContext* navigation_context) {}
+
   // Called when the current page has started loading.
   virtual void DidStartLoading() {}
 
   // Called when the current page has stopped loading.
   virtual void DidStopLoading() {}
 
   // Called when the current page is loaded.
   virtual void PageLoaded(PageLoadCompletionStatus load_completion_status) {}
 
   // Called when the interstitial is dismissed by the user.
   virtual void InterstitialDismissed() {}
 
   // Called on URL hash change events.
+  // TODO(crbug.com/692331): Remove this method and use |DidFinishNavigation|.
   virtual void UrlHashChanged() {}
 
   // Called on history state change events.
+  // TODO(crbug.com/692331): Remove this method and use |DidFinishNavigation|.
   virtual void HistoryStateChanged() {}
 
   // Notifies the observer that the page has made some progress loading.
   // |progress| is a value between 0.0 (nothing loaded) to 1.0 (page fully
   // loaded).
   virtual void LoadProgressChanged(double progress) {}
 
   // Called on form submission. |user_initiated| is true if the user
   // interacted with the page.
   virtual void DocumentSubmitted(const std::string& form_name,
@@ skipping 89 matching lines @@
   void ResetWebState();
 
   WebState* web_state_;
 
   DISALLOW_COPY_AND_ASSIGN(WebStateObserver);
 };
 
 }  // namespace web
 
 #endif  // IOS_WEB_PUBLIC_WEB_STATE_WEB_STATE_OBSERVER_H_