Adding ripple effect for clicks on MD slider

ui/views/controls/md_slider.h

 // Copyright 2016 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_CONTROLS_MD_SLIDER_H_
 #define UI_VIEWS_CONTROLS_MD_SLIDER_H_
 
 #include "base/macros.h"
 #include "ui/views/controls/slider.h"
 #include "ui/views/view.h"
@@ skipping 11 matching lines @@
   // ui::Slider:
   void UpdateState(bool control_on) override;
 
   // views::View:
   void OnPaint(gfx::Canvas* canvas) override;
   const char* GetClassName() const override;
 
  protected:
   // ui::Slider:
   int GetThumbWidth() override;
+  void SetHighlighted(bool is_highlighted) override;
 
  private:
+  // gfx::AnimationDelegate:
+  void AnimationProgressed(const gfx::Animation* animation) override;
+  void AnimationEnded(const gfx::Animation* animation) override;
+
   // Record whether the slider is in the active state or the disabled state.
   bool is_active_;
+
+  // Animating value of the current radius size of the thumb's highlight.
+  float thumb_highlight_radius_;
+
+  std::unique_ptr<gfx::SlideAnimation> highlight_animation_;

[bruthig, 2016/09/23 22:24:39]

nit: You should forward declare SlideAnimation above.  The only reason this
isn't currently a compile time error is because ui/views/controls/slider.h is
already forward declaring it, but you shouldn't be relying on that because
slider.h should be able to remove it without creating an error in this .h file.

[yiyix, 2016/09/24 06:24:36]

Done.

 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_MD_SLIDER_H_