WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 98404
126574
Make abort event sending compliant with XHR spec
https://bugs.webkit.org/show_bug.cgi?id=126574
Summary
Make abort event sending compliant with XHR spec
youenn fablet
Reported
2014-01-07 05:03:25 PST
WebKit always send abort event when XHR abort() method is called.
http://www.w3.org/TR/XMLHttpRequest/#the-abort()-method
mandates to do that only if the XHR state is unset or opened. Related WTP test is XMLHttpRequest/abort-after-receive.htm
Attachments
potential fix over 120828 patch
(7.15 KB, patch)
2014-01-07 07:53 PST
,
youenn fablet
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
youenn fablet
Comment 1
2014-01-07 05:39:49 PST
XMLHttpRequestUpload events should be sent before XMLHttpRequest events. This reordering may be handled as part of code refactoring done in
bug 120828
youenn fablet
Comment 2
2014-01-07 07:53:39 PST
Created
attachment 220528
[details]
potential fix over 120828 patch
youenn fablet
Comment 3
2014-01-13 01:19:04 PST
WebKit XHR implementation currently sends abort event when its state is DONE and abort() is called. According to spec, the expected behavior should be to not send any abort event, but to ensure that load and loadend events are sent (since XHR is actually DONE). This should happen in particular even when xhr.abort is called in onreadystatechange callback in DONE case. Firefox is behaving this way (no abort event but load and loadend events). Blink currently does not send any load/loadend/abort event at all
youenn fablet
Comment 4
2014-01-13 01:22:37 PST
Potential duplicate of
bug 98404
Csaba Osztrogonác
Comment 5
2014-01-13 05:45:05 PST
Please set the r? flag if you would like to ask review.
youenn fablet
Comment 6
2016-01-15 09:28:08 PST
*** This bug has been marked as a duplicate of
bug 98404
***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug