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

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

Issue 2848943002: Allow storing multiple replacements on SpellCheckResult (Closed)
Patch Set: Created 3 years, 8 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/spellcheck_message_filter_unittest.cc
diff --git a/chrome/browser/spellchecker/spellcheck_message_filter_unittest.cc b/chrome/browser/spellchecker/spellcheck_message_filter_unittest.cc
index 62d445f9a7e1c218b5c92c81a31f12f19e77e452..a7eda6494b958cd9246498de00204ca40c0bc950 100644
--- a/chrome/browser/spellchecker/spellcheck_message_filter_unittest.cc
+++ b/chrome/browser/spellchecker/spellcheck_message_filter_unittest.cc
@@ -96,9 +96,11 @@ TEST(SpellCheckMessageFilterTest, OnTextCheckCompleteTestCustomDictionary) {
std::vector<SpellCheckResult> results;
results.push_back(SpellCheckResult(
- SpellCheckResult::SPELLING, 0, 6, base::ASCIIToUTF16("Hello")));
+ SpellCheckResult::SPELLING, 0, 6,
+ std::vector<base::string16>({base::ASCIIToUTF16("Hello")})));
results.push_back(
- SpellCheckResult(kDecoration, kLocation, kLength, kReplacement));
+ SpellCheckResult(kDecoration, kLocation, kLength,
+ std::vector<base::string16>({kReplacement})));
scoped_refptr<TestingSpellCheckMessageFilter> filter(
new TestingSpellCheckMessageFilter);
@@ -121,15 +123,17 @@ TEST(SpellCheckMessageFilterTest, OnTextCheckCompleteTestCustomDictionary) {
EXPECT_EQ(kDecoration, sent_results[0].decoration);
EXPECT_EQ(kLocation, sent_results[0].location);
EXPECT_EQ(kLength, sent_results[0].length);
- EXPECT_EQ(kReplacement, sent_results[0].replacement);
+ EXPECT_EQ(kReplacement, sent_results[0].replacements[0]);
}
TEST(SpellCheckMessageFilterTest, OnTextCheckCompleteTest) {
std::vector<SpellCheckResult> results;
results.push_back(SpellCheckResult(
- SpellCheckResult::SPELLING, 0, 6, base::ASCIIToUTF16("Hello")));
+ SpellCheckResult::SPELLING, 0, 6,
+ std::vector<base::string16>({base::ASCIIToUTF16("Hello")})));
results.push_back(SpellCheckResult(
- SpellCheckResult::SPELLING, 7, 7, base::ASCIIToUTF16("world")));
+ SpellCheckResult::SPELLING, 7, 7,
+ std::vector<base::string16>({base::ASCIIToUTF16("world")})));
scoped_refptr<TestingSpellCheckMessageFilter> filter(
new TestingSpellCheckMessageFilter);

Powered by Google App Engine
This is Rietveld 408576698