PlzNavigate: Chrome UI changes for new methods of WebContentsObserver

chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelContent.java

Index: chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelContent.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelContent.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelContent.java
index 46c665da527748b0209233f4c6db3940d18c4517..499dcc9c8361aa5fe433e38502c769293ca24aed 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelContent.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/bottombar/OverlayPanelContent.java
@@ -299,22 +299,25 @@ public class OverlayPanelContent {
                     }
 
                     @Override
-                    public void didStartProvisionalLoadForFrame(long frameId, long parentFrameId,
-                            boolean isMainFrame, String validatedUrl, boolean isErrorPage) {
-                        if (isMainFrame) {
-                            mContentDelegate.onMainFrameLoadStarted(validatedUrl,
-                                    !TextUtils.equals(validatedUrl, mLoadedUrl));
+                    public void didStartNavigation(
+                            String url, boolean isInMainFrame, boolean isErrorPage) {
+                        if (isInMainFrame) {

[jam, 2017/02/07 05:10:32]

same comment ToolbarManager.java, I think yo ualso want to check !isSamePage to
maintain same behavior as before

[shaktisahu, 2017/02/08 01:00:46]

Done.

+                            mContentDelegate.onMainFrameLoadStarted(
+                                    url, !TextUtils.equals(url, mLoadedUrl));
                         }
                     }
 
                     @Override
-                    public void didNavigateMainFrame(String url, String baseUrl,
-                            boolean isNavigationToDifferentPage, boolean isNavigationInPage,
-                            int httpResultCode) {
-                        mIsProcessingPendingNavigation = false;
-                        mContentDelegate.onMainFrameNavigation(url,
-                                !TextUtils.equals(url, mLoadedUrl),
-                                isHttpFailureCode(httpResultCode));
+                    public void didFinishNavigation(String url, boolean isInMainFrame,
+                            boolean isErrorPage, boolean hasCommitted, boolean isSamePage,
+                            boolean isFragmentNavigation, Integer pageTransition, int errorCode,
+                            int httpStatusCode) {
+                        if (hasCommitted && isInMainFrame) {
+                            mIsProcessingPendingNavigation = false;
+                            mContentDelegate.onMainFrameNavigation(url,
+                                    !TextUtils.equals(url, mLoadedUrl),
+                                    isHttpFailureCode(httpStatusCode));
+                        }
                     }
                 };