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

Unified Diff: chrome/browser/spellchecker/spell_check_host_impl_unittest.cc

Issue 2911253003: [Reland] Allow storing multiple replacements on SpellCheckResult (Closed)
Patch Set: Fix typo Created 3 years, 7 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: chrome/browser/spellchecker/spell_check_host_impl_unittest.cc
diff --git a/chrome/browser/spellchecker/spell_check_host_impl_unittest.cc b/chrome/browser/spellchecker/spell_check_host_impl_unittest.cc
index 22cadb67282e9d34ee23bf9eeb748214c932cd3e..a1500170119fdc0f97fd17923b535e702699d77b 100644
--- a/chrome/browser/spellchecker/spell_check_host_impl_unittest.cc
+++ b/chrome/browser/spellchecker/spell_check_host_impl_unittest.cc
@@ -63,7 +63,9 @@ TEST(SpellCheckHostImplTest, CustomSpellingResults) {
EXPECT_EQ(service_results[1].decoration, results[0].decoration);
EXPECT_EQ(service_results[1].location, results[0].location);
EXPECT_EQ(service_results[1].length, results[0].length);
- EXPECT_EQ(service_results[1].replacement, results[0].replacement);
+ EXPECT_EQ(service_results[1].replacements.size(),
+ results[0].replacements.size());
+ EXPECT_EQ(service_results[1].replacements[0], results[0].replacements[0]);
}
// Spelling corrections of words that are not in the custom dictionary should
@@ -84,6 +86,8 @@ TEST(SpellCheckHostImplTest, SpellingServiceResults) {
EXPECT_EQ(service_results[i].decoration, results[i].decoration);
EXPECT_EQ(service_results[i].location, results[i].location);
EXPECT_EQ(service_results[i].length, results[i].length);
- EXPECT_EQ(service_results[i].replacement, results[i].replacement);
+ EXPECT_EQ(service_results[i].replacements.size(),
+ results[i].replacements.size());
+ EXPECT_EQ(service_results[i].replacements, results[i].replacements);
Noel Gordon 2017/05/31 05:46:21 x[i].replacements -> x[i].replacements[0] here I t
}
}

Powered by Google App Engine
This is Rietveld 408576698