mash: Replace int ShelfIDs with AppLaunchID strings.

ash/public/interfaces/shelf.mojom

Index: ash/public/interfaces/shelf.mojom
diff --git a/ash/public/interfaces/shelf.mojom b/ash/public/interfaces/shelf.mojom
index 19e0c2213a1422f3419d76cde8912a93af6a60c8..338fd1378733ea317e13ff6c7a86a9ae0794d446 100644
--- a/ash/public/interfaces/shelf.mojom
+++ b/ash/public/interfaces/shelf.mojom
@@ -115,16 +115,21 @@ struct MenuItem {
   uint32 radio_group_id;             // The radio group id.
 };
 
+// Identifier for shelf items and their windows.
+// This structure matches ash::ShelfID.
+struct ShelfID {
+  string app_id;            // An app id string, used to match app windows.

[James Cook, 2017/05/04 16:38:49]

nit: maybe call out that "app id" is usually an extension id? and that there are
special hard-coded ones for things like browser, task mgr, etc?

[msw, 2017/05/04 19:05:57]

Done.

+  string launch_id;         // A string used to support multiple items per app.
+                            // (eg. Citrix may use 'Word' or 'Excel' launch ids)
+};
+
 // ShelfItems are used to populate the shelf.
 // This structure matches ash::ShelfItem.
 struct ShelfItem {
   ShelfItemType type;       // The type of the shelf item.
   skia.mojom.Bitmap image;  // An icon image Bitmap, shown on the shelf.
-  int64 shelf_id;           // A shelf model ID used by the shelf and clients.
   ShelfItemStatus status;   // The running/closed/etc. status of the item.
-  string app_id;            // An app id string, used to match app windows.
-  string launch_id;         // A string used to support multiple items per app.
-                            // (eg. Citrix may use 'Word' or 'Excel' launch ids)
+  ShelfID shelf_id;         // The id for the shelf item and its windows.
   mojo.common.mojom.String16 title;  // The title to display for tooltips, etc.
   bool shows_tooltip;       // Whether the tooltip should be shown on hover.
   bool pinned_by_policy;    // Whether the item is pinned by policy preferences,