Change toolbar height for material design on Ash

chrome/browser/ui/views/frame/glass_browser_frame_view.cc

Index: chrome/browser/ui/views/frame/glass_browser_frame_view.cc
diff --git a/chrome/browser/ui/views/frame/glass_browser_frame_view.cc b/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
index ae8d6f72541850731250e92616c24432e7ada3a4..596ce549bfbed36b9bf998fa41252536432f8764 100644
--- a/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
+++ b/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
@@ -360,9 +360,11 @@ void GlassBrowserFrameView::PaintToolbarBackground(gfx::Canvas* canvas) {
   int x = toolbar_bounds.x();
   int w = toolbar_bounds.width();
 
+  const bool mode_material = ui::MaterialDesignController::IsModeMaterial();

[Peter Kasting, 2015/09/09 22:41:53]

When I asked for the opaque and glass frames to be updated, I intended for you
to make a much more substantial change, like the one you've made in the ash
frame, to actually bypass all the existing code in MD and instead draw exactly
what we need where in a simple fashion.

That would still be helpful, so if you can do that, please go ahead.

Otherwise, just revert all the changes in this file and the opaque frame.  The
only functional changes you're making here won't have any effect in my client
anyway since I'm entirely bypassing the code you're modifying, since it's
clearly not what we're going to want in MD.  It will mean more work for me as
I'll need to do the changes I refer to above myself based on your work in the
ash frame, but I'd rather that than add the hacks here, which to some degree
obscure what the correct implementation should be.

[tdanderson, 2015/09/10 19:48:25]

Reverted these changes for now, though I'd like to follow up with you offline
about this.

+
   gfx::ImageSkia* theme_toolbar = tp->GetImageSkiaNamed(IDR_THEME_TOOLBAR);
   gfx::ImageSkia* toolbar_center = tp->GetImageSkiaNamed(
-      IDR_CONTENT_TOP_CENTER);
+      mode_material ? IDR_THEME_TOOLBAR : IDR_CONTENT_TOP_CENTER);
 
   // Tile the toolbar image starting at the frame edge on the left and where
   // the tabstrip is on the top.
@@ -419,7 +421,7 @@ void GlassBrowserFrameView::PaintToolbarBackground(gfx::Canvas* canvas) {
   }
 
   // Draw the content/toolbar separator.
-  if (ui::MaterialDesignController::IsModeMaterial()) {
+  if (mode_material) {
     toolbar_bounds.Inset(kClientEdgeThickness, 0);
     BrowserView::Paint1pxHorizontalLine(
         canvas,