Chromium Code Reviews| Index: chrome/browser/ui/views/toolbar_view.cc |
| diff --git a/chrome/browser/ui/views/toolbar_view.cc b/chrome/browser/ui/views/toolbar_view.cc |
| index b619a347ba6e63a27155db5670dc1295e56f251c..39d66d29824428f1508b1f3dc4af60cb6325aee4 100644 |
| --- a/chrome/browser/ui/views/toolbar_view.cc |
| +++ b/chrome/browser/ui/views/toolbar_view.cc |
| @@ -61,6 +61,7 @@ |
| #if defined(OS_WIN) |
| #include "chrome/browser/enumerate_modules_model_win.h" |
| #include "chrome/browser/ui/views/critical_notification_bubble_view.h" |
| +#include "chrome/browser/ui/views/upgrade_bubble_view.h" |
| #if !defined(USE_AURA) |
| #include "chrome/browser/ui/views/app_menu_button_win.h" |
| #endif |
| @@ -192,6 +193,8 @@ ToolbarView::ToolbarView(Browser* browser) |
| #if defined(OS_WIN) |
| registrar_.Add(this, chrome::NOTIFICATION_CRITICAL_UPGRADE_INSTALLED, |
| content::NotificationService::AllSources()); |
| + registrar_.Add(this, chrome::NOTIFICATION_OUTDATED_INSTALL, |
| + content::NotificationService::AllSources()); |
| #endif |
| registrar_.Add(this, |
| chrome::NOTIFICATION_MODULE_INCOMPATIBILITY_BADGE_CHANGE, |
| @@ -553,6 +556,9 @@ void ToolbarView::Observe(int type, |
| case chrome::NOTIFICATION_CRITICAL_UPGRADE_INSTALLED: |
| ShowCriticalNotification(); |
| break; |
| + case chrome::NOTIFICATION_OUTDATED_INSTALL: |
| + ShowOutdatedInstallNotification(); |
|
Finnur
2012/12/06 19:53:16
So... Are we certain that we can not have both thi
MAD
2013/01/22 15:18:29
Thanks, I asked the PMs, but I guess the outdated
|
| + break; |
| #endif |
| default: |
| NOTREACHED(); |
| @@ -890,6 +896,12 @@ void ToolbarView::ShowCriticalNotification() { |
| #endif |
| } |
| +void ToolbarView::ShowOutdatedInstallNotification() { |
| +#if defined(OS_WIN) |
| + UpgradeBubbleView::ShowBubble(app_menu_, browser_); |
| +#endif |
| +} |
| + |
| void ToolbarView::UpdateAppMenuState() { |
| string16 accname_app = l10n_util::GetStringUTF16(IDS_ACCNAME_APP); |
| if (ShouldShowUpgradeRecommended()) { |