Bug 37269

Summary: [Qt] (X11 only) Ignore timer events while waiting on a synchronous XHR
Product: WebKit Reporter: Andreas Kling <kling>
Component: New BugsAssignee: QtWebKit Unassigned <webkit-qt-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: benjamin, hausmann, markus
Priority: P2 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 35784    
Attachments:
Description Flags
Patch
none
Patch with fleshed-out ChangeLog hausmann: review-, hausmann: commit-queue-

Andreas Kling
Reported 2010-04-08 07:37:19 PDT
[Qt] (X11 only) Ignore timer events while waiting on a synchronous XHR
Attachments
Patch (1.36 KB, patch)
2010-04-08 07:38 PDT, Andreas Kling 		no flags
DetailsFormatted DiffDiff
Patch with fleshed-out ChangeLog (1.49 KB, patch)
2010-04-08 07:48 PDT, Andreas Kling 		hausmann: review-
hausmann: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2010-04-08 07:38:20 PDT
Created attachment 52862 [details] Patch
Kenneth Rohde Christiansen
Comment 2 2010-04-08 07:41:07 PDT
Comment on attachment 52862 [details] Patch > + The Qt implementation of sync XHR is broken, but needs new API to be done properly. > + In the meantime, ignoring timer events on X11 is a quick way to avoid some badness. > + New API where? in Qt? What kind of badness. Consider: The Qt implementation of sync XHR is broken, and needs new API in Qt to be done properly. For now, ignore timer events on X11 in order to ...
Andreas Kling
Comment 3 2010-04-08 07:48:18 PDT
Created attachment 52865 [details] Patch with fleshed-out ChangeLog
Eric Seidel (no email)
Comment 4 2010-04-18 16:26:55 PDT
Comment on attachment 52862 [details] Patch Cleared Kenneth Rohde Christiansen's review+ from obsolete attachment 52862 [details] so that this bug does not appear in http://webkit.org/pending-commit.
Simon Hausmann
Comment 5 2010-04-19 17:41:34 PDT
Comment on attachment 52865 [details] Patch with fleshed-out ChangeLog r- because on earlier discussion that gives a false security on X11 where issues remain on other platforms. In addition I heard rumors that you were working on a real syncxhr fix anyway :)
Benjamin Poulain
Comment 6 2010-04-30 04:21:09 PDT
Closing the task as duplicate of 37191. We need to make sure there is a proper implementation of QNetworkAccessManager for 4.8. *** This bug has been marked as a duplicate of bug 37191 ***
Note You need to log in before you can comment on or make changes to this bug.