Add ripple effect to action buttons in new-style notification.

ui/message_center/views/notification_view_md.cc

 // Copyright 2017 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 "ui/message_center/views/notification_view_md.h"
 
 #include <stddef.h>
 
 #include "base/strings/string_util.h"
 #include "ui/base/cursor/cursor.h"
@@ skipping 31 matching lines @@
 
 namespace message_center {
 
 namespace {
 
 // Dimensions.
 constexpr gfx::Insets kContentRowPadding(4, 12, 12, 12);
 constexpr gfx::Insets kActionsRowPadding(8, 8, 8, 8);
 constexpr int kActionsRowHorizontalSpacing = 8;
 constexpr gfx::Insets kImageContainerPadding(0, 12, 12, 12);
+constexpr gfx::Insets kActionButtonPadding(0, 12, 0, 12);
+constexpr gfx::Size kActionButtonMinSize(88, 32);
 
 // Foreground of small icon image.
 constexpr SkColor kSmallImageBackgroundColor = SK_ColorWHITE;
 // Background of small icon image.
 const SkColor kSmallImageColor = SkColorSetRGB(0x43, 0x43, 0x43);
 // Background of inline actions area.
 const SkColor kActionsRowBackgroundColor = SkColorSetRGB(0xee, 0xee, 0xee);
+// Ink drop color of action buttons.
+const SkColor kActionButtonInkDropBaseColor = SkColorSetRGB(0x0, 0x0, 0x0);
+// Ink drop opacity of action buttons when the ripple is visible.
+const float kActionButtonInkDropVisibleOpacity = 0.08f;
+// Text color of action button.
+const SkColor kActionButtonTextColor = SkColorSetRGB(0x33, 0x67, 0xD6);
 
 // Max number of lines for message_view_.
 constexpr int kMaxLinesForMessageView = 1;
 constexpr int kMaxLinesForExpandedMessageView = 4;
 
 constexpr int kListNotificationOverflowIndicatorSpacing = 4;
 constexpr int kCompactTitleMessageViewSpacing = 12;
 
 constexpr int kProgressBarHeight = 4;
 
@@ skipping 603 matching lines @@
     action_buttons_.clear();
   }
 
   DCHECK_EQ(this, actions_row_->parent());
 
   for (size_t i = 0; i < buttons.size(); ++i) {
     ButtonInfo button_info = buttons[i];
     if (new_buttons) {
       views::LabelButton* button = new views::LabelButton(
           this, button_info.title, views::style::CONTEXT_BUTTON_MD);
+      button->SetInkDropMode(views::LabelButton::InkDropMode::ON);
+      button->set_has_ink_drop_action_on_click(true);
+      // TODO(tetsui): Hover background color is slightly different from the
+      // mock. Investigate how to set hover color and ripple color separately.

[fukino, 2017/06/28 02:11:57]

(record from offline discussion) As this change is not urgent, could you
investigate how the inkdrop effect and hover color works?

[tetsui, 2017/06/28 06:30:24]

Done. Hover color is precisely accurate. Ripple color is slightly different from
the mock, but I think the original mock RGB color might not be accurate here.
(It depends on alpha blend formula? gamma? I'm not familiar with them...)

Ripple color in mock: #CFCFCF
Ripple color in the implementation: #CACACA
Manual calculation: 0xee * (1-0.08) * (1-0.08) = #C9C9C9

+      button->set_ink_drop_base_color(kActionButtonInkDropBaseColor);
+      button->set_ink_drop_visible_opacity(kActionButtonInkDropVisibleOpacity);
+      button->SetEnabledTextColors(kActionButtonTextColor);
+      button->SetBorder(views::CreateEmptyBorder(kActionButtonPadding));
+      button->SetMinSize(kActionButtonMinSize);
       button->SetFocusForPlatform();
       action_buttons_.push_back(button);
       actions_row_->AddChildView(button);
     } else {
       action_buttons_[i]->SetText(button_info.title);
       action_buttons_[i]->SchedulePaint();
       action_buttons_[i]->Layout();
     }
   }
 
@@ skipping 81 matching lines @@
                                   header_row_->expand_button()->HasFocus()) ||
                                  (header_row_->IsCloseButtonEnabled() &&
                                   header_row_->close_button()->HasFocus()) ||
                                  (header_row_->IsSettingsButtonEnabled() &&
                                   header_row_->settings_button()->HasFocus());
 
   header_row_->SetControlButtonsVisible(target_visibility);
 }
 
 }  // namespace message_center