Bug 151091 - Remove WebKit specific XHR timeout tests in favor of WPT version
Summary: Remove WebKit specific XHR timeout tests in favor of WPT version
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords:
Depends on:
Blocks: 151103
  Show dependency tree / graph
 
Reported: 2015-11-10 05:45 PST by youenn fablet
Modified: 2016-02-23 09:24 PST (History)
6 users (show)

See Also:

Attachments
Patch (53.18 KB, patch)
2015-11-10 05:59 PST, youenn fablet 		no flags Details | Formatted Diff | Diff
Removing unrelated changes (52.34 KB, patch)
2015-11-10 06:29 PST, youenn fablet 		no flags 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 youenn fablet 2015-11-10 05:45:15 PST 
Several WebKit XHR timeout tests (in http/tests/xmlhttprequest/timeout/) have more up-to-date versions in imported/w3c/web-platform-tests/XMLHttpRequest.
It seems WPT versions are runned more consistently by bots.
Comment 1 youenn fablet 2015-11-10 05:59:07 PST 
Created attachment 265170 [details]
Patch
Comment 2 youenn fablet 2015-11-10 06:29:24 PST 
Created attachment 265173 [details]
Removing unrelated changes
Comment 3 WebKit Commit Bot 2015-11-10 09:57:32 PST 
Comment on attachment 265173 [details]
Removing unrelated changes

Clearing flags on attachment: 265173

Committed r192251: <http://trac.webkit.org/changeset/192251>
Comment 4 WebKit Commit Bot 2015-11-10 09:57:36 PST 
All reviewed patches have been landed.  Closing bug.
Comment 5 Alexey Proskuryakov 2016-02-23 09:24:33 PST 
Comment on attachment 265173 [details]
Removing unrelated changes

View in context: https://bugs.webkit.org/attachment.cgi?id=265173&action=review

> LayoutTests/ChangeLog:47
> +        * http/tests/xmlhttprequest/timeout/xmlhttprequest-timeout.js: Removed.

This file was used from tests/navigation/page-cache-xhr.html (for no good reason AFAICT). Will fix.