Generate the print messages in components/printing

chrome/common/BUILD.gn

Index: chrome/common/BUILD.gn
diff --git a/chrome/common/BUILD.gn b/chrome/common/BUILD.gn
index 6fd61117e9a4a049e95fa4ef9036655bf05d7513..95c37706044cd764976239982ef25ee8f276837e 100644
--- a/chrome/common/BUILD.gn
+++ b/chrome/common/BUILD.gn
@@ -59,7 +59,6 @@ static_library("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",
@@ -136,13 +135,11 @@ static_library("common") {
   }
 
   # 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,