MacViews: Run native Cocoa context menus to support Services.

ui/views/controls/menu/menu_runner_impl_interface.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_RUNNER_IMPL_INTERFACE_H_
+#define UI_VIEWS_CONTROLS_MENU_MENU_RUNNER_IMPL_INTERFACE_H_
+
+#include "ui/views/controls/menu/menu_runner.h"
+
+namespace views {
+
+class MenuItemView;
+
+namespace internal {
+
+// An abstract interface for menu runner implementations.
+// Invoke Release() to destroy. Release() deletes immediately if the menu isn't
+// showing. If the menu is showing Release() cancels the menu and when the
+// nested RunMenuAt() call returns deletes itself and the menu.
+class MenuRunnerImplInterface {
+ public:
+  // Creates a concrete instance for running |menu_model|.
+  // |run_types| is a bitmask of MenuRunner::RunTypes.
+  static MenuRunnerImplInterface* Create(ui::MenuModel* menu_model,
+                                         int32 run_types);
+
+  // Returns the menu view, or NULL if the runner doesn't use a MenuItemView to
+  // run the menu.
+  virtual MenuItemView* GetMenu() const = 0;

[sky, 2014/07/15 20:36:38]

It's unfortunate the MenuItemView has to be exposed as it makes no sense for
coca. Did you verify all call sites deal with NULL correctly?

[Andre, 2014/07/16 18:43:28]

Done.
I've removed GetMenu, see https://codereview.chromium.org/393943006.

+
+  // Returns true if we're in a nested message loop running the menu.
+  virtual bool IsRunning() const = 0;
+
+  // See description above class for details.
+  virtual void Release() = 0;
+
+  // Runs the menu. See MenuRunner::RunMenuAt for more details.
+  virtual MenuRunner::RunResult RunMenuAt(Widget* parent,
+                                          MenuButton* button,
+                                          const gfx::Rect& bounds,
+                                          MenuAnchorPosition anchor,
+                                          int32 run_types)
+      WARN_UNUSED_RESULT = 0;
+
+  // Hides and cancels the menu.
+  virtual void Cancel() = 0;
+
+  // Returns the time from the event which closed the menu - or 0.
+  virtual base::TimeDelta GetClosingEventTime() const = 0;
+
+ protected:
+  // Call Release() to delete.
+  virtual ~MenuRunnerImplInterface() {}
+};
+
+}  // namespace internal
+}  // namespace views
+
+#endif  // UI_VIEWS_CONTROLS_MENU_MENU_RUNNER_IMPL_INTERFACE_H_