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-

Description Andreas Kling 2010-04-08 07:37:19 PDT 
[Qt] (X11 only) Ignore timer events while waiting on a synchronous XHR
Comment 1 Andreas Kling 2010-04-08 07:38:20 PDT 
Created attachment 52862 [details]
Patch
Comment 2 Kenneth Rohde Christiansen 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 ...
Comment 3 Andreas Kling 2010-04-08 07:48:18 PDT 
Created attachment 52865 [details]
Patch with fleshed-out ChangeLog
Comment 4 Eric Seidel (no email) 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.
Comment 5 Simon Hausmann 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 :)
Comment 6 Benjamin Poulain 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 ***