Add MD ink drop ripple to app list button

ui/views/animation/ink_drop_host_view.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_ANIMATION_INK_DROP_HOST_VIEW_H_
 #define UI_VIEWS_ANIMATION_INK_DROP_HOST_VIEW_H_
 
 #include <memory>
 
 #include "third_party/skia/include/core/SkColor.h"
@@ skipping 20 matching lines @@
 
   // Overridden from InkDropHost:
   void AddInkDropLayer(ui::Layer* ink_drop_layer) override;
   void RemoveInkDropLayer(ui::Layer* ink_drop_layer) override;
   std::unique_ptr<InkDropRipple> CreateInkDropRipple() const override;
   std::unique_ptr<InkDropHighlight> CreateInkDropHighlight() const override;
 
   void set_ink_drop_size(const gfx::Size& size) { ink_drop_size_ = size; }
 
  protected:
+  // Used in SetHasInkDrop() to specify whether the ink drop effect is enabled
+  // or not for the view. In case of having an ink drop, it also specifies
+  // whether the default gesture event handler for the ink drop should be
+  // installed or the subclass will handle gesture events itself.
+  enum InkDropType {
+    NO_INK_DROP,
+    INK_DROP_WITHOUT_GESTURE_HANDLING,
+    INK_DROP_WITH_GESTURE_HANDLING,
+  };
+
   static const int kInkDropSmallCornerRadius;
 
   void set_ink_drop_visible_opacity(float visible_opacity) {
     ink_drop_visible_opacity_ = visible_opacity;
   }
   float ink_drop_visible_opacity() const { return ink_drop_visible_opacity_; }
 
   // Returns the default InkDropRipple centered on |center_point|.
   std::unique_ptr<InkDropRipple> CreateDefaultInkDropRipple(
       const gfx::Point& center_point) const;
@@ skipping 26 matching lines @@
   virtual SkColor GetInkDropBaseColor() const;
 
   // Should return true if the ink drop is also used to depict focus.
   virtual bool ShouldShowInkDropForFocus() const;
 
   InkDrop* ink_drop() { return ink_drop_.get(); }
 
   // Toggle to enable/disable an InkDrop on this View.  Descendants can override
   // CreateInkDropHighlight() and CreateInkDropRipple() to change the look/feel
   // of the InkDrop.
-  void SetHasInkDrop(bool has_an_ink_drop);
+  void SetHasInkDrop(InkDropType ink_drop_type);

[sky, 2016/06/29 22:58:34]

SetHasInkDrop makes sense when the type is boolean, but now that it's an enum I
think SetInkDropType() makes more sense. WDYT?

[mohsen, 2016/06/30 00:03:45]

Yeah, I had thought of SetInkDropType(), but that might be confused with
different types of InkDrop's and InkDropRipple's that we have. But, I think
InkDropMode and SetInkDropMode() should be good. Renamed.

 
  private:
   class InkDropGestureHandler;
   friend class InkDropGestureHandler;
   friend class test::InkDropHostViewTestApi;
 
   // The last user Event to trigger an ink drop ripple animation.
   std::unique_ptr<ui::LocatedEvent> last_ripple_triggering_event_;
 
   std::unique_ptr<InkDrop> ink_drop_;
@@ skipping 10 matching lines @@
   // drop layer.
   bool old_paint_to_layer_;
 
   bool destroying_;
 
   DISALLOW_COPY_AND_ASSIGN(InkDropHostView);
 };
 }  // namespace views
 
 #endif  // UI_VIEWS_ANIMATION_INK_DROP_HOST_VIEW_H_