Bug 121585

Summary: Add XHR tests checking readyState transition when abort() is invoked in various states
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: New BugsAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, ap, beidson, commit-queue, darin
Priority: P2 Keywords: BlinkMergeCandidate
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Adds the test none

Ryosuke Niwa
Reported 2013-09-18 20:08:45 PDT
Add XHR tests checking readyState transition when abort() is invoked in various states
Attachments
Adds the test (7.12 KB, patch)
2013-09-18 20:09 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2013-09-18 20:09:34 PDT
Created attachment 212032 [details] Adds the test
Alexey Proskuryakov
Comment 2 2013-09-19 09:29:32 PDT
Comment on attachment 212032 [details] Adds the test View in context: https://bugs.webkit.org/attachment.cgi?id=212032&action=review > LayoutTests/http/tests/xmlhttprequest/readystatechange-and-abort.html:5 > +<script src="../w3c/resources/testharness.js"></script> > +<script src="../w3c/resources/testharnessreport.js"></script> This concerns me a little - aren't w3c test resources subject to change without notice? Might be worth checking with people who work on importing these.
Ryosuke Niwa
Comment 3 2013-09-19 10:33:23 PDT
(In reply to comment #2) > (From update of attachment 212032 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=212032&action=review > > > LayoutTests/http/tests/xmlhttprequest/readystatechange-and-abort.html:5 > > +<script src="../w3c/resources/testharness.js"></script> > > +<script src="../w3c/resources/testharnessreport.js"></script> > > This concerns me a little - aren't w3c test resources subject to change without notice? Might be worth checking with people who work on importing these. In theory but I highly doubt they'll make frequent changes to their test harness script used by tens of thousands of tests at this point.
WebKit Commit Bot
Comment 4 2013-09-19 10:56:17 PDT
Comment on attachment 212032 [details] Adds the test Clearing flags on attachment: 212032 Committed r156103: <http://trac.webkit.org/changeset/156103>
WebKit Commit Bot
Comment 5 2013-09-19 10:56:19 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.