Make all text buttons on download shelf work well with themes, not just

chrome/browser/ui/views/download/download_item_view.cc

 // Copyright (c) 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/download/download_item_view.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <vector>
@@ skipping 785 matching lines @@
   LoadIcon();
 }
 
 void DownloadItemView::UpdateColorsFromTheme() {
   if (!GetThemeProvider())
     return;
 
   SetBorder(base::MakeUnique<SeparatorBorder>(GetThemeProvider()->GetColor(
       ThemeProperties::COLOR_TOOLBAR_VERTICAL_SEPARATOR)));
 
-  SkColor text_color = GetTextColor();
   if (dangerous_download_label_)
-    dangerous_download_label_->SetEnabledColor(text_color);
+    dangerous_download_label_->SetEnabledColor(GetTextColor());
   if (save_button_)
-    save_button_->SetEnabledTextColors(text_color);
+    shelf_->ConfigureButtonForTheme(save_button_);
   if (discard_button_)
-    discard_button_->SetEnabledTextColors(text_color);
+    shelf_->ConfigureButtonForTheme(discard_button_);
 }
 
 void DownloadItemView::ShowContextMenuImpl(const gfx::Rect& rect,
                                            ui::MenuSourceType source_type) {
   // Similar hack as in MenuButton.
   // We're about to show the menu from a mouse press. By showing from the
   // mouse press event we block RootView in mouse dispatching. This also
   // appears to cause RootView to get a mouse pressed BEFORE the mouse
   // release is seen, which means RootView sends us another mouse press no
   // matter where the user pressed. To force RootView to recalculate the
@@ skipping 285 matching lines @@
     SchedulePaint();
 }
 
 SkColor DownloadItemView::GetTextColor() const {
   return GetTextColorForThemeProvider(GetThemeProvider());
 }
 
 SkColor DownloadItemView::GetDimmedTextColor() const {
   return SkColorSetA(GetTextColor(), 0xC7);
 }