Refactor LauncherItemController and LauncherItemDelegate

ash/launcher/launcher_util.h

Index: ash/launcher/launcher_util.h
diff --git a/ash/launcher/launcher_util.h b/ash/launcher/launcher_util.h
index 89551f064abdc76900d65593cdd9864112e7eea0..0d7a753b5152ac59ecc7ce44416819a1dfdd6c9f 100644
--- a/ash/launcher/launcher_util.h
+++ b/ash/launcher/launcher_util.h
@@ -6,6 +6,7 @@
 #define ASH_LAUNCHER_LAUNCHER_UTIL_H_
 
 #include "ash/ash_export.h"
+#include "ash/launcher/launcher_types.h"
 
 namespace ash {
 class LauncherModel;
@@ -15,6 +16,14 @@ namespace launcher {
 // Return the index of the browser item from a given |launcher_model|.
 ASH_EXPORT int GetBrowserItemIndex(const LauncherModel& launcher_model);
 
+// Return the id of the browser item from a given |launcher_model|.
+ASH_EXPORT ash::LauncherID GetBrowserItemID(

[sky, 2013/09/11 01:42:57]

How about a single method that takes the type?

[simonhong_, 2013/09/11 09:05:26]

Done.

+    const LauncherModel& launcher_model);
+
+// Return the id of the app list item from a given |launcher_model|.
+ASH_EXPORT ash::LauncherID GetAppListItemID(
+    const LauncherModel& launcher_model);
+
 }  // namespace launcher
 }  // namespace ash