mash: Remove shelf app menu item objects.

ash/common/shelf/shelf_controller.cc

Index: ash/common/shelf/shelf_controller.cc
diff --git a/ash/common/shelf/shelf_controller.cc b/ash/common/shelf/shelf_controller.cc
index 4e836733547918329d161f98472c4fde6acf87c8..4da9401777e8000770cabbd02f8be51721f5558f 100644
--- a/ash/common/shelf/shelf_controller.cc
+++ b/ash/common/shelf/shelf_controller.cc
@@ -74,11 +74,15 @@ class ShelfItemDelegateMus : public ShelfItemDelegate {
     ShelfAppMenuItemList items;
     for (const auto& window : window_id_to_title_) {
       items.push_back(
-          base::MakeUnique<ShelfApplicationMenuItem>(window.second));
+          base::MakeUnique<ShelfApplicationMenuItem>(0, window.second));

[James Cook, 2017/03/01 19:50:48]

Does 0 mean invalid command, or first command, or both? If it means both it
might be nice to use -1 or something for invalid.

[msw, 2017/03/02 02:59:20]

I removed this code and left the standard no-menu comment, since this is unused
at the moment. I'll likely want to revise the |window_id_to_title_| map into a
vector of window structures with ids and titles for O(1) indexed access, etc.
Someday I'll be done with the prerequisite work to make this useful again.

     }
     return items;
   }
 
+  void ExecuteCommand(uint32_t command_id, int32_t event_flags) override {
+    NOTIMPLEMENTED();
+  }
+
   void Close() override { NOTIMPLEMENTED(); }
 
   mojom::ShelfItemDelegateAssociatedPtr delegate_;