Patch 2: Added a preprocessor flag for platform spellcheck.

chrome/renderer/spellchecker/spellcheck_provider.h

Index: chrome/renderer/spellchecker/spellcheck_provider.h
diff --git a/chrome/renderer/spellchecker/spellcheck_provider.h b/chrome/renderer/spellchecker/spellcheck_provider.h
index 793b39187dbe1b1354dffee0470bd6d413e07aa2..0abaacbec2d835335ace9396cbd11fa8ebae1c58 100644
--- a/chrome/renderer/spellchecker/spellcheck_provider.h
+++ b/chrome/renderer/spellchecker/spellcheck_provider.h
@@ -91,25 +91,25 @@ class SpellCheckProvider
   void updateSpellingUIWithMisspelledWord(
       const blink::WebString& word) override;
 
-#if !defined(OS_MACOSX)
+#if !defined(USE_PLATFORM_SPELLCHECKER)
   void OnRespondSpellingService(
       int identifier,
       bool succeeded,
       const base::string16& text,
       const std::vector<SpellCheckResult>& results);
-#endif
+#endif // !USE_PLATFORM_SPELLCHECKER
 
   // Returns whether |text| has word characters, i.e. whether a spellchecker
   // needs to check this text.
   bool HasWordCharacters(const base::string16& text, int index) const;
 
-#if defined(OS_MACOSX)
+#if defined(USE_PLATFORM_SPELLCHECKER)
   void OnAdvanceToNextMisspelling();
   void OnRespondTextCheck(
       int identifier,
       const std::vector<SpellCheckResult>& results);
   void OnToggleSpellPanel(bool is_currently_visible);
-#endif
+#endif // USE_PLATFORM_SPELLCHECKER

[please use gerrit instead, 2015/07/02 04:27:06]

Here and everywhere else: Let's try to preserve the existing styling of
#if-#else-#endif, because altering the style is not the purpose of this patch.
For example, please do not add new // after #endif.

[dylanking, 2015/07/07 02:31:22]

Fixed in https://codereview.chromium.org/1227553002 so that I could apply the
changes to all the new or already modified code that's not in this CL.

[dylanking, 2015/07/07 19:02:15]

Now fixed here, too.

 
   // Holds ongoing spellchecking operations, assigns IDs for the IPC routing.
   WebTextCheckCompletions text_check_completions_;