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

Side by Side Diff: third_party/WebKit/Source/core/editing/spellcheck/IdleSpellCheckCallbackTest.cpp

Issue 2701983002: Implement complete lifecycle transition for IdleSpellCheckCallback (Closed)
Patch Set: Fix compile error Created 3 years, 9 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 2017 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 "core/editing/spellcheck/IdleSpellCheckCallback.h"
6
7 #include "core/editing/spellcheck/SpellCheckTestBase.h"
8 #include "core/editing/spellcheck/SpellChecker.h"
9 #include "core/frame/LocalFrame.h"
10 #include "platform/RuntimeEnabledFeatures.h"
11
12 namespace blink {
13
14 using State = IdleSpellCheckCallback::State;
15
16 class IdleSpellCheckCallbackTest : public SpellCheckTestBase {
17 protected:
18 IdleSpellCheckCallback& idleChecker() {
19 return frame().spellChecker().idleSpellCheckCallback();
20 }
21
22 void SetUp() override {
23 SpellCheckTestBase::SetUp();
24 if (!RuntimeEnabledFeatures::idleTimeSpellCheckingEnabled())
25 idleChecker().documentAttached(frame().document());
26 }
27
28 void transitTo(State state) {
29 switch (state) {
30 case State::kInactive:
31 idleChecker().deactivate();
32 break;
33 case State::kHotModeRequested:
34 idleChecker().setNeedsInvocation();
35 break;
36 case State::kColdModeTimerStarted:
37 break;
38 case State::kColdModeRequested:
39 idleChecker().skipColdModeTimerForTesting();
40 break;
41 default:
tkent 2017/02/26 20:35:58 Do not use |default:| in |switch| for an enum. Pl
42 NOTREACHED();
43 }
44 }
45 };
46
47 // Test cases for lifecycle state transitions.
48
49 TEST_F(IdleSpellCheckCallbackTest, Initialization) {
50 EXPECT_EQ(State::kColdModeTimerStarted, idleChecker().state());
51 }
52
53 TEST_F(IdleSpellCheckCallbackTest, RequestWhenInactive) {
54 transitTo(State::kInactive);
55 idleChecker().setNeedsInvocation();
56 EXPECT_EQ(State::kHotModeRequested, idleChecker().state());
57 EXPECT_NE(-1, idleChecker().idleCallbackHandle());
58 }
59
60 TEST_F(IdleSpellCheckCallbackTest, RequestWhenHotModeRequested) {
61 transitTo(State::kHotModeRequested);
62 int handle = idleChecker().idleCallbackHandle();
63 idleChecker().setNeedsInvocation();
64 EXPECT_EQ(State::kHotModeRequested, idleChecker().state());
65 EXPECT_EQ(handle, idleChecker().idleCallbackHandle());
66 EXPECT_NE(-1, idleChecker().idleCallbackHandle());
67 }
68
69 TEST_F(IdleSpellCheckCallbackTest, RequestWhenColdModeTimerStarted) {
70 transitTo(State::kColdModeTimerStarted);
71 idleChecker().setNeedsInvocation();
72 EXPECT_EQ(State::kHotModeRequested, idleChecker().state());
73 EXPECT_NE(-1, idleChecker().idleCallbackHandle());
74 }
75
76 TEST_F(IdleSpellCheckCallbackTest, RequestWhenColdModeRequested) {
77 transitTo(State::kColdModeRequested);
78 int handle = idleChecker().idleCallbackHandle();
79 idleChecker().setNeedsInvocation();
80 EXPECT_EQ(State::kHotModeRequested, idleChecker().state());
81 EXPECT_EQ(handle, idleChecker().idleCallbackHandle());
82 EXPECT_NE(-1, idleChecker().idleCallbackHandle());
83 }
84
85 TEST_F(IdleSpellCheckCallbackTest, HotModeTransitToColdMode) {
86 transitTo(State::kHotModeRequested);
87 idleChecker().forceInvocationForTesting();
88 EXPECT_EQ(State::kColdModeTimerStarted, idleChecker().state());
89 }
90
91 TEST_F(IdleSpellCheckCallbackTest, ColdModeTimerStartedToRequested) {
92 transitTo(State::kColdModeTimerStarted);
93 idleChecker().skipColdModeTimerForTesting();
94 EXPECT_EQ(State::kColdModeRequested, idleChecker().state());
95 EXPECT_NE(-1, idleChecker().idleCallbackHandle());
96 }
97
98 TEST_F(IdleSpellCheckCallbackTest, ColdModeStayAtColdMode) {
99 transitTo(State::kColdModeRequested);
100 idleChecker().setNeedsMoreColdModeInvocationForTesting();
101 idleChecker().forceInvocationForTesting();
102 EXPECT_EQ(State::kColdModeTimerStarted, idleChecker().state());
103 }
104
105 TEST_F(IdleSpellCheckCallbackTest, ColdModeToInactive) {
106 transitTo(State::kColdModeRequested);
107 idleChecker().forceInvocationForTesting();
108 EXPECT_EQ(State::kInactive, idleChecker().state());
109 }
110
111 TEST_F(IdleSpellCheckCallbackTest, DetachWhenInactive) {
112 transitTo(State::kInactive);
113 document().shutdown();
114 EXPECT_EQ(State::kInactive, idleChecker().state());
115 }
116
117 TEST_F(IdleSpellCheckCallbackTest, DetachWhenHotModeRequested) {
118 transitTo(State::kHotModeRequested);
119 document().shutdown();
120 EXPECT_EQ(State::kInactive, idleChecker().state());
121 }
122
123 TEST_F(IdleSpellCheckCallbackTest, DetachWhenColdModeTimerStarted) {
124 transitTo(State::kColdModeTimerStarted);
125 document().shutdown();
126 EXPECT_EQ(State::kInactive, idleChecker().state());
127 }
128
129 TEST_F(IdleSpellCheckCallbackTest, DetachWhenColdModeRequested) {
130 transitTo(State::kColdModeRequested);
131 document().shutdown();
132 EXPECT_EQ(State::kInactive, idleChecker().state());
133 }
134
135 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698