Avoid to change layer type in InkDropHostView::AddInkDropLayer

ui/views/animation/ink_drop_host_view.cc

Index: ui/views/animation/ink_drop_host_view.cc
diff --git a/ui/views/animation/ink_drop_host_view.cc b/ui/views/animation/ink_drop_host_view.cc
index 4d56b362811719b2946030e148d45c7edd21af21..160351773c2f35f9a1b40d52799d20d9f8726ee9 100644
--- a/ui/views/animation/ink_drop_host_view.cc
+++ b/ui/views/animation/ink_drop_host_view.cc
@@ -126,6 +126,8 @@ InkDropHostView::~InkDropHostView() {
 }
 
 void InkDropHostView::AddInkDropLayer(ui::Layer* ink_drop_layer) {
+  // Prevent its subclass from created a non textured layer on this view.
+  DCHECK(!layer() || layer()->type() == ui::LAYER_TEXTURED);
   old_paint_to_layer_ = layer() != nullptr;
   SetPaintToLayer();

[sky, 2017/02/23 00:58:35]

Is this the problematic line? If we only do this then can we nuke the DCHECK?

[sky, 2017/02/24 00:58:44]

What goes wrong if a solid color layer was created?

   layer()->SetFillsBoundsOpaquely(false);