Do not use assets for Ash window control button backgrounds

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

Index: chrome/browser/ui/views/frame/browser_header_painter_ash.cc
diff --git a/chrome/browser/ui/views/frame/browser_header_painter_ash.cc b/chrome/browser/ui/views/frame/browser_header_painter_ash.cc
index d840736c28c796532486254d8920dfa8fd613588..6357db550f7fa5154c338732ef80b08dcff0508c 100644
--- a/chrome/browser/ui/views/frame/browser_header_painter_ash.cc
+++ b/chrome/browser/ui/views/frame/browser_header_painter_ash.cc
@@ -4,6 +4,7 @@
 
 #include "chrome/browser/ui/views/frame/browser_header_painter_ash.h"
 
+#include "ash/ash_layout_constants.h"
 #include "ash/frame/caption_buttons/frame_caption_button_container_view.h"
 #include "ash/frame/header_painter_util.h"
 #include "base/logging.h"  // DCHECK
@@ -238,7 +239,7 @@ void BrowserHeaderPainterAsh::LayoutHeader() {
   // on having laid out the window controls.
   painted_height_ = -1;
 
-  UpdateCaptionButtonImages();
+  UpdateCaptionButtons();
   caption_button_container_->Layout();
 
   gfx::Size caption_button_container_size =
@@ -399,52 +400,34 @@ gfx::ImageSkia BrowserHeaderPainterAsh::GetFrameImageForNonTabbedBrowser(
       IDR_THEME_FRAME_INCOGNITO_INACTIVE : IDR_THEME_FRAME_INACTIVE);
 }
 
-void BrowserHeaderPainterAsh::UpdateCaptionButtonImages() {
-  int hover_background_id = 0;
-  int pressed_background_id = 0;
-  if (frame_->IsMaximized() || frame_->IsFullscreen()) {
-    hover_background_id =
-        IDR_ASH_BROWSER_WINDOW_CONTROL_BACKGROUND_MAXIMIZED_H;
-    pressed_background_id =
-        IDR_ASH_BROWSER_WINDOW_CONTROL_BACKGROUND_MAXIMIZED_P;
-  } else {
-    hover_background_id =
-        IDR_ASH_BROWSER_WINDOW_CONTROL_BACKGROUND_RESTORED_H;
-    pressed_background_id =
-        IDR_ASH_BROWSER_WINDOW_CONTROL_BACKGROUND_RESTORED_P;
-  }
-  caption_button_container_->SetButtonImages(
+void BrowserHeaderPainterAsh::UpdateCaptionButtons() {
+  caption_button_container_->SetButtonImage(
       ash::CAPTION_BUTTON_ICON_MINIMIZE,
-      IDR_AURA_WINDOW_CONTROL_ICON_MINIMIZE,
-      hover_background_id,
-      pressed_background_id);
+      IDR_AURA_WINDOW_CONTROL_ICON_MINIMIZE);
 
   int size_icon_id = 0;
-  if (frame_->IsMaximized() || frame_->IsFullscreen())
+  gfx::Size button_size;
+  if (frame_->IsMaximized() || frame_->IsFullscreen()) {
     size_icon_id = IDR_AURA_WINDOW_CONTROL_ICON_RESTORE;
-  else
+    button_size = GetAshLayoutSize(BROWSER_MAXIMIZED_HEADER_BUTTON);
+  } else {
     size_icon_id = IDR_AURA_WINDOW_CONTROL_ICON_MAXIMIZE;
-  caption_button_container_->SetButtonImages(
+    button_size = GetAshLayoutSize(BROWSER_RESTORED_HEADER_BUTTON);
+  }

[Peter Kasting, 2015/12/18 18:59:15]

Nit: Shorter:

  int size_icon_id = IDR_AURA_WINDOW_CONTROL_ICON_MAXIMIZE;
  gfx::Size button_size(GetAshLayoutSize(BROWSER_RESTORED_HEADER_BUTTON));
  if (frame_->IsMaximized() || frame_->IsFullscreen()) {
    size_icon_id = IDR_AURA_WINDOW_CONTROL_ICON_RESTORE;
    button_size = GetAshLayoutSize(BROWSER_MAXIMIZED_HEADER_BUTTON);
  }

[tdanderson, 2016/01/04 23:11:15]

Done.

+  caption_button_container_->SetButtonImage(
       ash::CAPTION_BUTTON_ICON_MAXIMIZE_RESTORE,
-      size_icon_id,
-      hover_background_id,
-      pressed_background_id);
+      size_icon_id);
+  caption_button_container_->SetButtonSize(button_size);

[Peter Kasting, 2015/12/18 18:59:15]

Nit: I'd put this call either above or below all the SetButtonImage() calls.

[tdanderson, 2016/01/04 23:11:15]

Done.

 
-  caption_button_container_->SetButtonImages(
+  caption_button_container_->SetButtonImage(
       ash::CAPTION_BUTTON_ICON_CLOSE,
-      IDR_AURA_WINDOW_CONTROL_ICON_CLOSE,
-      hover_background_id,
-      pressed_background_id);
-  caption_button_container_->SetButtonImages(
+      IDR_AURA_WINDOW_CONTROL_ICON_CLOSE);
+  caption_button_container_->SetButtonImage(
       ash::CAPTION_BUTTON_ICON_LEFT_SNAPPED,
-      IDR_AURA_WINDOW_CONTROL_ICON_LEFT_SNAPPED,
-      hover_background_id,
-      pressed_background_id);
-  caption_button_container_->SetButtonImages(
+      IDR_AURA_WINDOW_CONTROL_ICON_LEFT_SNAPPED);
+  caption_button_container_->SetButtonImage(
       ash::CAPTION_BUTTON_ICON_RIGHT_SNAPPED,
-      IDR_AURA_WINDOW_CONTROL_ICON_RIGHT_SNAPPED,
-      hover_background_id,
-      pressed_background_id);
+      IDR_AURA_WINDOW_CONTROL_ICON_RIGHT_SNAPPED);
 }
 
 gfx::Rect BrowserHeaderPainterAsh::GetPaintedBounds() const {