Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(293)

Side by Side Diff: ui/views/animation/test/test_ink_drop.cc

Issue 2028303004: Hide hover effect on hidden ink drop host views. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix tests Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "ui/views/animation/test/test_ink_drop.h"
6
7 namespace views {
8
9 TestInkDrop::TestInkDrop() : state_(InkDropState::HIDDEN), is_hovered_(false) {}
10 TestInkDrop::~TestInkDrop() {}
11
12 InkDropState TestInkDrop::GetTargetInkDropState() const {
13 return state_;
14 }
15
16 bool TestInkDrop::IsVisible() const {
17 return true;
bruthig 2016/06/03 02:59:24 Shouldn't this be 'return state_ != InkDropState::
Evan Stade 2016/06/03 03:25:23 this function (on the InkDrop interface) is unused
bruthig 2016/06/03 14:30:50 Acknowledged.
18 }
19
20 void TestInkDrop::AnimateToState(InkDropState ink_drop_state) {
21 state_ = ink_drop_state;
22 }
23
24 void TestInkDrop::SnapToActivated() {
25 state_ = InkDropState::ACTIVATED;
26 }
27
28 void TestInkDrop::SetHovered(bool is_hovered) {
29 is_hovered_ = true;
30 }
31
32 void TestInkDrop::SetFocused(bool is_focused) {}
33
34 } // namespace views
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698