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

Unified Diff: chrome/browser/tabs/tab_strip_model.h

Issue 60066: Make the throbber throb sooner after you navigate. This fixes the new tab pag... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 11 years, 9 months 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/tabs/tab_strip_model.h
===================================================================
--- chrome/browser/tabs/tab_strip_model.h (revision 13028)
+++ chrome/browser/tabs/tab_strip_model.h (working copy)
@@ -70,7 +70,8 @@
// The specified TabContents at |index| changed in some way. |contents| may
// be an entirely different object and the old value is no longer available
// by the time this message is delivered.
- virtual void TabChangedAt(TabContents* contents, int index) { }
+ virtual void TabChangedAt(TabContents* contents, int index,
+ bool loading_only) { }
// The TabStripModel now no longer has any "significant" (user created or
// user manipulated) tabs. The implementer may use this as a trigger to try
// and close the window containing the TabStripModel, for example...
@@ -286,8 +287,9 @@
int GetIndexOfController(const NavigationController* controller) const;
// Notify any observers that the TabContents at the specified index has
- // changed in some way.
- void UpdateTabContentsStateAt(int index);
+ // changed in some way. Loading only specifies whether only the loading state
+ // has changed.
+ void UpdateTabContentsStateAt(int index, bool loading_only);
// Make sure there is an auto-generated New Tab tab in the TabStripModel.
// If |force_create| is true, the New Tab will be created even if the

Powered by Google App Engine
This is Rietveld 408576698