Change toolbar height for material design on Ash

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

 // Copyright 2012 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.
 
 #include "chrome/browser/ui/views/frame/browser_non_client_frame_view_ash.h"
 
 #include <algorithm>
 
 #include "ash/frame/caption_buttons/frame_caption_button_container_view.h"
 #include "ash/frame/default_header_painter.h"
@@ skipping 615 matching lines @@
   if (toolbar_bounds.IsEmpty())
     return;
   gfx::Point toolbar_origin(toolbar_bounds.origin());
   View::ConvertPointToTarget(browser_view(), this, &toolbar_origin);
   toolbar_bounds.set_origin(toolbar_origin);
 
   int x = toolbar_bounds.x();
   int w = toolbar_bounds.width();
   int y = toolbar_bounds.y();
   int h = toolbar_bounds.height();
-
-  // Gross hack: We split the toolbar images into two pieces, since sometimes
-  // (popup mode) the toolbar isn't tall enough to show the whole image.  The
-  // split happens between the top shadow section and the bottom gradient
-  // section so that we never break the gradient.
-  // NOTE(pkotwicz): If the computation for |bottom_y| is changed, Layout() must
-  // be changed as well.
-  int split_point = kFrameShadowThickness * 2;
-  int bottom_y = y + split_point;
   ui::ThemeProvider* tp = GetThemeProvider();
-  int bottom_edge_height = h - split_point;
-
-  canvas->FillRect(gfx::Rect(x, bottom_y, w, bottom_edge_height),
-                   tp->GetColor(ThemeProperties::COLOR_TOOLBAR));
-
-  // Paint the main toolbar image.  Since this image is also used to draw the
-  // tab background, we must use the tab strip offset to compute the image
-  // source y position.  If you have to debug this code use an image editor
-  // to paint a diagonal line through the toolbar image and ensure it lines up
-  // across the tab and toolbar.
-  gfx::ImageSkia* theme_toolbar = tp->GetImageSkiaNamed(IDR_THEME_TOOLBAR);
-  canvas->TileImageInt(
-      *theme_toolbar,
-      x + GetThemeBackgroundXInset(),
-      bottom_y - GetTopInset(),
-      x, bottom_y,
-      w, theme_toolbar->height());
 
   if (ui::MaterialDesignController::IsModeMaterial()) {
+    // Paint the main toolbar image.  Since this image is also used to draw the
+    // tab background, we must use the tab strip offset to compute the image
+    // source y position.  If you have to debug this code use an image editor
+    // to paint a diagonal line through the toolbar image and ensure it lines up
+    // across the tab and toolbar.
+    gfx::ImageSkia* theme_toolbar = tp->GetImageSkiaNamed(IDR_THEME_TOOLBAR);
+    canvas->TileImageInt(
+        *theme_toolbar,
+        x + GetThemeBackgroundXInset(),
+        y - GetTopInset(),
+        x, y,
+        w, theme_toolbar->height());
+
     // Draw the content/toolbar separator.
     toolbar_bounds.Inset(kClientEdgeThickness, 0);
     BrowserView::Paint1pxHorizontalLine(
         canvas,
         ThemeProperties::GetDefaultColor(
             ThemeProperties::COLOR_TOOLBAR_SEPARATOR),
         toolbar_bounds);
   } else {
+    // Gross hack: We split the toolbar images into two pieces, since sometimes
+    // (popup mode) the toolbar isn't tall enough to show the whole image.  The
+    // split happens between the top shadow section and the bottom gradient
+    // section so that we never break the gradient.
+    // NOTE(pkotwicz): If the computation for |bottom_y| is changed, Layout()
+    // must be changed as well.
+    int split_point = kFrameShadowThickness * 2;
+    int bottom_y = y + split_point;
+    int bottom_edge_height = h - split_point;
+
+    canvas->FillRect(gfx::Rect(x, bottom_y, w, bottom_edge_height),
+                     tp->GetColor(ThemeProperties::COLOR_TOOLBAR));
+
+    // Paint the main toolbar image.  Since this image is also used to draw the
+    // tab background, we must use the tab strip offset to compute the image
+    // source y position.  If you have to debug this code use an image editor
+    // to paint a diagonal line through the toolbar image and ensure it lines up
+    // across the tab and toolbar.
+    gfx::ImageSkia* theme_toolbar = tp->GetImageSkiaNamed(IDR_THEME_TOOLBAR);
+    canvas->TileImageInt(
+        *theme_toolbar,
+        x + GetThemeBackgroundXInset(),
+        bottom_y - GetTopInset(),
+        x, bottom_y,
+        w, theme_toolbar->height());
+
     // The pre-material design content area line has a shadow that extends a
     // couple of pixels above the toolbar bounds.
     const int kContentShadowHeight = 2;
     gfx::ImageSkia* toolbar_top = tp->GetImageSkiaNamed(IDR_TOOLBAR_SHADE_TOP);
     canvas->TileImageInt(*toolbar_top,
                          0, 0,
                          x, y - kContentShadowHeight,
                          w, split_point + kContentShadowHeight + 1);
 
     // Draw the "lightening" shade line around the edges of the toolbar.
@@ skipping 23 matching lines @@
   }
 }
 
 void BrowserNonClientFrameViewAsh::PaintContentEdge(gfx::Canvas* canvas) {
   DCHECK(!UsePackagedAppHeaderStyle() && !UseWebAppHeaderStyle());
   canvas->FillRect(gfx::Rect(0, caption_button_container_->bounds().bottom(),
                              width(), kClientEdgeThickness),
                    ThemeProperties::GetDefaultColor(
                        ThemeProperties::COLOR_TOOLBAR_SEPARATOR));
 }