Add a flag to change when android's progress bar completes

chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java

Index: chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java
index bb41b481944a616d00982eb6fce612e8429f5d0c..e2e9a633b5618640f7ee0dc33ed5c446acb3ebec 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java
@@ -317,6 +317,8 @@ public class ToolbarManager implements ToolbarTabController, UrlFocusChangeListe
                     int statusCode) {
                 if (isNavigationToDifferentPage) {
                     mToolbar.onNavigatedToDifferentPage();
+                    mToolbar.startLoadProgress();
+                    updateLoadProgress(tab.getProgress());

[mdjones, 2016/06/20 20:58:40]

If I'm not mistaken, this should always happen after
onDidStartProvisionalLoadForFrame. Did something change where this wouldn't be
the case?

[Nate Chapin, 2016/06/21 21:06:28]

I think there may have been a bug fix between when I originally made this change
and now. I can't reproduce the "progress bar never reappears" issue I was
having. Dropped this change.

                 }
             }
 
@@ -1149,6 +1151,7 @@ public class ToolbarManager implements ToolbarTabController, UrlFocusChangeListe
 
         progress = Math.max(progress, MINIMUM_LOAD_PROGRESS);
         mToolbar.setLoadProgress(progress / 100f);
+        if (progress == 100) finishLoadProgress(true);

[mdjones, 2016/06/20 20:58:41]

Can updateLoadProgress hit 100 without triggering onLoadStopped?

[Nate Chapin, 2016/06/21 21:06:28]

On trunk, no, 100 is only triggered as part of onLoadStopped. With this change,
the experimental flags allow the progress bar to complete before onLoadStopped.
onLoadStopped will still be called eventually, but there could easily be a delay
of several seconds between the progress bar completing and onLoadStopped..

     }
 
     private void finishLoadProgress(boolean delayed) {