Better distinguish didFinishLoad and didStopLoading

chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerChrome.java

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.compositor.layouts;
 
 import android.content.Context;
 import android.graphics.Bitmap;
 import android.text.TextUtils;
 import android.view.ViewGroup;
@@ skipping 236 matching lines @@
             }
         };
         selector.addObserver(mTabModelSelectorObserver);
         selector.setCloseAllTabsDelegate(this);
 
         mTabModelObserver = createTabModelObserver();
         for (TabModel model : selector.getModels()) model.addObserver(mTabModelObserver);
 
         mTabSelectorTabObserver = new TabModelSelectorTabObserver(selector) {
             @Override
-            public void onLoadStarted(Tab tab) {
-                tabLoadStarted(tab.getId(), tab.isIncognito());
+            public void onLoadStarted(Tab tab, boolean toDifferentDocument) {
+                if (toDifferentDocument) tabLoadStarted(tab.getId(), tab.isIncognito());

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

same comment

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

Done.

             }
 
             @Override
             public void onLoadStopped(Tab tab) {
                 tabLoadFinished(tab.getId(), tab.isIncognito());
             }
 
             @Override
             public void onPageLoadStarted(Tab tab, String url) {
                 tabPageLoadStarted(tab.getId(), tab.isIncognito());
@@ skipping 476 matching lines @@
 
     /**
      * @param id The id of the {@link Tab} to search for.
      * @return   A {@link Tab} instance or {@code null} if it could be found.
      */
     protected Tab getTabById(int id) {
         TabModelSelector selector = getTabModelSelector();
         return selector == null ? null : selector.getTabById(id);
     }
 }