Better distinguish didFinishLoad and didStopLoading

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 3d74cb934588cbd29a339a7ba551c25aa2a8e1b3..fbfb8b9605c402dfbefe15e9d09fb2d111942701 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
@@ -132,9 +132,9 @@ public class OverlayPanelContent {
 
         mWebContentsDelegate = new WebContentsDelegateAndroid() {
             @Override
-            public void onLoadStarted() {
-                super.onLoadStarted();
-                mProgressObserver.onProgressBarStarted();
+            public void onLoadStarted(boolean toDifferentDocument) {
+                super.onLoadStarted(toDifferentDocument);
+                if (toDifferentDocument) mProgressObserver.onProgressBarStarted();

[Ted C, 2015/10/28 23:41:02]

this wasn't guarded before by toDifferentDocument, so I'm hesitant to say you
should here.

And if we do guard on it, I think we need to do the same in onLoadStopped below.
 Otherwise you'll get mismatched calls.

[Nate Chapin, 2015/10/30 20:41:34]

Yeah, I saw "onProgressBarStarted" and went into auto-pilot. if() dropped.

             }
 
             @Override