PrintPreview: Added a helper function to get print job settings from dictionary.

printing/printing_context_cairo.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_cairo.h"
 
 #include "base/logging.h"
 #include "base/values.h"
-#include "printing/print_job_constants.h"
 #include "printing/print_settings_initializer_gtk.h"
 #include "printing/units.h"
 
 #if defined(OS_CHROMEOS)
 #include <unicode/ulocdata.h>
 #else
 #include <gtk/gtk.h>
 #include <gtk/gtkprintunixdialog.h>
 #endif  // defined(OS_CHROMEOS)
 
@@ skipping 127 matching lines @@
 #endif  // defined(OS_CHROMEOS)
 
   return OK;
 }
 
 PrintingContext::Result PrintingContextCairo::UpdatePrintSettings(
     const DictionaryValue& job_settings, const PageRanges& ranges) {
   DCHECK(!in_print_job_);
 
   bool landscape;
-  if (!job_settings.GetBoolean(kSettingLandscape, &landscape))
+  if (!GetSettingsFromDict(job_settings, &landscape, NULL))
     return OnError();
+
   settings_.SetOrientation(landscape);
-
   settings_.ranges = ranges;
 
   // TODO(kmadhusu): Update other print settings such as number of copies,
   // collate, duplex printing, etc.,
 
   return OK;
 }
 
 PrintingContext::Result PrintingContextCairo::InitWithSettings(
     const PrintSettings& settings) {
@@ skipping 53 matching lines @@
 void PrintingContextCairo::ReleaseContext() {
   // Intentional No-op.
 }
 
 gfx::NativeDrawingContext PrintingContextCairo::context() const {
   // Intentional No-op.
   return NULL;
 }
 
 }  // namespace printing