Add check to debug issue 96063

printing/page_setup.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/page_setup.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 
@@ skipping 101 matching lines @@
   content_area_.set_x(effective_margins_.left);
   content_area_.set_y(effective_margins_.top);
   content_area_.set_width(std::max(0,
                                    physical_size.width() -
                                        effective_margins_.right -
                                        content_area_.x()));
   content_area_.set_height(std::max(0,
                                     physical_size.height() -
                                         effective_margins_.bottom -
                                         content_area_.y()));
+  // TODO(vandebo) Remove once bug 96063 is resolved.
+  CHECK(content_area_.width() > 0 && content_area_.height() > 0);

[Lei Zhang, 2011/09/27 23:07:56]

How about doing this as two separate checks so you can see which one failed?

[vandebo (ex-Chrome), 2011/09/27 23:09:44]

I can see which one failed by looking at the minidump.  But, the problem turned
up by the previously added check is that they are both zero.

The check will let me examine the callers to see how they ended up being zero.

 }
 
 void PageSetup::SetRequestedMargins(const PageMargins& requested_margins) {
   requested_margins_ = requested_margins;
   if (physical_size_.width() && physical_size_.height())
     Init(physical_size_, printable_area_, text_height_);
 }
 
 void PageSetup::FlipOrientation() {
   if (physical_size_.width() && physical_size_.height()) {
     gfx::Size new_size(physical_size_.height(), physical_size_.width());
     int new_y = physical_size_.width() -
                 (printable_area_.width() + printable_area_.x());
     gfx::Rect new_printable_area(printable_area_.y(),
                                  new_y,
                                  printable_area_.height(),
                                  printable_area_.width());
     Init(new_size, new_printable_area, text_height_);
   }
 }
 
 }  // namespace printing