Refactor LauncherDelegate

chrome/browser/ui/ash/launcher/chrome_launcher_controller_per_browser.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/ui/ash/launcher/chrome_launcher_controller_per_browser.h"
 
 #include <vector>
 
 #include "ash/ash_switches.h"
 #include "ash/launcher/launcher_model.h"
@@ skipping 715 matching lines @@
 void ChromeLauncherControllerPerBrowser::CreateNewWindow() {
   chrome::NewEmptyWindow(
       GetProfileForNewWindows(), chrome::HOST_DESKTOP_TYPE_ASH);
 }
 
 void ChromeLauncherControllerPerBrowser::CreateNewIncognitoWindow() {
   chrome::NewEmptyWindow(GetProfileForNewWindows()->GetOffTheRecordProfile(),
                          chrome::HOST_DESKTOP_TYPE_ASH);
 }
 
-bool ChromeLauncherControllerPerBrowser::CanPin() const {
-  const PrefService::Preference* pref =
-      profile_->GetPrefs()->FindPreference(prefs::kPinnedLauncherApps);
-  return pref && pref->IsUserModifiable();
-}
-
 ash::ShelfAutoHideBehavior
     ChromeLauncherControllerPerBrowser::GetShelfAutoHideBehavior(
         aura::RootWindow* root_window) const {
   // Don't show the shelf in the app mode.
   if (chrome::IsRunningInAppMode())
     return ash::SHELF_AUTO_HIDE_ALWAYS_HIDDEN;
 
   // See comment in |kShelfAlignment| as to why we consider two prefs.
   const std::string behavior_value(
       GetPrefForRootWindow(profile_->GetPrefs(),
@@ skipping 222 matching lines @@
   launcher->shelf_widget()->shelf_layout_manager()->AddObserver(this);
 }
 
 void ChromeLauncherControllerPerBrowser::OnLauncherDestroyed(
     ash::Launcher* launcher) {
   launchers_.erase(launcher);
   // RemoveObserver is not called here, since by the time this method is called
   // Launcher is already in its destructor.
 }
 
+ash::LauncherItemDelegate*
+ChromeLauncherControllerPerBrowser::GetLauncherItemDelegate(
+    const ash::LauncherItem& item) {
+  // PerBrowserLauncher don't have LauncherItemController.
+  if (item.type == ash::TYPE_BROWSER_SHORTCUT)
+    return this;
+
+  IDToItemControllerMap::iterator iter =
+      id_to_item_controller_map_.find(item.id);
+  return (iter != id_to_item_controller_map_.end()) ? this : NULL;
+}
+
+bool ChromeLauncherControllerPerBrowser::CanPin() const {
+  const PrefService::Preference* pref =
+      profile_->GetPrefs()->FindPreference(prefs::kPinnedLauncherApps);
+  return pref && pref->IsUserModifiable();
+}
+
 void ChromeLauncherControllerPerBrowser::LauncherItemAdded(int index) {
 }
 
 void ChromeLauncherControllerPerBrowser::LauncherItemRemoved(
     int index,
     ash::LauncherID id) {
 }
 
 void ChromeLauncherControllerPerBrowser::LauncherItemMoved(
     int start_index,
     int target_index) {
   ash::LauncherID id = model_->items()[target_index].id;
   if (HasItemController(id) && IsPinned(id))
     PersistPinnedState();
   else if (!HasItemController(id) &&
            model_->items()[target_index].type == ash::TYPE_BROWSER_SHORTCUT)
     PersistPinnedState();
 }
 
 void ChromeLauncherControllerPerBrowser::LauncherItemChanged(
     int index,
     const ash::LauncherItem& old_item) {
   ash::LauncherID id = model_->items()[index].id;
-  id_to_item_controller_map_[id]->LauncherItemChanged(index, old_item);
+  if (HasItemController(id))
+    id_to_item_controller_map_[id]->LauncherItemChanged(index, old_item);
 }
 
 void ChromeLauncherControllerPerBrowser::LauncherStatusChanged() {
 }
 
 void ChromeLauncherControllerPerBrowser::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   switch (type) {
@@ skipping 450 matching lines @@
     ash::LauncherID id) {
   return id_to_item_controller_map_[id]->app_id();
 }
 
 void ChromeLauncherControllerPerBrowser::
     MoveItemWithoutPinnedStateChangeNotification(int source_index,
                                                  int target_index) {
   base::AutoReset<bool> auto_reset(&ignore_persist_pinned_state_change_, true);
   model_->Move(source_index, target_index);
 }