PrintPreview: [LINUX] Update the margin values after flipping the paper orientation.

printing/page_setup.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_PAGE_SETUP_H_
 #define PRINTING_PAGE_SETUP_H_
 
 #include "printing/printing_export.h"
 #include "ui/gfx/rect.h"
 
 namespace printing {
 
 // Margins for a page setup.
 class PRINTING_EXPORT PageMargins {
  public:
   PageMargins();
 
+  // Set |rotate_margins| to true to rotate the margins anti-clockwise or
+  // set |rotate_margins| to false to use the specified margins.
+  PageMargins(const PageMargins& page_margins, bool rotate_margins);

[vandebo (ex-Chrome), 2011/11/02 20:26:03]

How about a rotate method instead of a modified copy constructor?

[kmadhusu, 2011/11/02 23:15:03]

Done.

+
   void Clear();
 
   // Equality operator.
   bool Equals(const PageMargins& rhs) const;
 
   // Vertical space for the overlay from the top of the sheet.
   int header;
   // Vertical space for the overlay from the bottom of the sheet.
   int footer;
   // Margin on each side of the sheet.
@@ skipping 52 matching lines @@
 
   // The printable area as selected by the user's margins.
   gfx::Rect content_area_;
 
   // Effective margins.
   PageMargins effective_margins_;
 
   // Requested margins.
   PageMargins requested_margins_;
 
+  // Store whether the |requested_margins_| were set forcefully.

[vandebo (ex-Chrome), 2011/11/02 20:26:03]

nit: were set forcefully -> respect printable_area_

[kmadhusu, 2011/11/02 23:15:03]

Done.

+  bool has_forced_margins_value;

[vandebo (ex-Chrome), 2011/11/02 20:26:03]

nit: forced_margins_;

[kmadhusu, 2011/11/02 23:15:03]

Done.

+
   // Space that must be kept free for the overlays.
   int text_height_;
 };
 
 }  // namespace printing
 
 #endif  // PRINTING_PAGE_SETUP_H_