Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7348)

Unified Diff: chrome/browser/views/tabs/tab_strip.h

Issue 10761: Rewire the throbber so that the timer for updating lives on BrowserView, not ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 12 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/views/tabs/tab_strip.h
===================================================================
--- chrome/browser/views/tabs/tab_strip.h (revision 5492)
+++ chrome/browser/views/tabs/tab_strip.h (working copy)
@@ -92,6 +92,9 @@
// Retrieve the ideal bounds for the Tab at the specified index.
gfx::Rect GetIdealBounds(int index);
+ // Updates loading animations for the TabStrip.
+ void UpdateLoadingAnimations();
+
// views::View overrides:
virtual void PaintChildren(ChromeCanvas* canvas);
virtual views::View* GetViewByID(int id) const;
@@ -122,7 +125,6 @@
bool user_gesture);
virtual void TabMoved(TabContents* contents, int from_index, int to_index);
virtual void TabChangedAt(TabContents* contents, int index);
- virtual void TabValidateAnimations();
// Tab::Delegate implementation:
virtual bool IsTabSelected(const Tab* tab) const;
@@ -204,9 +206,6 @@
void AddMessageLoopObserver();
void RemoveMessageLoopObserver();
- // Called to update the frame of the Loading animations.
- void LoadingAnimationCallback();
-
// -- Link Drag & Drop ------------------------------------------------------
// Returns the bounds to render the drop at, in screen coordinates. Sets
@@ -290,9 +289,6 @@
// TODO(beng): (Cleanup) this would be better named "needs_resize_layout_".
bool resize_layout_scheduled_;
- // The timer used to update frames for the Loading Animation.
- base::RepeatingTimer<TabStrip> loading_animation_timer_;
-
// The "New Tab" button.
views::Button* newtab_button_;
gfx::Size newtab_button_size_;

Powered by Google App Engine
This is Rietveld 408576698