Generate the print messages in components/printing

chrome/common/BUILD.gn

 # Copyright 2014 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.
 
 import("//tools/grit/grit_rule.gni")
 
 gypi_values = exec_script("//build/gypi_to_gn.py",
                           [ rebase_path("../chrome_common.gypi") ],
                           "scope",
                           [ "../chrome_common.gypi" ])
@@ skipping 41 matching lines @@
     "//chrome/app/theme:theme_resources",
     "//chrome/common:constants",
     "//chrome/common/net",
     "//chrome/common/safe_browsing:proto",
     "//chrome/installer/util",
     "//components/cloud_devices/common",
     "//components/content_settings/core/common",
     "//components/json_schema",
     "//components/metrics",
     "//components/policy:policy_component_common",
-    "//components/printing/common:printing_common",
     "//components/translate/core/common",
     "//components/variations",
     "//content/public/common",
     "//crypto",
     "//extensions/common:common_constants",
     "//net",
     "//skia",
     "//third_party/icu",
     "//third_party/libxml",
     "//third_party/sqlite",
@@ skipping 56 matching lines @@
         rebase_path(gypi_values.chrome_common_mac_sources, ".", "//chrome")
   }
 
   if (enable_nacl) {
     deps += [
       #'<(DEPTH)/components/nacl.gyp:nacl_common',  TODO(GYP)
     ]
   }
 
   # Printing.
-  if (!enable_basic_printing && !enable_print_preview) {
-    sources -= [
-      "print_messages.cc",
-      "print_messages.h",
+  if (enable_basic_printing || enable_print_preview) {
+    deps += [
+      "//components/printing/common:printing_common",

[Vitaly Buka (NO REVIEWS), 2015/01/21 00:59:59]

I guess printing_common should have printing in deps

[dgn, 2015/01/22 00:14:29]

But there are references to //printing directly, not only through
//components/printing. Isn't it better to make it explicit that we are using it
instead of relying on components to bring it in?

+      "//printing" ,
     ]
-  } else {
-    deps += [ "//printing" ]
     if (enable_print_preview) {
       # Full printing support.
       sources += rebase_path(gypi_values.chrome_common_service_process_sources,
                              ".",
                              "//chrome")
     }
   }
 
   if (enable_service_discovery) {
     sources += [
@@ skipping 139 matching lines @@
     ]
   }
 
   if (enable_extensions) {
     sources += [
       "extensions/extension_test_util.cc",
       "extensions/extension_test_util.h",
     ]
   }
 }