Abstract media galleries context menu code and use it in scan result dialog.

chrome/browser/media_galleries/media_galleries_dialog_controller.cc

 // Copyright (c) 2012 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 "chrome/browser/media_galleries/media_galleries_dialog_controller.h"
 
 #include "base/base_paths.h"
 #include "base/path_service.h"
 #include "base/stl_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/api/file_system/file_system_api.h"
 #include "chrome/browser/media_galleries/media_file_system_registry.h"
 #include "chrome/browser/media_galleries/media_galleries_histograms.h"
+#include "chrome/browser/media_galleries/media_gallery_context_menu.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/storage_monitor/storage_info.h"
 #include "chrome/browser/storage_monitor/storage_monitor.h"
 #include "chrome/browser/ui/chrome_select_file_policy.h"
 #include "chrome/common/extensions/permissions/media_galleries_permission.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_view.h"
 #include "extensions/browser/extension_prefs.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/permissions/permissions_data.h"
@@ skipping 15 matching lines @@
   if (a.allowed && !b.allowed)
     return true;
   if (!a.allowed && b.allowed)
     return false;
 
   return a.pref_info.AbsolutePath() < b.pref_info.AbsolutePath();
 }
 
 }  // namespace
 
-class GalleryContextMenuModel : public ui::SimpleMenuModel::Delegate {
- public:
-  explicit GalleryContextMenuModel(MediaGalleriesDialogController* controller)
-      : controller_(controller), id_(kInvalidMediaGalleryPrefId) {}
-  virtual ~GalleryContextMenuModel() {}
-
-  void set_media_gallery_pref_id(MediaGalleryPrefId id) {
-    id_ = id;
-  }
-
-  virtual bool IsCommandIdChecked(int command_id) const OVERRIDE {
-    return false;
-  }
-  virtual bool IsCommandIdEnabled(int command_id) const OVERRIDE {
-    return true;
-  }
-  virtual bool IsCommandIdVisible(int command_id) const OVERRIDE {
-    return true;
-  }
-
-  virtual bool GetAcceleratorForCommandId(
-      int command_id, ui::Accelerator* accelerator) OVERRIDE {
-    return false;
-  }
-
-  virtual void ExecuteCommand(int command_id, int event_flags) OVERRIDE {
-    controller_->DidForgetGallery(id_);
-  }
-
- private:
-  MediaGalleriesDialogController* controller_;
-  MediaGalleryPrefId id_;
-};
-
 MediaGalleriesDialogController::MediaGalleriesDialogController(
     content::WebContents* web_contents,
     const Extension& extension,
     const base::Closure& on_finish)
       : web_contents_(web_contents),
         extension_(&extension),
         on_finish_(on_finish) {
   preferences_ =
       g_browser_process->media_file_system_registry()->GetPreferences(
           GetProfile());
   // Passing unretained pointer is safe, since the dialog controller
   // is self-deleting, and so won't be deleted until it can be shown
   // and then closed.
   preferences_->EnsureInitialized(
       base::Bind(&MediaGalleriesDialogController::OnPreferencesInitialized,
                  base::Unretained(this)));
 
-  gallery_menu_model_.reset(new GalleryContextMenuModel(this));
-  ui::SimpleMenuModel* menu_model =
-      new ui::SimpleMenuModel(gallery_menu_model_.get());
-  menu_model->AddItem(
-      1, l10n_util::GetStringUTF16(IDS_MEDIA_GALLERIES_DIALOG_DELETE));
-  context_menu_model_.reset(menu_model);
+  // Unretained is safe because |this| owns |context_menu_|.
+  context_menu_.reset(
+      new MediaGalleryContextMenu(
+          base::Bind(&MediaGalleriesDialogController::DidForgetGallery,
+                     base::Unretained(this))));
 }
 
 void MediaGalleriesDialogController::OnPreferencesInitialized() {
   InitializePermissions();
 
   dialog_.reset(MediaGalleriesDialog::Create(this));
 
   StorageMonitor::GetInstance()->AddObserver(this);
 
   preferences_->AddGalleryChangeObserver(this);
@@ skipping 348 matching lines @@
   }
 
   dialog_->UpdateGalleries();
 }
 
 void MediaGalleriesDialogController::UpdateGalleriesOnDeviceEvent(
     const std::string& device_id) {
   dialog_->UpdateGalleries();
 }
 
-ui::MenuModel* MediaGalleriesDialogController::GetContextMenuModel(
+ui::MenuModel* MediaGalleriesDialogController::GetContextMenu(
     MediaGalleryPrefId id) {
-  gallery_menu_model_->set_media_gallery_pref_id(id);
-  return context_menu_model_.get();
+  context_menu_->set_pref_id(id);
+  return context_menu_.get();
 }
 
 Profile* MediaGalleriesDialogController::GetProfile() {
   return Profile::FromBrowserContext(web_contents_->GetBrowserContext());
 }
 
 // MediaGalleries dialog -------------------------------------------------------
 
 MediaGalleriesDialog::~MediaGalleriesDialog() {}