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

Unified Diff: ui/views/bubble/bubble_dialog_delegate.h

Issue 1759453002: Convert location bar bubble delegates to bubble dialog delegates (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix compile errs Created 4 years, 10 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: ui/views/bubble/bubble_dialog_delegate.h
diff --git a/ui/views/bubble/bubble_dialog_delegate.h b/ui/views/bubble/bubble_dialog_delegate.h
index fc61adc0a7cb92faaa5a3b19fe7bff8deb57501d..89b3cb879dba094d8ab8df96ac7990235920c2b0 100644
--- a/ui/views/bubble/bubble_dialog_delegate.h
+++ b/ui/views/bubble/bubble_dialog_delegate.h
@@ -30,7 +30,6 @@ class VIEWS_EXPORT BubbleDialogDelegateView : public DialogDelegateView,
enum class CloseReason {
DEACTIVATION,
- ESCAPE,
CLOSE_BUTTON,
UNKNOWN,
};
@@ -56,9 +55,6 @@ class VIEWS_EXPORT BubbleDialogDelegateView : public DialogDelegateView,
void OnWidgetBoundsChanged(Widget* widget,
const gfx::Rect& new_bounds) override;
- bool close_on_esc() const { return close_on_esc_; }
- void set_close_on_esc(bool close_on_esc) { close_on_esc_ = close_on_esc; }
-
bool close_on_deactivate() const { return close_on_deactivate_; }
void set_close_on_deactivate(bool close) { close_on_deactivate_ = close; }
@@ -134,7 +130,6 @@ class VIEWS_EXPORT BubbleDialogDelegateView : public DialogDelegateView,
virtual const gfx::FontList& GetTitleFontList() const;
// View overrides:
- bool AcceleratorPressed(const ui::Accelerator& accelerator) override;
void OnNativeThemeChanged(const ui::NativeTheme* theme) override;
// Perform view initialization on the contents for bubble sizing.
@@ -165,8 +160,7 @@ class VIEWS_EXPORT BubbleDialogDelegateView : public DialogDelegateView,
// Handles widget visibility changes.
void HandleVisibilityChanged(Widget* widget, bool visible);
- // Flags controlling bubble closure on the escape key and deactivation.
- bool close_on_esc_;
+ // A flag controlling bubble closure on deactivation.
bool close_on_deactivate_;
// The view and widget to which this bubble is anchored. Since an anchor view

Powered by Google App Engine
This is Rietveld 408576698