PrintPreview: Modified PrintingContextMac code to use the copy of sharedPrintInfo object.

printing/printing_context_mac.h

Index: printing/printing_context_mac.h
diff --git a/printing/printing_context_mac.h b/printing/printing_context_mac.h
index 84a5c05409832105d53cbab4cf07ef5417f20459..9d716c058233023167ed3368413f6e71e0b7d7fa 100644
--- a/printing/printing_context_mac.h
+++ b/printing/printing_context_mac.h
@@ -42,12 +42,17 @@ class PrintingContextMac : public PrintingContext {
   virtual gfx::NativeDrawingContext context() const;
 
  private:
-  // Read the settings from the given NSPrintInfo (and cache it for later use).
-  void ParsePrintInfo(NSPrintInfo* print_info);
+  // Reset the cached |print_info_| with |print_info|.
+  void ResetPrintInfo(NSPrintInfo* print_info);
 
-  // Initializes PrintSettings from native print info object.
+  // Initializes PrintSettings from cached print info object.

[stuartmorgan, 2011/06/02 21:07:20]

s/cached print info object/|print_info_|/

I think it would be worthwhile to add a sentence here like "This must be called
after changes to |print_info_| in order for the changes to take effect in
printing."

[kmadhusu, 2011/06/02 23:07:00]

Done.

+  // This function ignores the page range information specified in the print
+  // info object and use |ranges| instead.
   void InitPrintSettingsFromPrintInfo(const PageRanges& ranges);
 
+  // Returns the set of page ranges constructed from cached print info object.

[stuartmorgan, 2011/06/02 21:07:20]

s/cached print info object/|print_info_|/

[kmadhusu, 2011/06/02 23:07:00]

Done.

+  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);