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

Unified Diff: chrome/browser/ui/layout_constants.h

Issue 1505223004: Do not use assets for Ash window control button backgrounds (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: WIP for pkasting and estade feedback Created 5 years 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/layout_constants.h
diff --git a/chrome/browser/ui/layout_constants.h b/chrome/browser/ui/layout_constants.h
index 603f094bc76b00b599a5b179df4fd7fcdca47253..afff77af4b6d2f0ebcd6aac97e44079bd2a6cafa 100644
--- a/chrome/browser/ui/layout_constants.h
+++ b/chrome/browser/ui/layout_constants.h
@@ -8,6 +8,18 @@
#include "ui/gfx/geometry/insets.h"
enum LayoutConstant {
+ // The height of a button in in a maximized Ash browser window.
+ ASH_BROWSER_MAXIMIZED_HEADER_BUTTON_HEIGHT,
tdanderson 2015/12/17 18:00:54 Peter, is it appropriate to put these here since t
Peter Kasting 2015/12/17 20:56:08 Covered in my other reply.
tdanderson 2015/12/18 17:42:53 Acknowledged.
+
+ // The width of a button in in a maximized Ash browser window.
+ ASH_BROWSER_MAXIMIZED_HEADER_BUTTON_WIDTH,
+
+ // The height of a button in in a restored Ash browser window.
+ ASH_BROWSER_RESTORED_HEADER_BUTTON_HEIGHT,
+
+ // The width of a button in in a restored Ash browser window.
+ ASH_BROWSER_RESTORED_HEADER_BUTTON_WIDTH,
+
// Vertical offset from top of content to the top of find bar.
FIND_BAR_TOOLBAR_OVERLAP,

Powered by Google App Engine
This is Rietveld 408576698