Cleanup MenuRunner API

chrome/browser/ui/views/menu_model_adapter_test.cc

 // Copyright (c) 2012 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.
 
 #include "base/callback.h"
 #include "base/location.h"
 #include "base/macros.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread_task_runner_handle.h"
@@ skipping 156 matching lines @@
 
   ~MenuModelAdapterTest() override {}
 
   // ViewEventTestBase implementation.
 
   void SetUp() override {
     button_ = new views::MenuButton(base::ASCIIToUTF16("Menu Adapter Test"),
                                     this, true);
 
     menu_ = menu_model_adapter_.CreateMenu();
-    menu_runner_.reset(new views::MenuRunner(
-        menu_, views::MenuRunner::HAS_MNEMONICS | views::MenuRunner::ASYNC));
+    menu_runner_.reset(
+        new views::MenuRunner(menu_, views::MenuRunner::HAS_MNEMONICS));
 
     ViewEventTestBase::SetUp();
   }
 
   void TearDown() override {
     menu_runner_.reset(NULL);
     menu_ = NULL;
     ViewEventTestBase::TearDown();
   }
 
   views::View* CreateContentsView() override { return button_; }
 
   gfx::Size GetPreferredSize() const override {
     return button_->GetPreferredSize();
   }
 
   // views::MenuButtonListener implementation.
   void OnMenuButtonClicked(views::MenuButton* source,
                            const gfx::Point& point,
                            const ui::Event* event) override {
     gfx::Point screen_location;
     views::View::ConvertPointToScreen(source, &screen_location);
     gfx::Rect bounds(screen_location, source->size());
-    ignore_result(menu_runner_->RunMenuAt(source->GetWidget(),
-                                          button_,
-                                          bounds,
-                                          views::MENU_ANCHOR_TOPLEFT,
-                                          ui::MENU_SOURCE_NONE));
+    menu_runner_->RunMenuAt(source->GetWidget(), button_, bounds,
+                            views::MENU_ANCHOR_TOPLEFT, ui::MENU_SOURCE_NONE);
   }
 
   // ViewEventTestBase implementation
   void DoTestOnMessageLoop() override {
     Click(button_, CreateEventTask(this, &MenuModelAdapterTest::Step1));
   }
 
   // Open the submenu.
   void Step1() {
     views::SubmenuView* topmenu = menu_->GetSubmenu();
@@ skipping 55 matching lines @@
 
   views::MenuButton* button_;
   TopMenuModel top_menu_model_;
   views::MenuModelAdapter menu_model_adapter_;
   views::MenuItemView* menu_;
   std::unique_ptr<views::MenuRunner> menu_runner_;
 };
 
 // If this flakes, disable and log details in http://crbug.com/523255.
 VIEW_TEST(MenuModelAdapterTest, RebuildMenu)