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

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

Issue 591313002: mac: Fix spellcheck test on OSX 10.10. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/spellchecker/spellcheck_platform_mac_unittest.cc
diff --git a/chrome/browser/spellchecker/spellcheck_platform_mac_unittest.cc b/chrome/browser/spellchecker/spellcheck_platform_mac_unittest.cc
index a5510a6fca2bd6606f9452ebe92986e254d1c684..a0c6893af80237f39ee0284b79c276cc2c02ac4b 100644
--- a/chrome/browser/spellchecker/spellcheck_platform_mac_unittest.cc
+++ b/chrome/browser/spellchecker/spellcheck_platform_mac_unittest.cc
@@ -136,7 +136,10 @@ TEST_F(SpellcheckMacTest, SpellCheckSuggestions_EN_US) {
{"beleive", "believe"},
{"bellweather", "bellwether"},
{"benifit", "benefit"},
- {"bouy", "buoy"},
+ // This particular spelling correction was removed in OSX 10.10. Replacing
+ // it with another spelling correction that also works on older OSes.
+ // {"bouy", "buoy"},
+ {"bouy", "bout"},
Avi (use Gerrit) 2014/09/23 23:15:09 How far back does "bout" work? I'd personally kill
groby-ooo-7-16 2014/09/23 23:17:32 +1 from spellchecking.
erikchen 2014/09/23 23:28:19 I don't plan on merging this change until I've tes
{"briliant", "brilliant"},
{"burgler", "burglar"},
{"camoflage", "camouflage"},
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698