PrintContext::pageProperty() shouldn't use 0 as page height.

third_party/WebKit/Source/core/page/PrintContext.cpp

 /*
  * Copyright (C) 2007 Alp Toker <alp@atoker.com>
  * Copyright (C) 2007 Apple Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 241 matching lines @@
             continue;
         point.clampNegativeToZero();
         context.setURLDestinationLocation(entry.key, point);
     }
 }
 
 String PrintContext::pageProperty(LocalFrame* frame, const char* propertyName, int pageNumber)
 {
     Document* document = frame->document();
     PrintContext printContext(frame);
-    printContext.begin(800); // Any width is OK here.
-    document->updateLayout();
+    // Any non-zero size is OK here. We don't care about actual layout. We just want to collect
+    // @page rules and figure out what declarations apply on a given page (that may or may not exist).
+    printContext.begin(800, 1000);
     RefPtr<ComputedStyle> style = document->styleForPage(pageNumber);
 
     // Implement formatters for properties we care about.
     if (!strcmp(propertyName, "margin-left")) {
         if (style->marginLeft().isAuto())
             return String("auto");
         return String::number(style->marginLeft().value());
     }
     if (!strcmp(propertyName, "line-height"))
         return String::number(style->lineHeight().value());
@@ skipping 37 matching lines @@
 
 DEFINE_TRACE(PrintContext)
 {
 #if ENABLE(OILPAN)
     visitor->trace(m_frame);
     visitor->trace(m_linkedDestinations);
 #endif
 }
 
 }