Reduce calling count of UpdateAppState()

chrome/browser/ui/browser.cc

Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index 2056b53aa9aa3ad4c4a564706690a838d6fa7325..f507cb4ae2db0196069e56b0826b6a558c25da6b 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -1914,6 +1914,11 @@ void Browser::ScheduleUIUpdate(const WebContents* source,
     // the navigation commands since those would have already been updated
     // synchronously by NavigationStateChanged.
     UpdateToolbar(false);
+
+    tab_strip_model_->UpdateWebContentsStateAt(
+        tab_strip_model_->GetIndexOfWebContents(source),
+        TabStripModelObserver::URL_CHANGING_ONLY);

[sky, 2013/09/11 02:09:58]

Your usage here is not entirely correct so that URL_CHANGING_ONLY becomes
misleading and error prone.

If BrowserStatusMonitor only cares about the url of the active tab it should be
able to monitor that directly.

+
     changed_flags &= ~content::INVALIDATE_TYPE_URL;
   }
   if (changed_flags & content::INVALIDATE_TYPE_LOAD) {