Internals: throw an exception when page height or width is 0.

third_party/WebKit/Source/core/testing/Internals.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 1718 matching lines @@
 }
 
 String Internals::counterValue(Element* element)
 {
     if (!element)
         return String();
 
     return counterValueForElement(element);
 }
 
-int Internals::pageNumber(Element* element, float pageWidth, float pageHeight)
+int Internals::pageNumber(Element* element, float pageWidth, float pageHeight, ExceptionState& exceptionState)
 {
     if (!element)
         return 0;
 
+    if (pageWidth <= 0 || pageHeight <= 0) {
+        exceptionState.throwDOMException(V8TypeError, "Page width and height must be larger than 0.");
+        return 0;
+    }
+
     return PrintContext::pageNumberForElement(element, FloatSize(pageWidth, pageHeight));
 }
 
 Vector<String> Internals::iconURLs(Document* document, int iconTypesMask) const
 {
     Vector<IconURL> iconURLs = document->iconURLs(iconTypesMask);
     Vector<String> array;
 
     for (auto& iconURL : iconURLs)
         array.append(iconURL.m_iconURL.string());
 
     return array;
 }
 
 Vector<String> Internals::shortcutIconURLs(Document* document) const
 {
     return iconURLs(document, Favicon);
 }
 
 Vector<String> Internals::allIconURLs(Document* document) const
 {
     return iconURLs(document, Favicon | TouchIcon | TouchPrecomposedIcon);
 }
 
-int Internals::numberOfPages(float pageWidth, float pageHeight)
+int Internals::numberOfPages(float pageWidth, float pageHeight, ExceptionState& exceptionState)
 {
     if (!frame())
         return -1;
 
+    if (pageWidth <= 0 || pageHeight <= 0) {
+        exceptionState.throwDOMException(V8TypeError, "Page width and height must be larger than 0.");
+        return -1;
+    }
+
     return PrintContext::numberOfPages(frame(), FloatSize(pageWidth, pageHeight));
 }
 
 String Internals::pageProperty(String propertyName, int pageNumber, ExceptionState& exceptionState) const
 {
     if (!frame()) {
         exceptionState.throwDOMException(InvalidAccessError, "No frame is available.");
         return String();
     }
 
@@ skipping 764 matching lines @@
     mediaElement->setNetworkState(static_cast<WebMediaPlayer::NetworkState>(state));
 }
 
 // TODO(liberato): remove once autoplay gesture override experiment concludes.
 void Internals::triggerAutoplayViewportCheck(HTMLMediaElement* element)
 {
     element->triggerAutoplayViewportCheckForTesting();
 }
 
 } // namespace blink