Refactor menu controller to isolate aura dependency.

ui/views/controls/menu/menu_message_loop_aura.h

+// Copyright 2014 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 UI_VIEWS_CONTROLS_MENU_MENU_MESSAGE_LOOP_AURA_H_
+#define UI_VIEWS_CONTROLS_MENU_MENU_MESSAGE_LOOP_AURA_H_
+
+#include "base/compiler_specific.h"
+#include "base/memory/scoped_ptr.h"
+#include "ui/views/controls/menu/menu_message_loop.h"
+
+namespace ui {
+class ScopedEventDispatcher;
+}
+
+namespace views {
+
+class MenuMessageLoopAura : public MenuMessageLoop {
+ public:
+  MenuMessageLoopAura();
+  virtual ~MenuMessageLoopAura();
+
+  virtual void Run(MenuController* controller,

[sky, 2014/05/15 16:07:32]

Prefix section with something indicating these are overrides, eg
// MenuMessageLoop overrides:
or just
// MenuMessageLoop:

[Andre, 2014/05/15 18:45:59]

Done.

+                   Widget* owner,
+                   bool nested_menu) OVERRIDE;
+  virtual bool ShouldQuitNow() const OVERRIDE;
+  virtual void QuitNow() OVERRIDE;
+  virtual void RepostEventToWindow(const ui::LocatedEvent& event,
+                                   gfx::NativeWindow window,
+                                   gfx::Point screen_loc) OVERRIDE;
+  virtual int message_loop_depth() const OVERRIDE;
+
+ private:
+  // Owner of child windows.
+  // WARNING: this may be NULL.
+  Widget* owner_;
+
+  int message_loop_depth_;
+
+  scoped_ptr<ui::ScopedEventDispatcher> nested_dispatcher_;
+
+  DISALLOW_COPY_AND_ASSIGN(MenuMessageLoopAura);
+};
+
+}  // namespace views
+
+#endif  // UI_VIEWS_CONTROLS_MENU_MENU_MESSAGE_LOOP_AURA_H_