RESOLVED FIXED 188454
[LayoutTests] Test case which committed on r175053 was wrong.
https://bugs.webkit.org/show_bug.cgi?id=188454
Summary [LayoutTests] Test case which committed on r175053 was wrong.
Basuke Suzuki
Reported 2018-08-09 15:57:38 PDT
It was originally committed on https://bugs.webkit.org/show_bug.cgi?id=136938 and has kept unchanged, but the test case uses non-existing redirect.php scripts which was removed a few months before than this bug. Also result was compared to wrong values.
Attachments
fix (6.03 KB, patch)
2018-08-09 16:02 PDT, Basuke Suzuki
no flags
Archive of layout-test-results from ews115 for mac-sierra (3.08 MB, application/zip)
2018-08-09 17:42 PDT, EWS Watchlist
no flags
Basuke Suzuki
Comment 1 2018-08-09 16:02:00 PDT
Basuke Suzuki
Comment 2 2018-08-09 16:02:50 PDT
I confirmed the result is same with current Chrome.
Basuke Suzuki
Comment 3 2018-08-09 16:07:00 PDT
Comment on attachment 346868 [details] fix View in context: https://bugs.webkit.org/attachment.cgi?id=346868&action=review > LayoutTests/http/tests/xmlhttprequest/cross-origin-redirect-responseURL-expected.txt:23 > +PASS responseURL is "" redirect fails so that responseURL should be empty. Also there's no readyState 2,3 happens. > LayoutTests/http/tests/xmlhttprequest/cross-origin-redirect-responseURL-expected.txt:31 > +PASS responseURL is "" Ditto. > LayoutTests/http/tests/xmlhttprequest/cross-origin-redirect-responseURL.html:-37 > -runTest('resources/redirect.php?url=http://127.0.0.1:8000/xmlhttprequest/resources/access-control-basic-allow.cgi', 'text').then(function(req) { xmlhttprequest/resources/redirect.php was moved to /resources/ with this bug. https://bugs.webkit.org/show_bug.cgi?id=119641
EWS Watchlist
Comment 4 2018-08-09 17:42:35 PDT
Comment on attachment 346868 [details] fix Attachment 346868 [details] did not pass mac-debug-ews (mac): Output: https://webkit-queues.webkit.org/results/8813588 New failing tests: fast/loader/redirect-to-invalid-url-using-meta-refresh-calls-policy-delegate.html
EWS Watchlist
Comment 5 2018-08-09 17:42:37 PDT
Created attachment 346879 [details] Archive of layout-test-results from ews115 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews115 Port: mac-sierra Platform: Mac OS X 10.12.6
WebKit Commit Bot
Comment 6 2018-08-09 22:13:53 PDT
Comment on attachment 346868 [details] fix Clearing flags on attachment: 346868 Committed r234751: <https://trac.webkit.org/changeset/234751>
WebKit Commit Bot
Comment 7 2018-08-09 22:13:55 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 8 2018-08-09 22:14:25 PDT
Note You need to log in before you can comment on or make changes to this bug.