 Chromium Code Reviews
 Chromium Code Reviews 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
    
  
    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| 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 |