Allow InkDropRipple to co-exist with highlight

ui/views/animation/ink_drop_impl_unittest.cc

 // 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.
 
 #include "ui/views/animation/ink_drop_impl.h"
 
 #include "base/macros.h"
 #include "base/test/test_simple_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 276 matching lines @@
 TEST_F(InkDropImplTest, FocusHighlightComesBackImmediatelyAfterAction) {
   ink_drop_host_.set_should_show_highlight(true);
 
   EXPECT_EQ(0, ink_drop_host_.num_ink_drop_layers());
 
   ink_drop_.SetFocused(true);
   EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
 
   ink_drop_.AnimateToState(InkDropState::ACTION_PENDING);
   EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+  EXPECT_FALSE(test_api_.IsHighlightFadingInOrVisible());
 
   ink_drop_.AnimateToState(InkDropState::ACTION_TRIGGERED);
   EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
 
   test_api_.CompleteAnimations();
 
   // No delay (unlike in the hover case).
   EXPECT_FALSE(task_runner_->HasPendingTask());
 
   // Highlight should be back.
   EXPECT_TRUE(test_api_.IsHighlightFadingInOrVisible());
 }
 
+TEST_F(InkDropImplTest, HighlightCanCoexistWithRipple) {
+  ink_drop_host_.set_should_show_highlight(true);
+  ink_drop_host_.set_ripple_overrides_highlight(false);
+
+  EXPECT_EQ(0, ink_drop_host_.num_ink_drop_layers());
+
+  ink_drop_.SetHovered(true);
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+
+  ink_drop_.AnimateToState(InkDropState::ACTION_PENDING);
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+  EXPECT_TRUE(test_api_.IsHighlightFadingInOrVisible());
+
+  ink_drop_.AnimateToState(InkDropState::ACTION_TRIGGERED);
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+  EXPECT_TRUE(test_api_.IsHighlightFadingInOrVisible());
+
+  test_api_.CompleteAnimations();
+
+  // Nothing to fade in because the highlight has always been visible.
+  EXPECT_FALSE(task_runner_->HasPendingTask());
+  EXPECT_TRUE(test_api_.IsHighlightFadingInOrVisible());
+
+  // Now try with the ripple showing before the highlight comes in.
+  ink_drop_.AnimateToState(InkDropState::HIDDEN);
+  ink_drop_.SetHovered(false);
+  test_api_.CompleteAnimations();
+  EXPECT_EQ(0, ink_drop_host_.num_ink_drop_layers());
+
+  ink_drop_.AnimateToState(InkDropState::ACTION_PENDING);
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());

[bruthig, 2016/09/09 16:05:47]

Shouldn't this be 0?

[Evan Stade, 2016/09/09 18:55:48]

why? we're animating the ripple in

[bruthig, 2016/09/09 19:30:05]

I guess I was thinking of the IsVisible() clause but that's probably overkill,
withdrawn.

+  ink_drop_.SetHovered(true);
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+  EXPECT_TRUE(test_api_.IsHighlightFadingInOrVisible());
+}
+
+TEST_F(InkDropImplTest, HighlightCanCoexistWithSnapToActivatedRipple) {
+  ink_drop_host_.set_should_show_highlight(true);
+  ink_drop_host_.set_ripple_overrides_highlight(false);
+
+  EXPECT_EQ(0, ink_drop_host_.num_ink_drop_layers());
+
+  ink_drop_.SetHovered(true);
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+
+  ink_drop_.SnapToActivated();
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+  EXPECT_TRUE(test_api_.IsHighlightFadingInOrVisible());
+
+  // Now try with the ripple showing before the highlight comes in.
+  ink_drop_.AnimateToState(InkDropState::HIDDEN);
+  ink_drop_.SetHovered(false);
+  test_api_.CompleteAnimations();
+  EXPECT_EQ(0, ink_drop_host_.num_ink_drop_layers());
+
+  ink_drop_.SnapToActivated();
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());

[bruthig, 2016/09/09 16:05:47]

Shouldn't this be 0?

[Evan Stade, 2016/09/09 18:55:48]

after snapping to activated?

[bruthig, 2016/09/09 19:30:05]

Nevermind, I thought it was similar to above.

+  ink_drop_.SetHovered(true);
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+  EXPECT_TRUE(test_api_.IsHighlightFadingInOrVisible());
+}
+
+TEST_F(InkDropImplTest, HighlightCanExistWithoutRipple) {
+  ink_drop_host_.set_should_show_highlight(true);
+  ink_drop_host_.set_should_show_ripple(false);
+
+  EXPECT_EQ(0, ink_drop_host_.num_ink_drop_layers());
+
+  ink_drop_.SetHovered(true);
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+
+  ink_drop_.AnimateToState(InkDropState::ACTION_PENDING);
+  EXPECT_EQ(InkDropState::HIDDEN, ink_drop_.GetTargetInkDropState());
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+  EXPECT_TRUE(test_api_.IsHighlightFadingInOrVisible());
+
+  ink_drop_.AnimateToState(InkDropState::ACTION_TRIGGERED);
+  EXPECT_EQ(InkDropState::HIDDEN, ink_drop_.GetTargetInkDropState());
+  EXPECT_EQ(1, ink_drop_host_.num_ink_drop_layers());
+  EXPECT_TRUE(test_api_.IsHighlightFadingInOrVisible());
+
+  // Nothing to fade in because the highlight has always been visible.
+  EXPECT_FALSE(task_runner_->HasPendingTask());
+  EXPECT_TRUE(test_api_.IsHighlightFadingInOrVisible());
+}
 }  // namespace views