Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(991)

Unified Diff: printing/print_settings_conversion.cc

Issue 2524143003: Print Preview: Add option to rasterize PDFs and add JPEG compression. (Closed)
Patch Set: Fix initialization Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: printing/print_settings_conversion.cc
diff --git a/printing/print_settings_conversion.cc b/printing/print_settings_conversion.cc
index 02fe34ed22a16eeca6025790b45ea4569b151609..096cb900e03f0fcb3da5e9e02e757a0d7054b20e 100644
--- a/printing/print_settings_conversion.cc
+++ b/printing/print_settings_conversion.cc
@@ -177,6 +177,7 @@ bool PrintSettingsFromJobSettings(const base::DictionaryValue& job_settings,
bool collate = false;
int copies = 1;
int scale_factor = 100;
+ bool rasterize_pdf = false;
if (!job_settings.GetBoolean(kSettingCollate, &collate) ||
!job_settings.GetInteger(kSettingCopies, &copies) ||
@@ -184,7 +185,8 @@ bool PrintSettingsFromJobSettings(const base::DictionaryValue& job_settings,
!job_settings.GetInteger(kSettingDuplexMode, &duplex_mode) ||
!job_settings.GetBoolean(kSettingLandscape, &landscape) ||
!job_settings.GetString(kSettingDeviceName, &device_name) ||
- !job_settings.GetInteger(kSettingScaleFactor, &scale_factor)) {
+ !job_settings.GetInteger(kSettingScaleFactor, &scale_factor) ||
+ !job_settings.GetBoolean(kSettingRasterizePdf, &rasterize_pdf)) {
return false;
}
@@ -195,7 +197,7 @@ bool PrintSettingsFromJobSettings(const base::DictionaryValue& job_settings,
settings->set_duplex_mode(static_cast<DuplexMode>(duplex_mode));
settings->set_color(static_cast<ColorModel>(color));
settings->set_scale_factor(static_cast<double>(scale_factor) / 100.0);
-
+ settings->set_rasterize_pdf(rasterize_pdf);
#if defined(OS_WIN)
// Modifiable implies HTML and not other formats like PDF.
bool can_modify = false;

Powered by Google App Engine
This is Rietveld 408576698