Patch 2: Added a preprocessor flag for platform spellcheck.

chrome/renderer/spellchecker/spellcheck_provider.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_RENDERER_SPELLCHECKER_SPELLCHECK_PROVIDER_H_
 #define CHROME_RENDERER_SPELLCHECKER_SPELLCHECK_PROVIDER_H_
 
 #include <vector>
 
 #include "base/id_map.h"
@@ skipping 73 matching lines @@
       const blink::WebVector<unsigned>& marker_offsets,
       blink::WebTextCheckingCompletion* completion) override;
 
   blink::WebString autoCorrectWord(
       const blink::WebString& misspelled_word) override;
   void showSpellingUI(bool show) override;
   bool isShowingSpellingUI() override;
   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_;
 
   // The last text sent to the browser process to spellcheck it and its
   // spellchecking results.
   base::string16 last_request_;
   blink::WebVector<blink::WebTextCheckingResult> last_results_;
 
   // True if the browser is showing the spelling panel for us.
   bool spelling_panel_visible_;
 
   // Weak pointer to shared (per RenderView) spellcheck data.
   SpellCheck* spellcheck_;
 
   DISALLOW_COPY_AND_ASSIGN(SpellCheckProvider);
 };
 
 #endif  // CHROME_RENDERER_SPELLCHECKER_SPELLCHECK_PROVIDER_H_