Adjust shadow and stroke of MdTextButton.

chrome/browser/ui/views/download/download_shelf_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_shelf_view.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/logging.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/download/download_item_model.h"
 #include "chrome/browser/download/download_stats.h"
 #include "chrome/browser/themes/theme_properties.h"
+#include "chrome/browser/themes/theme_service.h"
+#include "chrome/browser/themes/theme_service_factory.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/chrome_pages.h"
 #include "chrome/browser/ui/view_ids.h"
 #include "chrome/browser/ui/views/download/download_item_view.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
 #include "chrome/grit/generated_resources.h"
 #include "chrome/grit/theme_resources.h"
 #include "components/strings/grit/components_strings.h"
 #include "content/public/browser/download_item.h"
 #include "content/public/browser/download_manager.h"
@@ skipping 277 matching lines @@
                         kStartPadding;
   if (available_width <= 0)
     return false;
 
   gfx::Size item_size = (*download_views_.rbegin())->GetPreferredSize();
   return item_size.width() < available_width;
 }
 
 void DownloadShelfView::UpdateColorsFromTheme() {
   if (show_all_view_ && close_button_ && GetThemeProvider()) {
-    set_background(views::Background::CreateSolidBackground(
-        GetThemeProvider()->GetColor(ThemeProperties::COLOR_TOOLBAR)));
+    const SkColor bg_color =
+        GetThemeProvider()->GetColor(ThemeProperties::COLOR_TOOLBAR);
+    set_background(views::Background::CreateSolidBackground(bg_color));
     show_all_view_->SetEnabledTextColors(
         GetThemeProvider()->GetColor(ThemeProperties::COLOR_BOOKMARK_TEXT));
+    if (ThemeServiceFactory::GetForProfile(browser_->profile())
+            ->UsingDefaultTheme()) {
+      // For the normal theme, just use the default button bg color.
+      show_all_view_->SetBgColorOverride(base::Optional<SkColor>());
+    } else {
+      // For custom themes, we have to make up a background color for the
+      // button. Use a slight tint of the shelf background.
+      show_all_view_->SetBgColorOverride(
+          color_utils::BlendTowardOppositeLuma(bg_color, 0x10));
+    }
+    show_all_view_->SchedulePaint();

[sky, 2016/10/06 17:09:45]

In general I would expect changing properties on views to trigger the necessary
painting for you. In this case that means SetBgColorOverriding should be the one
doing that. I think View::set_background() should as well.

[Evan Stade, 2016/10/06 17:43:37]

this was leftover from a previous iteration of the code (before I changed
set_bg_color_override to SetBgColorOverride). Removed.

   }
 }
 
 void DownloadShelfView::OnThemeChanged() {
   UpdateColorsFromTheme();
 }
 
 void DownloadShelfView::LinkClicked(views::Link* source, int event_flags) {
   chrome::ShowDownloads(browser_);
 }
@@ skipping 66 matching lines @@
   SetVisible(false);
 }
 
 bool DownloadShelfView::CanAutoClose() {
   for (size_t i = 0; i < download_views_.size(); ++i) {
     if (!download_views_[i]->download()->GetOpened())
       return false;
   }
   return true;
 }