mash: Cleanup ash shelf application menu code.

ash/common/test/test_shelf_item_delegate.h

 // Copyright 2013 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_COMMON_TEST_TEST_SHELF_ITEM_DELEGATE_H_
 #define ASH_COMMON_TEST_TEST_SHELF_ITEM_DELEGATE_H_
 
 #include "ash/common/shelf/shelf_item_delegate.h"
 #include "base/compiler_specific.h"

[James Cook, 2017/02/06 17:30:21]

drive-by: not needed

[msw, 2017/02/07 09:12:01]

Done.

 #include "base/macros.h"
 
 namespace ash {
 
 class WmWindow;
 
 namespace test {
 
 // Test implementation of ShelfItemDelegate.
 class TestShelfItemDelegate : public ShelfItemDelegate {
  public:
   explicit TestShelfItemDelegate(WmWindow* window);
   ~TestShelfItemDelegate() override;
 
   // ShelfItemDelegate:
   ShelfItemDelegate::PerformedAction ItemSelected(
       const ui::Event& event) override;
-  ui::SimpleMenuModel* CreateApplicationMenu(int event_flags) override;
+  ash::ShelfApplicationMenuItems GetAppMenuItems(int event_flags) override;

[James Cook, 2017/02/06 17:30:21]

I presume ash:: is needed because of the test namespace?

[msw, 2017/02/07 09:12:01]

Nope, I just missed this; removed.

   void Close() override;
 
  private:
   WmWindow* window_;
 
   DISALLOW_COPY_AND_ASSIGN(TestShelfItemDelegate);
 };
 
 }  // namespace test
 }  // namespace ash
 
 #endif  // ASH_COMMON_TEST_TEST_SHELF_ITEM_DELEGATE_H_