Use MD-style focus indicator on DL shelf items.

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"
@@ skipping 87 matching lines @@
 
 int GetCloseAndLinkPadding() {
   return ui::MaterialDesignController::IsModeMaterial() ? kCloseAndLinkPaddingMd
                                                         : kCloseAndLinkPadding;
 }
 
 int GetBetweenItemPadding() {
   return ui::MaterialDesignController::IsModeMaterial() ? 0 : kDownloadPadding;
 }
 
-int GetTopBottomPadding() {
+int GetTopPadding() {
+  return ui::MaterialDesignController::IsModeMaterial() ? 1 : kTopBottomPadding;

[Evan Stade, 2016/04/11 19:22:44]

this change leaves room for the top separator.

+}
+
+int GetBottomPadding() {
   return ui::MaterialDesignController::IsModeMaterial() ? 0 : kTopBottomPadding;
 }
 
 // Sets size->width() to view's preferred width + size->width().s
 // Sets size->height() to the max of the view's preferred height and
 // size->height();
 void AdjustSize(views::View* view, gfx::Size* size) {
   gfx::Size view_preferred = view->GetPreferredSize();
   size->Enlarge(view_preferred.width(), 0);
   size->set_height(std::max(view_preferred.height(), size->height()));
 }
 
 int CenterPosition(int size, int target_size) {
-  return std::max((target_size - size) / 2, GetTopBottomPadding());
+  return std::max((target_size - size) / 2, GetTopPadding());
 }
 
 }  // namespace
 
 DownloadShelfView::DownloadShelfView(Browser* browser, BrowserView* parent)
     : browser_(browser),
       new_item_animation_(this),
       shelf_animation_(this),
       arrow_image_(nullptr),
       show_all_view_(nullptr),
@@ skipping 117 matching lines @@
 gfx::Size DownloadShelfView::GetPreferredSize() const {
   gfx::Size prefsize(
       GetEndPadding() + GetStartPadding() + GetCloseAndLinkPadding(), 0);
   AdjustSize(close_button_, &prefsize);
   AdjustSize(show_all_view_, &prefsize);
   // Add one download view to the preferred size.
   if (!download_views_.empty()) {
     AdjustSize(*download_views_.begin(), &prefsize);
     prefsize.Enlarge(GetBetweenItemPadding(), 0);
   }
-  prefsize.Enlarge(0, 2 * GetTopBottomPadding());
+  prefsize.Enlarge(0, GetTopPadding() + GetBottomPadding());
   if (shelf_animation_.is_animating()) {
     prefsize.set_height(
         static_cast<int>(static_cast<double>(prefsize.height()) *
                          shelf_animation_.GetCurrentValue()));
   }
   return prefsize;
 }
 
 void DownloadShelfView::AnimationProgressed(const gfx::Animation* animation) {
   if (animation == &new_item_animation_) {
@@ skipping 214 matching lines @@
 
 content::DownloadItem* DownloadShelfView::GetDownloadItemForView(size_t i) {
   if (ui::MaterialDesignController::IsModeMaterial())
     return static_cast<DownloadItemViewMd*>(download_views_[i])->download();
   return static_cast<DownloadItemView*>(download_views_[i])->download();
 }
 
 SkColor DownloadShelfView::GetTextColorForIconMd() {
   return DownloadItemViewMd::GetTextColorForThemeProvider(GetThemeProvider());
 }