Added support for context menus to status icons.

chrome/browser/background_mode_manager.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
 #include "base/command_line.h"
+#include "chrome/app/chrome_dll_resource.h"
 #include "chrome/browser/background_mode_manager.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/metrics/user_metrics.h"
 #include "chrome/browser/pref_service.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/status_icons/status_icon.h"
 #include "chrome/browser/status_icons/status_tray.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/notification_service.h"
 #include "chrome/common/notification_type.h"
 #include "chrome/common/pref_names.h"
 #include "grit/browser_resources.h"
 #include "grit/chromium_strings.h"
+#include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 
 BackgroundModeManager::BackgroundModeManager(Profile* profile)
     : profile_(profile),
       background_app_count_(0),
       status_tray_(NULL) {
   // If background mode is disabled for unittests, just exit - don't listen for
   // any notifications.
   if (CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kDisableBackgroundMode))
@@ skipping 30 matching lines @@
 
   // Check for the presence of background apps after all extensions have been
   // loaded, to handle the case where an extension has been manually removed
   // while Chrome was not running.
   registrar_.Add(this, NotificationType::EXTENSIONS_READY,
                  Source<Profile>(profile));
 
 }
 
 BackgroundModeManager::~BackgroundModeManager() {
+  // If we're going away, remove our status tray icon so we don't get any events
+  // from it.
+  RemoveStatusTrayIcon();
 }
 
 bool BackgroundModeManager::IsBackgroundModeEnabled() {
   return profile_->GetPrefs()->GetBoolean(prefs::kBackgroundModeEnabled);
 }
 
 void BackgroundModeManager::Observe(NotificationType type,
                                     const NotificationSource& source,
                                     const NotificationDetails& details) {
   switch (type.value) {
@@ skipping 92 matching lines @@
     return;
   status_icon_ = status_tray_->CreateStatusIcon();
   if (!status_icon_)
     return;
 
   // Set the image and add ourselves as a click observer on it
   SkBitmap* bitmap = ResourceBundle::GetSharedInstance().GetBitmapNamed(
       IDR_STATUS_TRAY_ICON);
   status_icon_->SetImage(*bitmap);
   status_icon_->SetToolTip(l10n_util::GetStringUTF16(IDS_PRODUCT_NAME));
-  status_icon_->AddObserver(this);
+
+  // Create a context menu item for Chrome.
+  menus::SimpleMenuModel* menu = new menus::SimpleMenuModel(this);
+  menu->AddItem(IDC_ABOUT, l10n_util::GetStringFUTF16(IDS_ABOUT,
+      l10n_util::GetStringUTF16(IDS_PRODUCT_NAME)));
+  menu->AddSeparator();
+  menu->AddItemWithStringId(IDC_EXIT, IDS_EXIT);
+  status_icon_->SetContextMenu(menu);
+}
+
+bool BackgroundModeManager::IsCommandIdChecked(int command_id) const {
+  return false;
+}
+
+bool BackgroundModeManager::IsCommandIdEnabled(int command_id) const {
+  // For now, we do not support disabled items.
+  return true;
+}
+
+bool BackgroundModeManager::GetAcceleratorForCommandId(
+    int command_id,
+    menus::Accelerator* accelerator) {
+  // No accelerators for status icon context menus.
+  return false;
 }
 
 void BackgroundModeManager::RemoveStatusTrayIcon() {
   if (status_icon_)
     status_tray_->RemoveStatusIcon(status_icon_);
   status_icon_ = NULL;
 }
 
-void BackgroundModeManager::OnClicked() {
-  UserMetrics::RecordAction(UserMetricsAction("Exit"), profile_);
-  BrowserList::CloseAllBrowsersAndExit();
+
+void BackgroundModeManager::ExecuteCommand(int item) {
+  switch (item) {
+    case IDC_EXIT:
+      UserMetrics::RecordAction(UserMetricsAction("Exit"), profile_);
+      BrowserList::CloseAllBrowsersAndExit();
+      break;
+    case IDC_ABOUT: {
+      // Need to display a browser window to put up the about dialog.
+      Browser* browser = BrowserList::GetLastActive();
+      if (!browser) {
+        Browser::OpenEmptyWindow(profile_);
+        browser = BrowserList::GetLastActive();
+      }
+      browser->OpenAboutChromeDialog();
+      break;
+    }
+    default:
+      NOTREACHED();
+      break;
+  }
 }
 
 // static
 void BackgroundModeManager::RegisterUserPrefs(PrefService* prefs) {
   prefs->RegisterBooleanPref(prefs::kBackgroundModeEnabled, true);
 }