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

Unified Diff: chrome/renderer/spellchecker/spellcheck.cc

Issue 1227553002: Removed and replaced outdated mac-specific files and references. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@android_platform_skeleton
Patch Set: Comment updates for line length Created 5 years, 4 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 | « chrome/renderer/spellchecker/spellcheck.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/spellchecker/spellcheck.cc
diff --git a/chrome/renderer/spellchecker/spellcheck.cc b/chrome/renderer/spellchecker/spellcheck.cc
index f27f31e38570bba0a11bac83bdf586d7e407e643..db9fa7e2a090633fbbf7a97f8343b2a778501dd6 100644
--- a/chrome/renderer/spellchecker/spellcheck.cc
+++ b/chrome/renderer/spellchecker/spellcheck.cc
@@ -198,7 +198,7 @@ void SpellCheck::OnInit(
custom_dictionary_.Init(custom_words);
auto_spell_correct_turned_on_ = auto_spell_correct;
-#if !defined(OS_MACOSX)
+#if !defined(USE_BROWSER_SPELLCHECKER)
PostDelayedSpellCheckTask(pending_request_param_.release());
#endif
}
@@ -261,8 +261,8 @@ bool SpellCheck::SpellCheckWord(
bool SpellCheck::SpellCheckParagraph(
const base::string16& text,
WebVector<WebTextCheckingResult>* results) {
-#if !defined(OS_MACOSX)
- // Mac has its own spell checker, so this method will not be used.
+#if !defined(USE_BROWSER_SPELLCHECKER)
+ // Mac and Android have their own spell checkers,so this method won't be used
DCHECK(results);
std::vector<WebTextCheckingResult> textcheck_results;
size_t length = text.length();
@@ -301,7 +301,7 @@ bool SpellCheck::SpellCheckParagraph(
return false;
#else
// This function is only invoked for spell checker functionality that runs
- // on the render thread. OSX builds don't have that.
+ // on the render thread. OSX and Android builds don't have that.
NOTREACHED();
return true;
#endif
@@ -360,7 +360,8 @@ base::string16 SpellCheck::GetAutoCorrectionWord(const base::string16& word,
return autocorrect_word;
}
-#if !defined(OS_MACOSX) // OSX uses its own spell checker
+// OSX and Android use their own spell checkers
+#if !defined(USE_BROWSER_SPELLCHECKER)
void SpellCheck::RequestTextChecking(
const base::string16& text,
blink::WebTextCheckingCompletion* completion) {
@@ -382,7 +383,8 @@ bool SpellCheck::InitializeIfNeeded() {
return languages_.empty() ? true : languages_.front()->InitializeIfNeeded();
}
-#if !defined(OS_MACOSX) // OSX doesn't have |pending_request_param_|
+// OSX and Android don't have |pending_request_param_|
+#if !defined(USE_BROWSER_SPELLCHECKER)
void SpellCheck::PostDelayedSpellCheckTask(SpellcheckRequest* request) {
if (!request)
return;
@@ -393,7 +395,8 @@ void SpellCheck::PostDelayedSpellCheckTask(SpellcheckRequest* request) {
}
#endif
-#if !defined(OS_MACOSX) // Mac uses its platform engine instead.
+// Mac and Android use their platform engines instead.
+#if !defined(USE_BROWSER_SPELLCHECKER)
void SpellCheck::PerformSpellCheck(SpellcheckRequest* param) {
DCHECK(param);
« no previous file with comments | « chrome/renderer/spellchecker/spellcheck.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698