Harmony: disable ripples on Mac

ui/views/style/platform_style.h

 // Copyright 2015 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.
 
 #ifndef UI_VIEWS_STYLE_PLATFORM_STYLE_H_
 #define UI_VIEWS_STYLE_PLATFORM_STYLE_H_
 
 #include <memory>
 
 #include "base/macros.h"
@@ skipping 35 matching lines @@
   // the left or right end of the text from the cursor, respectively.
   static const bool kTextfieldDragVerticallyDragsToEnd;
 
   // The menu button's action to show the menu.
   static const CustomButton::NotifyAction kMenuNotifyActivationAction;
 
   // Whether selecting a row in a TreeView selects the entire row or only the
   // label for that row.
   static const bool kTreeViewSelectionPaintsEntireRow;
 
+  // Whether ripples should be used for visual feedback on control activation.
+  static const bool kUseRipples;

[sky, 2016/08/25 23:44:50]

Is there a reason you're going with the more generic ripples instead of a more
specific kUseInkDrop? Are the two ever not the same?

[Elly Fong-Jones, 2016/08/26 15:46:44]

In principle, there could be other kinds of ripples later, I guess. The
terminology I used comes from the Harmony specs, which say "ripples are not used
on Mac".

+
   // Creates an ImageSkia containing the image to use for the combobox arrow.
   // The |is_enabled| argument is true if the control the arrow is for is
   // enabled, and false if the control is disabled. The |style| argument is the
   // style of the combobox the arrow is being drawn for.
   static gfx::ImageSkia CreateComboboxArrow(bool is_enabled,
                                             Combobox::Style style);
 
   // Creates the appropriate border for a focusable Combobox.
   static std::unique_ptr<FocusableBorder> CreateComboboxBorder();
 
@@ skipping 12 matching lines @@
   // Returns the current text color for the current button state.
   static SkColor TextColorForButton(const ButtonColorByState& color_by_state,
                                     const LabelButton& button);
 
   // Applies platform styles to |label| and fills |color_by_state| with the text
   // colors for normal, pressed, hovered, and disabled states, if the colors for
   // Button::STYLE_BUTTON buttons differ from those provided by ui::NativeTheme.
   static void ApplyLabelButtonTextStyle(Label* label,
                                         ButtonColorByState* color_by_state);
 
+  // Returns the background color that should be used for an MdTextButton or
+  // other MD controls when in the given state.
+  static SkColor BackgroundColorForMdButton(SkColor color,
+                                            Button::ButtonState state);
+
   // Applies the current system theme to the default border created by |button|.
   static std::unique_ptr<Border> CreateThemedLabelButtonBorder(
       LabelButton* button);
 
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(PlatformStyle);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_STYLE_PLATFORM_STYLE_H_