Bug 37269 - [Qt] (X11 only) Ignore timer events while waiting on a synchronous XHR
Summary: [Qt] (X11 only) Ignore timer events while waiting on a synchronous XHR
Status: RESOLVED DUPLICATE of bug 37191
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: QtWebKit Unassigned
URL:
Keywords: Qt
Depends on:
Blocks: 35784
  Show dependency tree / graph
 
Reported: 2010-04-08 07:37 PDT by Andreas Kling
Modified: 2010-04-30 04:21 PDT (History)
3 users (show)

See Also:

Attachments
Patch (1.36 KB, patch)
2010-04-08 07:38 PDT, Andreas Kling 		no flags Details | Formatted Diff | Diff
Patch with fleshed-out ChangeLog (1.49 KB, patch)
2010-04-08 07:48 PDT, Andreas Kling 		hausmann: review-
hausmann: commit-queue-
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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 ***