Renamed DISABLE_BASIC_PRINTING and ENABLE_FULL_PRINTING.

build/common.gypi

Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 91156ec1e85f9cdcef702490946ac6bbf3b4b1b6..aadd060625ead1e57f4a2b51bb94cd1ef73068cb 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -2100,7 +2100,7 @@
         'grit_defines': ['-D', 'enable_printing'],
       }],
       ['enable_printing==1', {
-        'grit_defines': ['-D', 'enable_full_printing'],
+        'grit_defines': ['-D', 'enable_print_preview'],
       }],
       ['enable_themes==1', {
         'grit_defines': ['-D', 'enable_themes'],
@@ -2923,11 +2923,11 @@
         # chrome://translate-internals
         'defines': ['CLD2_DATA_SOURCE=<(cld2_data_source)'],
       }],
-      ['enable_printing==1', {
-        'defines': ['ENABLE_FULL_PRINTING=1', 'ENABLE_PRINTING=1'],
+      ['enable_printing!=0', {
+        'defines': ['ENABLE_PRINTING=1', 'ENABLE_BASIC_PRINTING=1'],

[brettw, 2014/11/05 18:02:11]

So does ENABLE_PRINTING have the same meaning as ENABLE_BASIC_PRINTING? If so,
can we search-and-replace to ENABLE_BASIC_PRINTING in a separate patch after
this?

[Vitaly Buka (NO REVIEWS), 2014/11/05 19:14:50]

it's would be ENABLE_PRINTING = (ENABLE_PRINT_PREVIEW || ENABLE_BASIC_PRINTING),
and we can replace this in new patch

Next patch will separate ENABLE_PRINTING and ENABLE_BASIC_PRINTING
I just can't do here because 'enable_printing' has no state for this. 
Check here:
https://codereview.chromium.org/702023002/diff/100001/build/common.gypi#newco...



On 2014/11/05 18:02:11, brettw wrote:

       }],
-      ['enable_printing==2', {
-        'defines': ['ENABLE_PRINTING=1'],
+      ['enable_printing==1', {
+        'defines': ['ENABLE_PRINT_PREVIEW=1'],
       }],
       ['enable_spellcheck==1', {
         'defines': ['ENABLE_SPELLCHECK=1'],