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

Unified Diff: third_party/WebKit/Source/core/editing/spellcheck/SpellCheckerTest.cpp

Issue 2748083006: Minor revision related to SpellChecker::advanceToNextMisspelling() (Closed)
Patch Set: Wed Mar 15 11:19:30 PDT 2017 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/editing/spellcheck/SpellCheckerTest.cpp
diff --git a/third_party/WebKit/Source/core/editing/spellcheck/SpellCheckerTest.cpp b/third_party/WebKit/Source/core/editing/spellcheck/SpellCheckerTest.cpp
index 1b6fbca103e33089c6677e7eeecb86c8c956ebde..a4f6a1378bbf1cf5fb78c1ea1164752e98fc6b6a 100644
--- a/third_party/WebKit/Source/core/editing/spellcheck/SpellCheckerTest.cpp
+++ b/third_party/WebKit/Source/core/editing/spellcheck/SpellCheckerTest.cpp
@@ -35,9 +35,8 @@ TEST_F(SpellCheckerTest, AdvanceToNextMisspellingWithEmptyInputNoCrash) {
updateAllLifecyclePhases();
Element* input = document().querySelector("input");
input->focus();
- // Do not crash in AdvanceToNextMisspelling command.
- EXPECT_TRUE(
- document().frame()->editor().executeCommand("AdvanceToNextMisspelling"));
+ // Do not crash in advanceToNextMisspelling.
+ document().frame()->spellChecker().advanceToNextMisspelling(false);
}
// Regression test for crbug.com/701309

Powered by Google App Engine
This is Rietveld 408576698