[webkit-reviews] review denied: [Bug 37163] [Qt] inputMethodQuery returns coordinates in web page coordinates rather than in item coordinates. : [Attachment 54078] Fix applied to QWebPage. Tests added for QGraphicsWebView and QWebView

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Apr 22 14:37:56 PDT 2010


Laszlo Gombos <laszlo.1.gombos at nokia.com> has denied John Pavan
<john.pavan at nokia.com>'s request for review:
Bug 37163: [Qt] inputMethodQuery returns coordinates in web page coordinates
rather than in item coordinates.
https://bugs.webkit.org/show_bug.cgi?id=37163

Attachment 54078: Fix applied to QWebPage.  Tests added for QGraphicsWebView
and QWebView
https://bugs.webkit.org/attachment.cgi?id=54078&action=review

------- Additional Comments from Laszlo Gombos <laszlo.1.gombos at nokia.com>
WebKit/qt/tests/qwebview/tst_qwebview.cpp:227
 +  
This qWait does not seems to be needed, and will slow down running the
automated tests.

WebKit/qt/ChangeLog:7
 +	    
We need a reference here (link) to the bugzilla bugreport - this is
automatically generated if you use the prepare-ChangeLog script.



Overall the patch looks great, I would give an r+ if you would be a committer.
Let's fix this 2 small issues in a new patch.


More information about the webkit-reviews mailing list