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

Unified Diff: chrome/browser/ui/views/extensions/media_galleries_dialog_views.h

Issue 2394123002: Views: Expose an on_closed callback via the MenuRunner constructor. (Closed)
Patch Set: default arg Created 4 years, 2 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: chrome/browser/ui/views/extensions/media_galleries_dialog_views.h
diff --git a/chrome/browser/ui/views/extensions/media_galleries_dialog_views.h b/chrome/browser/ui/views/extensions/media_galleries_dialog_views.h
index 29714146696d1c6842707ea3fdf2d44f3886cc5f..5ca9e9b257a1d314497f00181973ca5ec97ffae2 100644
--- a/chrome/browser/ui/views/extensions/media_galleries_dialog_views.h
+++ b/chrome/browser/ui/views/extensions/media_galleries_dialog_views.h
@@ -18,7 +18,6 @@
namespace views {
class Checkbox;
class LabelButton;
-class MenuModelAdapter;
class MenuRunner;
class Widget;
}
@@ -88,7 +87,7 @@ class MediaGalleriesDialogViews : public MediaGalleriesDialog,
// In unit tests, it may not.
bool ControllerHasWebContents() const;
- // Callback for MenuModelAdapter.
+ // Callback for MenuRunner.
void OnMenuClosed();
MediaGalleriesDialogController* controller_;
@@ -111,7 +110,6 @@ class MediaGalleriesDialogViews : public MediaGalleriesDialog,
// True if the user has pressed accept.
bool accepted_;
- std::unique_ptr<views::MenuModelAdapter> menu_model_adapter_;
std::unique_ptr<views::MenuRunner> context_menu_runner_;
DISALLOW_COPY_AND_ASSIGN(MediaGalleriesDialogViews);

Powered by Google App Engine
This is Rietveld 408576698