My patch http://crrev.com/b3b697fc8bcc938e8b9ab32a34fc5933494faaa1 broke
the PingLoader code path for CSP violation reporting by always passing
AllowStoredCredentials by mistake.
The test report-cross-origin-no-cookies.html (now named .php) introduced
by http://crrev.com/a161a28377d8d71b63a02066574fb47f03dd4b3b included a
sync XHR to set a cookie for the remote host, but it has been not
working because:
- its withCredentials is not set to true
- testRunner is not configured to accept third party cookies
- setCookies.cgi emits wildcard Access-Control-Allow-Origin which is
invalid for credentialled CORS requests
- it has non-CORS-safelisted header SET-COOKIE
So, the test didn't catch this breakage.
Patch: https://codereview.chromium.org/2345463002/
The revision number where the bug was introduced is r411630. So, M54 (current beta) and later are affected.
Comment 1 by bugdroid1@chromium.org
, Oct 4 2016