Enhance chrome.app.window API for shelf integration with pinning support

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 65db56634dff1b990929fd9d79bc75431dfcef5c..3be31e10d5d351f44986ad053cbd17711a2d0189 100644
--- a/chrome/browser/ui/ash/launcher/launcher_item_controller.h
+++ b/chrome/browser/ui/ash/launcher/launcher_item_controller.h
@@ -42,6 +42,7 @@ class LauncherItemController : public ash::ShelfItemDelegate {
 
   LauncherItemController(Type type,
                          const std::string& app_id,
+                         const std::string& launch_id,
                          ChromeLauncherController* launcher_controller);
   ~LauncherItemController() override;
 
@@ -49,6 +50,7 @@ class LauncherItemController : public ash::ShelfItemDelegate {
   ash::ShelfID shelf_id() const { return shelf_id_; }
   void set_shelf_id(ash::ShelfID id) { shelf_id_ = id; }
   const std::string& app_id() const { return app_id_; }
+  const std::string& launch_id() const { return launch_id_; }
   ChromeLauncherController* launcher_controller() const {
     return launcher_controller_;
   }
@@ -90,6 +92,8 @@ class LauncherItemController : public ash::ShelfItemDelegate {
   const Type type_;
   // App id will be empty if there is no app associated with the window.
   const std::string app_id_;
+  // The launch id associated with the window.

[stevenjb, 2016/08/31 18:24:28]

This doesn't explain what a 'launch id' is. This should be something like:
// An id that can be passed to an app when launched in order to support multiple
shelf (launcher) items per app.

[Andra Paraschiv, 2016/09/01 08:27:25]

Done.

+  const std::string launch_id_;
   ash::ShelfID shelf_id_;

[stevenjb, 2016/08/31 18:24:27]

Add a comment for shelf_id to help reduce the naming confusion, e.g.:
// A unique id for the shelf item (based on app_id and launch_id).

[Andra Paraschiv, 2016/09/01 08:27:25]

Done.

   ChromeLauncherController* launcher_controller_;