Bug 179607 – Several XHR and CORS tests are flaky due to inconsistent order of console messages

• Status: RESOLVED
• Resolution: FIXED
• Component: XML

[Ms2ger (he/him; ⌚ UTC+1/+2)]

... but only due to inconsistency in console logs:

-CONSOLE MESSAGE: Cannot use wildcard in Access-Control-Allow-Origin when credentials flag is true.
-CONSOLE MESSAGE: XMLHttpRequest cannot load http://127.0.0.1:8800/XMLHttpRequest/resources/access-control-basic-allow-star.py due to access control checks.
 CONSOLE MESSAGE: Origin http://localhost:8800 is not allowed by Access-Control-Allow-Origin.
 CONSOLE MESSAGE: XMLHttpRequest cannot load http://127.0.0.1:8800/XMLHttpRequest/resources/access-control-basic-allow.py due to access control checks.
 CONSOLE MESSAGE: Origin http://localhost:8800 is not allowed by Access-Control-Allow-Origin.
 CONSOLE MESSAGE: XMLHttpRequest cannot load http://127.0.0.1:8800/XMLHttpRequest/resources/access-control-basic-allow-no-credentials.py due to access control checks.
+CONSOLE MESSAGE: Cannot use wildcard in Access-Control-Allow-Origin when credentials flag is true.
+CONSOLE MESSAGE: XMLHttpRequest cannot load http://127.0.0.1:8800/XMLHttpRequest/resources/access-control-basic-allow-star.py due to access control checks.

[Ms2ger (he/him; ⌚ UTC+1/+2)]

webkit.org/b/179607 imported/w3c/web-platform-tests/cors/basic.htm [ Pass Failure ]
webkit.org/b/179607 imported/w3c/web-platform-tests/cors/preflight-failure.htm [ Pass Failure ]
webkit.org/b/179607 imported/w3c/web-platform-tests/cors/redirect-origin.htm [ Pass Failure ]
webkit.org/b/179607 imported/w3c/web-platform-tests/cors/redirect-preflight.htm [ Pass Failure ]
webkit.org/b/179607 imported/w3c/web-platform-tests/cors/status-preflight.htm [ Pass Failure ]
webkit.org/b/179607 imported/w3c/web-platform-tests/cors/status-async.htm [ Pass Failure ]

[Brendan McLoughlin]

I've opened a pull request on the web-platform-test repo to fix the inconsistent ordering of console messages. https://github.com/web-platform-tests/wpt/pull/11489/

Once this lands in WPT and is imported into webkit we should be able to remove the expectations for the following tests.

imported/w3c/web-platform-tests/cors/basic.htm
imported/w3c/web-platform-tests/cors/preflight-failure.htm
imported/w3c/web-platform-tests/cors/status-async.htm
imported/w3c/web-platform-tests/cors/status-preflight.htm
imported/w3c/web-platform-tests/xhr/access-control-and-redirects-async-same-origin.htm
imported/w3c/web-platform-tests/xhr/access-control-and-redirects-async.htm

The remaining tests mentioned in this issue will still be failing because they contain unique output in their console messages.

[youenn fablet]

*** Bug 179608 has been marked as a duplicate of this bug. ***

[youenn fablet]

Created attachment 383418 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 383418 [details]
Patch

Clearing flags on attachment: 383418

Committed r252427: <https://trac.webkit.org/changeset/252427>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/57165642>

[Karl Rackler]

imported/w3c/web-platform-tests/cors/preflight-failure.htm and imported/w3c/web-platform-tests/cors/status-async.htm are no longer failing - remove expectations

Current history is green: https://results.webkit.org/?suite=layout-tests&suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fcors%2Fremote-origin.htm&test=imported%2Fw3c%2Fweb-platform-tests%2Fcors%2Fstatus-async.htm

These test was isolated from the others in the group as ones that are unexpectedly passing.