Changeset 53997 in webkit

Timestamp:

Jan 28, 2010 5:06:49 AM (10 years ago)

Author:

Simon Hausmann

Message:

[Qt] Fix for endless print loop when printing web pages

Patch by Trond Kjernåsen <​trond@trolltech.com> on 2010-01-28
Reviewed by Simon Hausmann.

• Api/qwebframe.cpp:

(QWebFrame::print):

Location:

trunk/WebKit/qt

Files:

• Api/qwebframe.cpp (modified) (1 diff)
• ChangeLog (modified) (1 diff)

trunk/WebKit/qt/Api/qwebframe.cpp

@@ -1394 +1394 @@
     fromPage = qMax(1, fromPage);
     toPage = qMin(printContext.pageCount(), toPage);
+    if (toPage < fromPage) {
+        // if the user entered a page range outside the actual number
+        // of printable pages, just return
+        return;
+    }
 
     if (printer->pageOrder() == QPrinter::LastPageFirst) {

trunk/WebKit/qt/ChangeLog

@@ +1 @@
+2010-01-28  Trond Kjernåsen <trond@trolltech.com>
+
+        Reviewed by Simon Hausmann.
+
+        [Qt] Fix for endless print loop when printing web pages
+
+        * Api/qwebframe.cpp:
+        (QWebFrame::print):
+
 2010-01-27  Diego Gonzalez  <diego.gonzalez@openbossa.org>