mash: Use mojo for ShelfItemDelegate and [app] MenuItem.

chrome/browser/ui/ash/launcher/chrome_launcher_controller_mus.cc

 // Copyright 2016 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/ui/ash/launcher/chrome_launcher_controller_mus.h"
 
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/ash/app_launcher_id.h"
 #include "chrome/browser/ui/ash/chrome_launcher_prefs.h"
 #include "chrome/browser/ui/ash/launcher/launcher_controller_helper.h"
 #include "extensions/grit/extensions_browser_resources.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/events/event_constants.h"
 
-class ChromeShelfItemDelegate : public ash::mojom::ShelfItemDelegate {
- public:
-  explicit ChromeShelfItemDelegate(const std::string& app_id,
-                                   ChromeLauncherController* controller)
-      : app_id_(app_id),
-        item_delegate_binding_(this),
-        controller_(controller) {}
-  ~ChromeShelfItemDelegate() override {}
-
-  ash::mojom::ShelfItemDelegateAssociatedPtrInfo
-  CreateInterfacePtrInfoAndBind() {
-    DCHECK(!item_delegate_binding_.is_bound());
-    ash::mojom::ShelfItemDelegateAssociatedPtrInfo ptr_info;
-    item_delegate_binding_.Bind(&ptr_info);
-    return ptr_info;
-  }
-
- private:
-  // ash::mojom::ShelfItemDelegate:
-  void LaunchItem() override {
-    controller_->LaunchApp(ash::AppLauncherId(app_id_),
-                           ash::LAUNCH_FROM_UNKNOWN, ui::EF_NONE);
-  }
-  void ExecuteCommand(uint32_t command_id, int event_flags) override {
-    NOTIMPLEMENTED();
-  }
-  void ItemPinned() override { NOTIMPLEMENTED(); }
-  void ItemUnpinned() override { NOTIMPLEMENTED(); }
-  void ItemReordered(uint32_t order) override { NOTIMPLEMENTED(); }
-
-  std::string app_id_;
-  mojo::AssociatedBinding<ash::mojom::ShelfItemDelegate> item_delegate_binding_;
-
-  // Not owned.
-  ChromeLauncherController* controller_;
-
-  DISALLOW_COPY_AND_ASSIGN(ChromeShelfItemDelegate);
-};
-
 ChromeLauncherControllerMus::ChromeLauncherControllerMus() {
   AttachProfile(ProfileManager::GetActiveUserProfile());
 }
 
 ChromeLauncherControllerMus::~ChromeLauncherControllerMus() {}
 
 ash::ShelfID ChromeLauncherControllerMus::CreateAppLauncherItem(
     LauncherItemController* controller,
     const std::string& app_id,
     ash::ShelfItemStatus status) {
@@ skipping 108 matching lines @@
 void ChromeLauncherControllerMus::ActiveUserChanged(
     const std::string& user_email) {
   NOTIMPLEMENTED();
 }
 
 void ChromeLauncherControllerMus::AdditionalUserAddedToSession(
     Profile* profile) {
   NOTIMPLEMENTED();
 }
 
-ash::ShelfAppMenuItemList
-ChromeLauncherControllerMus::GetAppMenuItemsForTesting(
+MenuItemList ChromeLauncherControllerMus::GetAppMenuItemsForTesting(
     const ash::ShelfItem& item) {
   NOTIMPLEMENTED();
-  return ash::ShelfAppMenuItemList();
+  return MenuItemList();
 }
 
 std::vector<content::WebContents*>
 ChromeLauncherControllerMus::GetV1ApplicationsFromAppId(
     const std::string& app_id) {
   NOTIMPLEMENTED();
   return std::vector<content::WebContents*>();
 }
 
 void ChromeLauncherControllerMus::ActivateShellApp(const std::string& app_id,
@@ skipping 83 matching lines @@
     if (app_launcher_id.app_id() == ash::launcher::kPinnedAppsPlaceholder)
       continue;
 
     ash::mojom::ShelfItemPtr item(ash::mojom::ShelfItem::New());
     item->app_id = app_id;
     item->app_title = base::UTF16ToUTF8(
         launcher_controller_helper()->GetAppTitle(profile(), app_id));
     ResourceBundle& rb = ResourceBundle::GetSharedInstance();
     const gfx::Image& image = rb.GetImageNamed(IDR_APP_DEFAULT_ICON);
     item->image = *image.ToSkBitmap();
-    std::unique_ptr<ChromeShelfItemDelegate> delegate(
-        new ChromeShelfItemDelegate(app_id, this));
-    shelf_controller()->PinItem(std::move(item),
-                                delegate->CreateInterfacePtrInfoAndBind());
-    app_id_to_item_delegate_.insert(
-        std::make_pair(app_id, std::move(delegate)));
-
+    // TOOD(msw): Actually pin the item and install its delegate; this code is
+    // unused at the moment. See http://crbug.com/647879
     AppIconLoader* app_icon_loader = GetAppIconLoaderForApp(app_id);
     if (app_icon_loader) {
       app_icon_loader->FetchImage(app_id);
       app_icon_loader->UpdateImage(app_id);
     }
   }
 }