Enable print as image by default

chrome/common/chrome_features.cc

Index: chrome/common/chrome_features.cc
diff --git a/chrome/common/chrome_features.cc b/chrome/common/chrome_features.cc
index 5e8d2bd096ab997822ecbfa9b662def50992f9e4..a337307972a71f5ccbeacd30ec34a6abdba7fb54 100644
--- a/chrome/common/chrome_features.cc
+++ b/chrome/common/chrome_features.cc
@@ -284,10 +284,10 @@ const base::Feature kPreloadLockScreen{"PreloadLockScreen",
                                        base::FEATURE_ENABLED_BY_DEFAULT};
 #endif
 
-#if BUILDFLAG(ENABLE_PRINT_PREVIEW)
+#if BUILDFLAG(ENABLE_PRINT_PREVIEW) && !defined(OS_WIN) && !defined(OS_MACOSX)

[dpapad, 2017/05/17 17:32:37]

Probably need something as follows to fix  compilation errors on Mac and Win.

#if BUILDFLAG(ENABLE_PRINT_PREVIEW)

# if defined(OS_WIN) || defined(OS_MACOSX)
const base::Feature kPrintPdfAsImage{"PrintPdfAsImage", 
FEATURE_DISABLED_BY_DEFAULT};
#else
const base::Feature kPrintPdfAsImage{"PrintPdfAsImage", 
FEATURE_ENABLED_BY_DEFAULT};
#endif

#endif

[rbpotter, 2017/05/17 19:17:26]

I think it would be better if this feature just isn't defined
for Mac and Win at all, since the intent is for it not to exist
on those platforms. Have updated the other files where this was
referenced to reflect that.

 // Enables the Print as Image feature in print preview.
 const base::Feature kPrintPdfAsImage{"PrintPdfAsImage",
-                                     base::FEATURE_DISABLED_BY_DEFAULT};
+                                     base::FEATURE_ENABLED_BY_DEFAULT};
 #endif
 
 // Enables or disables push subscriptions keeping Chrome running in the