Introduce new WebApp header style for hosted apps and fizzy apps on ash.

chrome/browser/ui/views/frame/webapp_header_painter_ash.h

+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_VIEWS_FRAME_WEBAPP_HEADER_PAINTER_ASH_H_
+#define CHROME_BROWSER_UI_VIEWS_FRAME_WEBAPP_HEADER_PAINTER_ASH_H_
+
+#include "ash/frame/default_header_painter.h"
+
+namespace views {
+class View;
+class Widget;
+}
+
+namespace ash {
+class FrameCaptionButton;
+class FrameCaptionButtonContainerView;
+}

[calamity, 2014/08/06 07:07:07]

nit: Move this above the views namespace.

[benwells, 2014/08/06 08:13:59]

Done.

+
+// Helper class for painting the WebApp style window header.
+class WebAppHeaderPainterAsh : public ash::DefaultHeaderPainter {
+ public:
+  WebAppHeaderPainterAsh();
+  virtual ~WebAppHeaderPainterAsh();
+
+  // DefaultHeaderPainter does not take ownership of any of the parameters.

[calamity, 2014/08/06 07:07:07]

nit: // WebAppHeaderPainterAsh does not take ownership of any of the parameters.

[benwells, 2014/08/06 08:13:59]

Done.

+  void Init(views::Widget* frame,
+            views::View* header_view,
+            ash::FrameCaptionButton* back_button,
+            ash::FrameCaptionButtonContainerView* caption_button_container);

[calamity, 2014/08/06 07:07:07]

It's a bit scary that this name collides with the parent class' method. You can
even call the parent's Init() with these args.

I'm not sure what the right way to deal with this is (or if it's even a big
issue).

[benwells, 2014/08/06 08:13:59]

Oh yeah, good catch. Done.

+
+ private:
+  // DefaultHeaderPainter overrides:
+  virtual void LayoutHeader() OVERRIDE;
+  virtual views::View* GetViewToLeftOfTitle() const OVERRIDE;
+
+  ash::FrameCaptionButton* back_button_;
+
+  DISALLOW_COPY_AND_ASSIGN(WebAppHeaderPainterAsh);
+};
+
+#endif  // CHROME_BROWSER_UI_VIEWS_FRAME_WEBAPP_HEADER_PAINTER_ASH_H_