Refactor LauncherItemController and LauncherItemDelegate

ash/launcher/launcher_util.h

 // 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.
 
 #ifndef ASH_LAUNCHER_LAUNCHER_UTIL_H_
 #define ASH_LAUNCHER_LAUNCHER_UTIL_H_
 
 #include "ash/ash_export.h"
+#include "ash/launcher/launcher_types.h"
 
 namespace ash {
 class LauncherModel;
 
 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
 
 #endif  // ASH_LAUNCHER_LAUNCHER_UTIL_H_