Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(620)

Unified Diff: content/browser/renderer_host/popup_menu_helper_mac.h

Issue 197533008: Mac: ExternalPopupMenu::close should hide a showing popup menu. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: test fix Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/popup_menu_helper_mac.h
diff --git a/content/browser/renderer_host/popup_menu_helper_mac.h b/content/browser/renderer_host/popup_menu_helper_mac.h
index 332fecdc54f57baf454e8d9afe5d439f7d8e11fe..8a6f3176c7e793802384a85ed348d156150cf120 100644
--- a/content/browser/renderer_host/popup_menu_helper_mac.h
+++ b/content/browser/renderer_host/popup_menu_helper_mac.h
@@ -13,6 +13,12 @@
#include "content/public/browser/notification_registrar.h"
#include "ui/gfx/rect.h"
+#ifdef __OBJC__
+@class WebMenuRunner;
+#else
+class WebMenuRunner;
+#endif
+
namespace content {
class RenderViewHost;
class RenderViewHostImpl;
@@ -24,6 +30,7 @@ class PopupMenuHelper : public NotificationObserver {
// Creates a PopupMenuHelper that will notify |render_view_host| when a user
// selects or cancels the popup.
explicit PopupMenuHelper(RenderViewHost* render_view_host);
+ void Hide();
// Shows the popup menu and notifies the RenderViewHost of the selection/
// cancel.
@@ -48,8 +55,9 @@ class PopupMenuHelper : public NotificationObserver {
const NotificationDetails& details) OVERRIDE;
NotificationRegistrar notification_registrar_;
-
RenderViewHostImpl* render_view_host_;
+ WebMenuRunner* menu_runner_;
+ bool popup_was_hidden_;
DISALLOW_COPY_AND_ASSIGN(PopupMenuHelper);
};
« no previous file with comments | « content/browser/frame_host/interstitial_page_impl.cc ('k') | content/browser/renderer_host/popup_menu_helper_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698