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

Unified Diff: ash/frame/header_painter_util.h

Issue 441803004: Introduce new WebApp header style for hosted apps and fizzy apps on ash. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Real assets, crushed, review feedback Created 6 years, 4 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: ash/frame/header_painter_util.h
diff --git a/ash/frame/header_painter_util.h b/ash/frame/header_painter_util.h
index 5e4bde42506d0368061f4c80a80549711913533f..8114a18e3541fb661dbaf1317af58414f75f6316 100644
--- a/ash/frame/header_painter_util.h
+++ b/ash/frame/header_painter_util.h
@@ -28,20 +28,17 @@ class ASH_EXPORT HeaderPainterUtil {
// Returns the distance between the left edge of the window and the header
// icon.
pkotwicz 2014/08/19 00:17:07 Can you please update the stale comment?
benwells 2014/08/19 01:42:42 Done.
- static int GetIconXOffset();
-
- // Returns the size of the header icon.
- static int GetDefaultIconSize();
+ static int GetDefaultLeftViewXInset();
// Returns the amount that the frame background is inset from the left edge of
// the window.
static int GetThemeBackgroundXInset();
- // Returns the bounds for the header's title given the header icon, the
- // caption button container and the font used.
- // |icon| should be NULL if the header does not use an icon.
- static gfx::Rect GetTitleBounds(const views::View* icon,
- const views::View* caption_button_container,
+ // Returns the bounds for the header's title given the views to the left and
+ // right of the title, and the font used.
+ // |left_view| should be NULL if there is no view to the left of the title.
+ static gfx::Rect GetTitleBounds(const views::View* left_view,
+ const views::View* right_view,
const gfx::FontList& title_font_list);
// Returns true if the header for |widget| can animate to new visuals when the

Powered by Google App Engine
This is Rietveld 408576698