Make SpellCheckPanel compiled and used only on Mac

chrome/renderer/chrome_content_renderer_client.cc

 // 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.
 
 #include "chrome/renderer/chrome_content_renderer_client.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 146 matching lines @@
 #include "components/printing/renderer/print_web_view_helper.h"
 #include "printing/print_settings.h"
 #endif
 
 #if BUILDFLAG(ENABLE_PRINT_PREVIEW)
 #include "chrome/renderer/pepper/chrome_pdf_print_client.h"
 #endif
 
 #if BUILDFLAG(ENABLE_SPELLCHECK)
 #include "components/spellcheck/renderer/spellcheck.h"
+#include "components/spellcheck/renderer/spellcheck_provider.h"
+
+#if BUILDFLAG(HAS_SPELLCHECK_PANEL)
 #include "components/spellcheck/renderer/spellcheck_panel.h"
-#include "components/spellcheck/renderer/spellcheck_provider.h"
+#endif

[Lei Zhang, 2017/04/14 00:55:38]

Which #endif is which? Once you have nested #ifs, I would recommend adding
comments to them.

[Xiaocheng, 2017/04/14 01:20:18]

Done.

 #endif
 
 #if BUILDFLAG(ENABLE_WEBRTC)
 #include "chrome/renderer/media/webrtc_logging_message_filter.h"
 #endif
 
 using autofill::AutofillAgent;
 using autofill::PasswordAutofillAgent;
 using autofill::PasswordGenerationAgent;
 using base::ASCIIToUTF16;
@@ skipping 410 matching lines @@
   ChromeExtensionsRendererClient::GetInstance()->RenderViewCreated(render_view);
 #endif
 #if BUILDFLAG(ENABLE_SPELLCHECK)
   // This is a workaround keeping the behavior that, the Blink side spellcheck
   // enabled state is initialized on RenderView creation.
   // TODO(xiaochengh): Design better way to sync between Chrome-side and
   // Blink-side spellcheck enabled states.  See crbug.com/710097.
   if (SpellCheckProvider* provider =
           SpellCheckProvider::Get(render_view->GetMainRenderFrame()))
     provider->EnableSpellcheck(spellcheck_->IsSpellcheckEnabled());
+
+#if BUILDFLAG(HAS_SPELLCHECK_PANEL)
   new SpellCheckPanel(render_view);
 #endif
+#endif
   new prerender::PrerendererClient(render_view);
 
   new ChromeRenderViewObserver(render_view, web_cache_impl_.get());
 
   new password_manager::CredentialManagerClient(render_view);
 }
 
 
 SkBitmap* ChromeContentRendererClient::GetSadPluginBitmap() {
   return const_cast<SkBitmap*>(ResourceBundle::GetSharedInstance().
@@ skipping 934 matching lines @@
 
   RecordYouTubeRewriteUMA(result);
   return corrected_url.ReplaceComponents(r);
 }
 
 std::unique_ptr<base::TaskScheduler::InitParams>
 ChromeContentRendererClient::GetTaskSchedulerInitParams() {
   return task_scheduler_util::
       GetRendererTaskSchedulerInitParamsFromCommandLine();
 }