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

printing/printing_context_win.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_win.h"
 
 #include <winspool.h>
 
 #include "base/i18n/file_util_icu.h"
 #include "base/i18n/time_formatting.h"
 #include "base/message_loop.h"
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
-#include "printing/print_job_constants.h"
 #include "printing/print_settings_initializer_win.h"
 #include "printing/printed_document.h"
 #include "skia/ext/platform_device_win.h"
 
 using base::Time;
 
 namespace printing {
 
 class PrintingContextWin::CallbackHandler : public IPrintDialogCallback,
                                             public IObjectWithSite {
@@ skipping 182 matching lines @@
     return FAILED;
   }
   return ParseDialogResult(dialog_options);
 }
 
 PrintingContext::Result PrintingContextWin::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);
 
   // TODO(kmadhusu): Update other print settings such as number of copies,
   // collate, duplex printing, job title, etc.,
 
   settings_.ranges = ranges;
 
   return OK;
 }
 
@@ skipping 388 matching lines @@
   if (buf_size) {
     buffer->reset(new uint8[buf_size]);
     memset(buffer->get(), 0, buf_size);
     if (!GetPrinter(printer, level, buffer->get(), buf_size, &buf_size)) {
       buffer->reset();
     }
   }
 }
 
 }  // namespace printing