diff options
| author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-04 09:32:57 +0000 |
|---|---|---|
| committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-04 09:32:57 +0000 |
| commit | 19cd84b3cf2b0bc58837dbbe69bb0aea3d3ce841 (patch) | |
| tree | b97f2931485337498ac94602e2a6e19c24d3ba39 /chrome/browser/tabs | |
| parent | de21492cdd9d039bfb91742f2b33301df516e179 (diff) | |
| download | chromium_src-19cd84b3cf2b0bc58837dbbe69bb0aea3d3ce841.zip chromium_src-19cd84b3cf2b0bc58837dbbe69bb0aea3d3ce841.tar.gz chromium_src-19cd84b3cf2b0bc58837dbbe69bb0aea3d3ce841.tar.bz2 | |
Fix a regression that the sad tab favicon is not displayed when a tab crashes.
This regressed in my favicon/title updating change because the crashed state
is not updated on the codepath, since it comes from the tab.
The other approach would have been to make this call through the "update
everything" codepath for tbe state changing, but that's executed on a timer, and
it's nice to show the tab crash right away since the check is very inexpensive.
BUG=http://crbug.com/10756
Review URL: http://codereview.chromium.org/99341
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15201 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/tabs')
| -rw-r--r-- | chrome/browser/tabs/tab_strip_model.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/chrome/browser/tabs/tab_strip_model.h b/chrome/browser/tabs/tab_strip_model.h index 6f35d47..66c6b395 100644 --- a/chrome/browser/tabs/tab_strip_model.h +++ b/chrome/browser/tabs/tab_strip_model.h @@ -77,7 +77,9 @@ class TabStripModelObserver { // by the time this message is delivered. // // If only the loading state was updated, the loading_only flag should be - // specified. The tab model will update only the throbber and loading status. + // specified. The tab model will update only the throbber, loading status, + // and crashed state. + // // If other things change, set this flag to false to update all state, // including the title and favicon. This allows us to start/stop throbbing // without updating the title (which may be an ugly URL if the real title |