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

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

Issue 10695003: Disable using the Spelling service when spellchecking text on Mac. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 6 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/spelling_service_client_unittest.cc
===================================================================
--- chrome/browser/spellchecker/spelling_service_client_unittest.cc (revision 144326)
+++ chrome/browser/spellchecker/spelling_service_client_unittest.cc (working copy)
@@ -263,6 +263,7 @@
PrefService* pref = profile_.GetPrefs();
pref->SetBoolean(prefs::kEnableSpellCheck, true);
pref->SetBoolean(prefs::kSpellCheckUseSpellingService, true);
+ pref->SetString(prefs::kSpellCheckDictionary, "");
for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kTests); ++i) {
client_.SetHTTPRequest(kTests[i].request_type, kTests[i].request_text);
@@ -304,7 +305,10 @@
// SpellingServiceClient::IsAvailable() describes why this function returns
// false for suggestions.)
static const char* kSupported[] = {
- "", "en-AU", "en-CA", "en-GB", "en-US",
+ "",
+#if !defined(OS_MACOSX)
+ "en-AU", "en-CA", "en-GB", "en-US",
+#endif
};
for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kSupported); ++i) {
pref->SetString(prefs::kSpellCheckDictionary, kSupported[i]);
@@ -315,6 +319,9 @@
// On the other hand, this function returns true for suggestions and false for
// spellcheck for unsupported locales.
static const char* kUnsupported[] = {
+#if defined(OS_MACOSX)
+ "en-AU", "en-CA", "en-GB", "en-US",
+#endif
"af-ZA", "bg-BG", "ca-ES", "cs-CZ", "da-DK", "de-DE", "el-GR", "es-ES",
"et-EE", "fo-FO", "fr-FR", "he-IL", "hi-IN", "hr-HR", "hu-HU", "id-ID",
"it-IT", "lt-LT", "lv-LV", "nb-NO", "nl-NL", "pl-PL", "pt-BR", "pt-PT",

Powered by Google App Engine
This is Rietveld 408576698