De-duplicate job_settings parsing code.

printing/printing_context_no_system_dialog.cc

 // Copyright (c) 2011 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.
 
 #include "printing/printing_context_no_system_dialog.h"
 
 #include <unicode/ulocdata.h>
 
 #include "base/logging.h"
 #include "base/values.h"
@@ skipping 55 matching lines @@
   }
 
   physical_size_device_units.SetSize(width, height);
 
   // Assume full page is printable for now.
   printable_area_device_units.SetRect(0, 0, width, height);
 
   settings_.set_dpi(dpi);
   settings_.SetPrinterPrintableArea(physical_size_device_units,
                                     printable_area_device_units,
-                                    dpi);
+                                    dpi, true);
 
   return OK;
 }
 
 PrintingContext::Result PrintingContextNoSystemDialog::UpdatePrinterSettings(
-    const base::DictionaryValue& job_settings, const PageRanges& ranges) {
-  bool landscape = false;
-
-  if (!job_settings.GetBoolean(kSettingLandscape, &landscape))
-    return OnError();
-
-  if (settings_.dpi() == 0)
-    UseDefaultSettings();
-
-  settings_.SetOrientation(landscape);
-  settings_.ranges = ranges;
-
+    bool target_is_pdf,
+    bool external_preview) {
   return OK;
 }
 
 PrintingContext::Result PrintingContextNoSystemDialog::InitWithSettings(
     const PrintSettings& settings) {
   DCHECK(!in_print_job_);
 
   settings_ = settings;
 
   return OK;
@@ skipping 45 matching lines @@
   // Intentional No-op.
 }
 
 gfx::NativeDrawingContext PrintingContextNoSystemDialog::context() const {
   // Intentional No-op.
   return NULL;
 }
 
 }  // namespace printing