De-duplicate job_settings parsing code.

printing/printing_context_mac.h

 // 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.
 
 #ifndef PRINTING_PRINTING_CONTEXT_MAC_H_
 #define PRINTING_PRINTING_CONTEXT_MAC_H_
 
 #include <string>
 
 #include "base/mac/scoped_nsobject.h"
@@ skipping 13 matching lines @@
   explicit PrintingContextMac(const std::string& app_locale);
   virtual ~PrintingContextMac();
 
   // PrintingContext implementation.
   virtual void AskUserForSettings(
       gfx::NativeView parent_view,
       int max_pages,
       bool has_selection,
       const PrintSettingsCallback& callback) OVERRIDE;
   virtual Result UseDefaultSettings() OVERRIDE;
-  virtual Result UpdatePrinterSettings(
-      const base::DictionaryValue& job_settings,
-      const PageRanges& ranges) OVERRIDE;
+  virtual Result UpdatePrinterSettings(bool target_is_pdf,
+                                       bool external_preview) OVERRIDE;
   virtual Result InitWithSettings(const PrintSettings& settings) OVERRIDE;
   virtual Result NewDocument(const base::string16& document_name) OVERRIDE;
   virtual Result NewPage() OVERRIDE;
   virtual Result PageDone() OVERRIDE;
   virtual Result DocumentDone() OVERRIDE;
   virtual void Cancel() OVERRIDE;
   virtual void ReleaseContext() OVERRIDE;
   virtual gfx::NativeDrawingContext context() const OVERRIDE;
 
  private:
   // Initializes PrintSettings from |print_info_|. This must be called
   // after changes to |print_info_| in order for the changes to take effect in
   // printing.
   // This function ignores the page range information specified in the print
-  // info object and use |ranges| instead.
-  void InitPrintSettingsFromPrintInfo(const PageRanges& ranges);
+  // info object and use |settings_.ranges| instead.
+  void InitPrintSettingsFromPrintInfo();
 
   // Returns the set of page ranges constructed from |print_info_|.
   PageRanges GetPageRangesFromPrintInfo();
 
   // Updates |print_info_| to use the given printer.
   // Returns true if the printer was set else returns false.
   bool SetPrinter(const std::string& device_name);
 
   // Updates |print_info_| page format with user default paper information.
   // Returns true if the paper was set else returns false.
@@ skipping 29 matching lines @@
   // The current page's context; only valid between NewPage and PageDone call
   // pairs.
   CGContext* context_;
 
   DISALLOW_COPY_AND_ASSIGN(PrintingContextMac);
 };
 
 }  // namespace printing
 
 #endif  // PRINTING_PRINTING_CONTEXT_MAC_H_