[webkit-reviews] review requested: [Bug 35702] [Qt] Add more support for InputTextController : [Attachment 50670] Updated Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Mar 14 04:19:06 PDT 2010


Robert Hogan <robert at webkit.org> has asked  for review:
Bug 35702: [Qt] Add more support for InputTextController
https://bugs.webkit.org/show_bug.cgi?id=35702

Attachment 50670: Updated Patch
https://bugs.webkit.org/attachment.cgi?id=50670&action=review

------- Additional Comments from Robert Hogan <robert at webkit.org>
The previous patch didn't pass tst_qwebpage - now it does. I've removed the
maxlength check for now - will do it as a separate patch (promise!).

Ossy - I've added a couple of Qt-specific results. The cursor position is
certainly correct from the point of view of the test, though it might be that
the actual number reported is specific to my system so would appreciate if you,
or anyone else, could check whether you match the existing test results or mine
when running with this patch.


More information about the webkit-reviews mailing list