Rewire the throbber so that the timer for updating lives on BrowserView, not ...

chrome/browser/browser.cc

Index: chrome/browser/browser.cc
===================================================================
--- chrome/browser/browser.cc	(revision 5492)
+++ chrome/browser/browser.cc	(working copy)
@@ -394,6 +394,10 @@
   return l10n_util::GetStringF(IDS_BROWSER_WINDOW_TITLE_FORMAT, title);
 }
 
+bool Browser::IsCurrentPageLoading() const {
+  return GetSelectedTabContents()->is_loading();

[sky, 2008/11/14 22:07:21]

Should no null check this?

+}
+
 // static
 void Browser::FormatTitleForDisplay(std::wstring* title) {
   size_t current_index = 0;
@@ -1333,12 +1337,6 @@
   }
 }
 
-void Browser::ValidateLoadingAnimations() {
-  // TODO(beng): Remove this, per http://crbug.com/3297
-  if (window_)
-    window_->ValidateThrobber();
-}
-
 void Browser::CloseFrameAfterDragSession() {
   // This is scheduled to run after we return to the message loop because
   // otherwise the frame will think the drag session is still active and ignore
@@ -1684,8 +1682,7 @@
 }
 
 void Browser::LoadingStateChanged(TabContents* source) {
-  tabstrip_model_.UpdateTabContentsLoadingAnimations();
-
+  window_->UpdateLoadingAnimations(tabstrip_model_.TabsAreLoading());
   window_->UpdateTitleBar();
 
   // Let the go button know that it should change appearance if possible.