Chromium Code Reviews| Index: chrome/browser/ui/views/toolbar_view.cc |
| =================================================================== |
| --- chrome/browser/ui/views/toolbar_view.cc (revision 163351) |
| +++ chrome/browser/ui/views/toolbar_view.cc (working copy) |
| @@ -4,6 +4,7 @@ |
| #include "chrome/browser/ui/views/toolbar_view.h" |
| +#include "base/bind.h" |
| #include "base/i18n/number_formatting.h" |
| #include "base/utf_string_conversions.h" |
| #include "chrome/app/chrome_command_ids.h" |
| @@ -111,9 +112,6 @@ |
| const int kSearchTopLocationBarSpacing = 2; |
| const int kSearchToolbarSpacing = 5; |
| -// How often to show the disabled extension (sideload wipeout) bubble. |
| -const int kShowSideloadWipeoutBubbleMax = 3; |
| - |
| gfx::ImageSkia* kPopupBackgroundEdge = NULL; |
| // The omnibox border has some additional shadow, so we use less vertical |
| @@ -313,9 +311,6 @@ |
| location_bar_->Init(popup_parent_view); |
| show_home_button_.Init(prefs::kShowHomeButton, |
| browser_->profile()->GetPrefs(), this); |
| - sideload_wipeout_bubble_shown_.Init( |
| - prefs::kExtensionsSideloadWipeoutBubbleShown, |
| - browser_->profile()->GetPrefs(), NULL); |
| browser_actions_->Init(); |
| @@ -327,10 +322,9 @@ |
| l10n_util::GetStringUTF16(IDS_ACCNAME_TOOLTIP_FORWARD)); |
| } |
| - int bubble_shown_count = sideload_wipeout_bubble_shown_.GetValue(); |
| - if (bubble_shown_count < kShowSideloadWipeoutBubbleMax && |
| - DisabledExtensionsView::MaybeShow(browser_, app_menu_)) |
| - sideload_wipeout_bubble_shown_.SetValue(++bubble_shown_count); |
| + MessageLoop::current()->PostTask( |
|
sky
2012/10/23 16:57:09
Why do you need to delay this? I would rather you
|
| + FROM_HERE, |
| + base::Bind(&DisabledExtensionsView::MaybeShow, browser_, app_menu_)); |
| } |
| void ToolbarView::Update(WebContents* tab, bool should_restore_state) { |