See patch.
Created attachment 55003 [details] The patch
Patrick, the change looks good to me. Have you had a chance to test it on WinCE?
(In reply to comment #2) > Patrick, the change looks good to me. Have you had a chance to test it on > WinCE? No, I didn't. (It takes sooo much time) I can compile it with this patch if it's necessary, but I don't think so. ;-)
As we do not have a buildbot for WinCE I would feel better if we could actually test this build change. Only the QtWebKit port on WinCE is impacted. Should we find someone else to test this with QtWebKit ?
(In reply to comment #4) > As we do not have a buildbot for WinCE I would feel better if we could actually > test this build change. Only the QtWebKit port on WinCE is impacted. > > Should we find someone else to test this with QtWebKit ? I'll try to get it running, but I had many troubles compiling QtWebKit for WinCE directly from svn trunk the last time. It doesn't work as nice as from a Qt release. ;-)
Comment on attachment 55003 [details] The patch r+ for the patch, but cq- until it is confirmed that this does not breaks the build. Thanks for the clean-up, this moves the code to the right direction.
Triaged but I'm not adding it as QtWebKit 2.0 blocker as I don't know if you want this.
Comment on attachment 55003 [details] The patch It took a lot of time to compile it out of trunk for WinCE, but it works with this patch.
Comment on attachment 55003 [details] The patch Clearing flags on attachment: 55003 Committed r59600: <http://trac.webkit.org/changeset/59600>
All reviewed patches have been landed. Closing bug.
http://trac.webkit.org/changeset/59600 might have broken Qt Linux ARMv7 Release The following changes are on the blame list: http://trac.webkit.org/changeset/59600 http://trac.webkit.org/changeset/59599
Sorry. The qt builders just got disconnected. I think we may need to remove them from the sheriff-bot watch until they can get a better internet connection.