Add ENABLE_PRINTING and disable it on Android.

chrome/browser/browser_about_handler.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/browser/browser_about_handler.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 28 matching lines @@
   chrome::kChromeUIInspectHost,
   chrome::kChromeUIMediaInternalsHost,
   chrome::kChromeUIMemoryHost,
   chrome::kChromeUINetInternalsHost,
   chrome::kChromeUINetworkViewCacheHost,
   chrome::kChromeUINewTabHost,
   chrome::kChromeUIOmniboxHost,
   chrome::kChromeUIPluginsHost,
   chrome::kChromeUIPolicyHost,
   chrome::kChromeUIPredictorsHost,
-  chrome::kChromeUIPrintHost,
   chrome::kChromeUIProfilerHost,
   chrome::kChromeUIQuotaInternalsHost,
   chrome::kChromeUISessionsHost,
   chrome::kChromeUISettingsHost,
   chrome::kChromeUIStatsHost,
   chrome::kChromeUISyncInternalsHost,
   chrome::kChromeUITaskManagerHost,
   chrome::kChromeUITermsHost,
   chrome::kChromeUITracingHost,
   chrome::kChromeUIVersionHost,
@@ skipping 11 matching lines @@
   chrome::kChromeUIImageBurnerHost,
   chrome::kChromeUIKeyboardOverlayHost,
   chrome::kChromeUILoginHost,
   chrome::kChromeUINetworkHost,
   chrome::kChromeUIOobeHost,
   chrome::kChromeUIOSCreditsHost,
   chrome::kChromeUIProxySettingsHost,
   chrome::kChromeUISystemInfoHost,
   chrome::kChromeUIWallpaperHost,
 #endif
+#if defined(ENABLE_PRINTING)
+  chrome::kChromeUIPrintHost,

[Lei Zhang, 2012/06/08 19:08:41]

Can you also change the reference to kChromeUIPrintHost to be behind
ENABLE_PRINTING in chrome/browser/ui/webui/chrome_web_ui_controller_factory.cc?

I think if you do that, then you can put kChromeUIPrintHost behind
ENABLE_PRINTING as well.

[Xianzhu, 2012/06/08 22:50:43]

Done. 
Normally we don't disable the declarations when upstreaming Android specific
code that disables some features to reduce the number of ifdefs.

+#endif
 };
 
 }  // namespace
 
 bool WillHandleBrowserAboutURL(GURL* url,
                                content::BrowserContext* browser_context) {
   // TODO(msw): Eliminate "about:*" constants and literals from code and tests,
   //            then hopefully we can remove this forced fixup.
   *url = URLFixerUpper::FixupURL(url->possibly_invalid_spec(), std::string());
 
@@ skipping 67 matching lines @@
   return false;
 }
 
 std::vector<std::string> ChromePaths() {
   std::vector<std::string> paths;
   paths.reserve(arraysize(kChromePaths));
   for (size_t i = 0; i < arraysize(kChromePaths); i++)
     paths.push_back(kChromePaths[i]);
   return paths;
 }