WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
Formatted Diff
Diff
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
Details
View All
Add attachment
proposed patch, testcase, etc.
Basuke Suzuki
Comment 1
2018-08-09 16:02:00 PDT
Created
attachment 346868
[details]
fix
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
<
rdar://problem/43126703
>
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