[webkit-reviews] review requested: [Bug 73994] [Qt][WK2] Fix tst_QQuickWebView::scrollRequest() API test : [Attachment 120176] Patch to fix the problem.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 21 07:00:29 PST 2011


Michael Brüning <michael.bruning at nokia.com> has asked  for review:
Bug 73994: [Qt][WK2] Fix tst_QQuickWebView::scrollRequest() API test
https://bugs.webkit.org/show_bug.cgi?id=73994

Attachment 120176: Patch to fix the problem.
https://bugs.webkit.org/attachment.cgi?id=120176&action=review

------- Additional Comments from Michael Brüning <michael.bruning at nokia.com>
Found that the problem was actually in a wrong/missing viewport definition,
which cause the page to be scaled and not scrollable in xvfb.
Added a viewport meta tag and found out that parts of the previous patch to the
test weren't necessary, hence reverted those (see
Source/WebKit2/UIProcess/API/qt/test/qquickwebview/tst_qquickwebview.cpp for
details).
Removed unnecessary "This is a test page" text and corrected width and height
in the div style definition. 

Tested on host and in xvfb.


More information about the webkit-reviews mailing list