WebContentsObserver update for PlzNavigate methods

content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java

Index: content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java b/content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java
index 0134c19986bc752476b0721c907d638e1b5b061c..cb188a89eb487f8714ae1159f13ae6ff2b63756d 100644
--- a/content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java
+++ b/content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsObserverProxy.java
@@ -77,10 +77,20 @@ class WebContentsObserverProxy extends WebContentsObserver {
 
     @Override
     @CalledByNative
-    public void didFinishNavigation(
-            boolean isMainFrame, boolean isErrorPage, boolean hasCommitted) {
+    public void didStartNavigation(String validatedUrl, boolean isMainFrame, boolean isErrorPage) {
         for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
-            mObserversIterator.next().didFinishNavigation(isMainFrame, isErrorPage, hasCommitted);
+            mObserversIterator.next().didStartNavigation(validatedUrl, isMainFrame, isErrorPage);
+        }
+    }
+
+    @Override
+    @CalledByNative
+    public void didFinishNavigation(String validatedUrl, boolean isMainFrame, boolean isErrorPage,

[Ted C, 2016/12/28 20:00:21]

let's keep the java names the same as the navigation handle ones as well.

validatedUrl = url
isMainFrame = isInMainFrame
isFragmentNavigation = isSamePage
isReload ~= pageTransition

Anyone that is currently using isNavigationToDifferentPage will need to do

if (isMainFrame && !isSamePage) {...}

The goal is to have anyone familiar with the C++ navigation handle to be able to
look at the java api and immediately know how to use stuff.

+            boolean hasCommitted, boolean isNavigationToDifferentPage, boolean isFragmentNavigation,
+            boolean isReload) {
+        for (mObserversIterator.rewind(); mObserversIterator.hasNext();) {
+            mObserversIterator.next().didFinishNavigation(validatedUrl, isMainFrame, isErrorPage,
+                    hasCommitted, isNavigationToDifferentPage, isFragmentNavigation, isReload);
         }
     }