Mac printing: Autorotate printed pages so their orientation is the same as the destination page.

printing/metafile.h

Index: printing/metafile.h
===================================================================
--- printing/metafile.h	(revision 142466)
+++ printing/metafile.h	(working copy)
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 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.
 
@@ -39,6 +39,29 @@
 // (usually PDF or EMF).
 class PRINTING_EXPORT Metafile {
  public:
+#if defined(OS_MACOSX)
+  // |shrink_to_fit| specifies whether the output should be shrunk to fit a
+  // destination page if the source PDF is bigger than the destination page in
+  // any dimension. If this is false, parts of the source PDF page that lie
+  // outside the bounds will be clipped.
+  // |stretch_to_fit| specifies whether the output should be stretched to fit
+  // the destination page if the source page size is smaller in all dimensions.
+  // |center_horizontally| specifies whether the output (after any scaling is
+  // done) should be centered horizontally within the destination page.
+  // |center_vertically| specifies whether the output (after any scaling is
+  // done) should be centered vertically within the destination page.
+  // Note that all scaling preserves the original aspect ratio of the page.
+  // |autorotate| specifies whether the source PDF should be autorotated to fit
+  // on the destination page.
+  struct MacRenderPageParams {
+    bool shrink_to_fit;

[kmadhusu, 2012/06/17 20:18:28]

Can you write a default constructor which initializes all these member variables
to false?  Instances can overwrite the ones that need to be set to true. 

[Lei Zhang, 2012/06/18 20:58:25]

Done.

+    bool stretch_to_fit;
+    bool center_horizontally;
+    bool center_vertically;
+    bool autorotate;
+  };
+#endif  // defined(OS_MACOSX)
+
   virtual ~Metafile() {}
 
   // Initializes a fresh new metafile for rendering. Returns false on failure.
@@ -116,27 +139,12 @@
   virtual HENHMETAFILE emf() const = 0;
 #elif defined(OS_MACOSX)
   // Renders the given page into |rect| in the given context.
-  // Pages use a 1-based index. The rendering uses the following arguments
-  // to determine scaling and translation factors.
-  // |shrink_to_fit| specifies whether the output should be shrunk to fit the
-  // supplied |rect| if the page size is larger than |rect| in any dimension.
-  // If this is false, parts of the PDF page that lie outside the bounds will be
-  // clipped.
-  // |stretch_to_fit| specifies whether the output should be stretched to fit
-  // the supplied bounds if the page size is smaller than |rect| in all
-  // dimensions.
-  // |center_horizontally| specifies whether the final image (after any scaling
-  // is done) should be centered horizontally within the given |rect|.
-  // |center_vertically| specifies whether the final image (after any scaling
-  // is done) should be centered vertically within the given |rect|.
-  // Note that all scaling preserves the original aspect ratio of the page.
+  // Pages use a 1-based index. The rendering uses the arguments in
+  // |params| to determine scaling, translation, and rotation.
   virtual bool RenderPage(unsigned int page_number,
                           gfx::NativeDrawingContext context,
                           const CGRect rect,
-                          bool shrink_to_fit,
-                          bool stretch_to_fit,
-                          bool center_horizontally,
-                          bool center_vertically) const = 0;
+                          const MacRenderPageParams& params) const = 0;
 #elif defined(OS_CHROMEOS)
   // Saves the underlying data to the file associated with fd. This function
   // should ONLY be called after the metafile is closed.