Remove PerBrowser launcher

chrome/browser/ui/ash/launcher/launcher_item_controller.h

Index: chrome/browser/ui/ash/launcher/launcher_item_controller.h
diff --git a/chrome/browser/ui/ash/launcher/launcher_item_controller.h b/chrome/browser/ui/ash/launcher/launcher_item_controller.h
index 631a66fb5d8893ad5dd1eea4129d3acb3b51ed49..f8706bae1508bcf81b8483b25fc8479a99d739b9 100644
--- a/chrome/browser/ui/ash/launcher/launcher_item_controller.h
+++ b/chrome/browser/ui/ash/launcher/launcher_item_controller.h
@@ -45,7 +45,7 @@ class LauncherItemController {
   ash::LauncherID launcher_id() const { return launcher_id_; }
   void set_launcher_id(ash::LauncherID id) { launcher_id_ = id; }
   virtual const std::string& app_id() const;
-  ChromeLauncherController* launcher_controller() {
+  ChromeLauncherController* launcher_controller() const {

[James Cook, 2013/08/19 22:32:38]

we don't usually use "const" for accessor functions like this that return
pointers, since the caller can modify the data it points to

[simonhong_, 2013/08/20 01:11:20]

How about adding a const version of launcher_controller()?
This function is needed in some const functions.

     return launcher_controller_;
   }