Refactor menu controller to isolate aura dependency.

ui/views/controls/menu/menu_message_loop_mac.cc

+// Copyright (c) 2014 The Chromium Authors. All rights reserved.

[tapted, 2014/05/04 23:43:39]

nit: no c

[Andre, 2014/05/05 22:41:29]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "base/run_loop.h"
+#include "ui/views/controls/menu/menu_message_loop.h"

[tapted, 2014/05/04 23:43:39]

nit: first include

[Andre, 2014/05/05 22:41:29]

Done.

+
+namespace views {
+
+void MenuMessageLoop::RepostEvent(const ui::LocatedEvent& event) {
+}

[tapted, 2014/05/04 23:43:39]

should have a NOTIMPLEMENTED()?

[Andre, 2014/05/05 22:41:29]

Done.

+
+void MenuMessageLoop::Run(MenuController* controller, bool nested_menu) {
+  base::MessageLoopForUI* loop = base::MessageLoopForUI::current();

[tapted, 2014/05/04 23:43:39]

nit: should #include base/message_loop/message_loop.h (IWUY). run_loop.h should
really just have a forward declare of MessageLoop...

[Andre, 2014/05/05 22:41:29]

Done.

+  base::MessageLoop::ScopedNestableTaskAllower allow(loop);
+  base::RunLoop run_loop;
+  run_loop.Run();
+}
+
+bool MenuMessageLoop::ShouldQuitNow() const {
+  return true;
+}
+
+void MenuMessageLoop::QuitNow() {
+  base::MessageLoop::current()->QuitNow();
+}
+
+}  // namespace views