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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ui/views/views_delegate.h

Issue 2666803005: Position close buttons in bubbles based on a layout delegate constant. (Closed)
Patch Set: Remove extraneous brace Created 3 years, 11 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
« no previous file with comments | « ui/views/layout/layout_constants.h ('k') | ui/views/views_delegate.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/views_delegate.h
diff --git a/ui/views/views_delegate.h b/ui/views/views_delegate.h
index 11d39e89bbd90303315685167431dd25ae1a3333..20fd27bf9915bfb47880244dc444d91e614170e1 100644
--- a/ui/views/views_delegate.h
+++ b/ui/views/views_delegate.h
@@ -218,6 +218,10 @@ class VIEWS_EXPORT ViewsDelegate {
// Returns the padding that should be placed around a button's content.
virtual gfx::Insets GetButtonPadding() const;
+ // Returns the distance between a dialog's edge and the close button in the
+ // upper trailing corner.
+ virtual int GetCloseButtonMargin() const;
sky 2017/02/01 16:55:07 'close button' is rather generic, and it seems the
Peter Kasting 2017/02/07 04:45:29 SGTM
+
protected:
ViewsDelegate();
« no previous file with comments | « ui/views/layout/layout_constants.h ('k') | ui/views/views_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698