WebContentsObserver update for PlzNavigate methods

chrome/android/java/src/org/chromium/chrome/browser/dom_distiller/ReaderModeManager.java

Index: chrome/android/java/src/org/chromium/chrome/browser/dom_distiller/ReaderModeManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/dom_distiller/ReaderModeManager.java b/chrome/android/java/src/org/chromium/chrome/browser/dom_distiller/ReaderModeManager.java
index dde1407799026c24823577eb68948f6a91dd86bd..ab1a7cf5f67a2072e7a53715a6cacda441c05708 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/dom_distiller/ReaderModeManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/dom_distiller/ReaderModeManager.java
@@ -440,8 +440,8 @@ public class ReaderModeManager extends TabModelSelectorTabObserver
 
         return new WebContentsObserver(webContents) {
             @Override
-            public void didStartProvisionalLoadForFrame(long frameId, long parentFrameId,
-                    boolean isMainFrame, String validatedUrl, boolean isErrorPage) {
+            public void didStartNavigation(
+                    String validatedUrl, boolean isMainFrame, boolean isErrorPage) {
                 if (!isMainFrame) return;
                 // If there is a navigation in the current tab, hide the bar. It will show again
                 // once the distillability test is successful.
@@ -461,22 +461,22 @@ public class ReaderModeManager extends TabModelSelectorTabObserver
             }
 
             @Override
-            public void didNavigateMainFrame(String url, String baseUrl,
-                    boolean isNavigationToDifferentPage, boolean isNavigationInPage,
-                    int statusCode) {
+            public void didFinishNavigation(String validatedUrl, boolean isMainFrame,
+                    boolean isErrorPage, boolean hasCommitted, boolean isNavigationToDifferentPage,
+                    boolean isFragmentNavigation, boolean isReload) {
                 // TODO(cjhopman): This should possibly ignore navigations that replace the entry
                 // (like those from history.replaceState()).
-                if (isNavigationInPage) return;
-                if (DomDistillerUrlUtils.isDistilledPage(url)) return;
+                if (isFragmentNavigation) return;
+                if (DomDistillerUrlUtils.isDistilledPage(validatedUrl)) return;
 
                 // Make sure the tab was not destroyed.
                 ReaderModeTabInfo tabInfo = mTabStatusMap.get(readerTabId);
                 if (tabInfo == null) return;
 
                 tabInfo.setStatus(POSSIBLE);
-                if (!TextUtils.equals(url,
-                        DomDistillerUrlUtils.getOriginalUrlFromDistillerUrl(
-                                mReaderModePageUrl))) {
+                if (!TextUtils.equals(
+                            validatedUrl, DomDistillerUrlUtils.getOriginalUrlFromDistillerUrl(
+                                                  mReaderModePageUrl))) {
                     tabInfo.setStatus(NOT_POSSIBLE);
                     mIsUmaRecorded = false;
                 }