When a page is done loading, every tab change is important.

BUG=http://crbug.com/16087
TEST=as described in the bug

Review URL: http://codereview.chromium.org/149618

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20633 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 2 deletions

diff --git a/chrome/browser/cocoa/tab_strip_controller.mm b/chrome/browser/cocoa/tab_strip_controller.mm
index 4ad5f2c..2d37e5b 100644
--- a/chrome/browser/cocoa/tab_strip_controller.mm
+++ b/chrome/browser/cocoa/tab_strip_controller.mm
@@ -485,7 +485,7 @@ NSString* const kTabStripNumberOfTabsChanged = @"kTabStripNumberOfTabsChanged";
   // Update the current loading state, replacing the icon with a throbber, or
   // vice versa. This will get called repeatedly with the same state during a
   // load, so we need to make sure we're not creating the throbber view over and
-  // over.
+  // over. However, when the page is done, every state change is important.
   if (contents) {
     static NSImage* throbberWaitingImage =
         [nsimage_cache::ImageNamed(@"throbber_waiting.png") retain];
@@ -506,7 +506,7 @@ NSString* const kTabStripNumberOfTabsChanged = @"kTabStripNumberOfTabsChanged";
       throbberImage = throbberLoadingImage;
     }
 
-    if (oldState != newState) {
+    if (oldState != newState || newState == kTabDone) {
       NSView* iconView = nil;
       if (newState == kTabDone) {
         iconView = [self favIconImageViewForContents:contents];