Remove onLoadStopped/onLoadStarted from WebContentsDelegateAndroid

components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java

Index: components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java
diff --git a/components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java b/components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java
index 48022a86b07a75c865ae20c70a8953621846ed0a..5897fa2dfeef5bdbc97b9b01f77f379db7399aef 100644
--- a/components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java
+++ b/components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java
@@ -54,11 +54,7 @@ public class WebContentsDelegateAndroid {
     }
 
     @CalledByNative
-    public void onLoadStarted(boolean toDifferentDocument) {
-    }
-
-    @CalledByNative
-    public void onLoadStopped() {
+    public void loadingStateChanged(boolean toDifferentDocument) {

[mdjones, 2016/08/03 20:27:41]

Do we want to pass WebContents in here or should we just assume that the
implementor has access? This is more a question of consistency rather than
necessity; all the existing cases already work.

[Ted C, 2016/08/03 20:39:38]

I was conflicted on that as well.  I opted for being consistent with the other
delegate methods that do not pass the source web contents up.  We pass it when
we create a new web contents, but the majority of other places don't pass the
source up.  I think if we do it here, we should do it everywhere, and that is
out of scope IMO.

     }
 
     @CalledByNative