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

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: Fix test 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..45b9936effff35ec83cd86ccfe5d01b660608aec 100644
--- a/ash/frame/header_painter_util.h
+++ b/ash/frame/header_painter_util.h
@@ -37,11 +37,11 @@ class ASH_EXPORT HeaderPainterUtil {
// 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