New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 745385 link

Starred by 1 user
Status: Fixed
Owner:
█ aahlstrom@google.com
Email to this user bounced
Closed: Sep 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Porting CORS related XHR Layout Tests to Web Platform Tests

Project Member Reported by aahlstrom@google.com, Jul 18 2017 
There are CORS related XHR tests in two directories:
Layout Tests: src/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest
Web Platform Tests: src/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest

Consolidating them all into one location will be more convenient
Project Member

Comment 1 by sheriffbot@chromium.org, Jul 18 2017

Labels: Hotlist-Google
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 27 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a03f32888a033a56d15b552f888bad33f5a71ae5

commit a03f32888a033a56d15b552f888bad33f5a71ae5
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Jul 27 03:47:37 2017

Port XHR access-control LayoutTests to Web Platform Tests

Porting access-control-allow-lists-starting-with-comma.html
from LayoutTests to Web Platform Tests

Bug:  745385 
Change-Id: I035fe6b26463ff9e2bcf70af2be80464849766d4
Reviewed-on: https://chromium-review.googlesource.com/571249
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#489850}
[modify] https://crrev.com/a03f32888a033a56d15b552f888bad33f5a71ae5/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/allow-lists-starting-with-comma.htm
[modify] https://crrev.com/a03f32888a033a56d15b552f888bad33f5a71ae5/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-allow-lists.py
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 31 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b2795ddbc0983172dc42bec4b931028194f89811

commit b2795ddbc0983172dc42bec4b931028194f89811
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Mon Jul 31 09:26:22 2017

Removing ported LayoutTests

Removing failure expectation for removed tests

Bug:  745385 
Change-Id: Ica5c4b7f925ec2daed2c08b7f888e9cf03a4df82
Reviewed-on: https://chromium-review.googlesource.com/593384
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Cr-Commit-Position: refs/heads/master@{#490735}
[modify] https://crrev.com/b2795ddbc0983172dc42bec4b931028194f89811/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[delete] https://crrev.com/a863c3131fb85ef0da6634f6bfc7e6a91e6b6cb5/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-allow-lists-starting-with-comma-expected.txt
[delete] https://crrev.com/a863c3131fb85ef0da6634f6bfc7e6a91e6b6cb5/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-allow-lists-starting-with-comma.html
[delete] https://crrev.com/a863c3131fb85ef0da6634f6bfc7e6a91e6b6cb5/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-async-same-origin-expected.txt
[delete] https://crrev.com/a863c3131fb85ef0da6634f6bfc7e6a91e6b6cb5/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-async-same-origin.html
[delete] https://crrev.com/a863c3131fb85ef0da6634f6bfc7e6a91e6b6cb5/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-allow-no-credentials.cgi
Project Member

Comment 7 by bugdroid1@chromium.org, Aug 10 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1f7353d643f98cd29ad8ad09e25c32d60e513815

commit 1f7353d643f98cd29ad8ad09e25c32d60e513815
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Aug 10 00:55:01 2017

Porting access-control-basic-allow-preflight-cache-invalidation-by-header from LayoutTest to WPT


Bug:  745385 
Change-Id: Ib8964b1cc3f35713e7d5e90a7cefb6e0b6f286ef
Reviewed-on: https://chromium-review.googlesource.com/601811
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#493199}
[add] https://crrev.com/1f7353d643f98cd29ad8ad09e25c32d60e513815/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-basic-allow-preflight-cache-invalidation-by-header.htm
[add] https://crrev.com/1f7353d643f98cd29ad8ad09e25c32d60e513815/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-basic-preflight-cache-invalidation.py
[add] https://crrev.com/1f7353d643f98cd29ad8ad09e25c32d60e513815/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/reset-token.py
[delete] https://crrev.com/59d85acb04020ec97ea5b63c8e15fdc77a459e45/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-invalidation-by-header-expected.txt
[delete] https://crrev.com/59d85acb04020ec97ea5b63c8e15fdc77a459e45/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-invalidation-by-header.html
Project Member

Comment 9 by bugdroid1@chromium.org, Aug 10 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/182abef7308732fa392d65b000cb52fff45a88fb

commit 182abef7308732fa392d65b000cb52fff45a88fb
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Aug 10 05:45:40 2017

Porting access-control-and-redirects to WPT

This test has expected failures, where the original test did not.
The original test expected some of the CORS requests to be denied,
which is how they are currently implemented in Chrome,
but does not conform to the specs.

Chrome conforms with the specs for async XHR CORS redirect requests,
but does not currently support sync requests the same way.
Therefore, this test fails on sync test cases.

As a side note, in Firefox, all these test cases pass.

Converted from:
third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects.html

Bug:  745385 
Change-Id: Ib8c55620546e03f5e4b35ae954da034489040c1d
Reviewed-on: https://chromium-review.googlesource.com/583754
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#493299}
[modify] https://crrev.com/182abef7308732fa392d65b000cb52fff45a88fb/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/182abef7308732fa392d65b000cb52fff45a88fb/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-and-redirects-expected.txt
[add] https://crrev.com/182abef7308732fa392d65b000cb52fff45a88fb/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-and-redirects.htm
[add] https://crrev.com/182abef7308732fa392d65b000cb52fff45a88fb/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/redirect-cors.py
[delete] https://crrev.com/edb2f348f3abf7021492bbfd44299cbe613c821e/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-expected.txt
[delete] https://crrev.com/edb2f348f3abf7021492bbfd44299cbe613c821e/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects.html
Project Member

Comment 12 by bugdroid1@chromium.org, Aug 15 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f4167a6bd65b6bbbe6fba9a1958fc615129c3e9f

commit f4167a6bd65b6bbbe6fba9a1958fc615129c3e9f
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Tue Aug 15 03:38:51 2017

Porting access-control-basic-preflight-cache-timeout to WPT


Bug:  745385 
Change-Id: Ic1b2dcf8c0ff219040da1c4966bdcb0bdb71e663
Reviewed-on: https://chromium-review.googlesource.com/605153
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#494317}
[add] https://crrev.com/f4167a6bd65b6bbbe6fba9a1958fc615129c3e9f/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-basic-allow-preflight-cache-timeout.htm
[add] https://crrev.com/f4167a6bd65b6bbbe6fba9a1958fc615129c3e9f/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-basic-preflight-cache-timeout.py
[delete] https://crrev.com/fc32a59653e65cee968472d8c91fa060fdf981d9/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-timeout-expected.txt
[delete] https://crrev.com/fc32a59653e65cee968472d8c91fa060fdf981d9/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-timeout.html
[delete] https://crrev.com/fc32a59653e65cee968472d8c91fa060fdf981d9/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache-timeout.php
Project Member

Comment 15 by bugdroid1@chromium.org, Aug 16 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a752d6bd864234c762b9bb9d5522369fcdec4e9c

commit a752d6bd864234c762b9bb9d5522369fcdec4e9c
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Wed Aug 16 07:55:02 2017

Porting access-control-and-redirects-async (XHR) from LayoutTest to WPT


Bug:  745385 
Change-Id: Ib27d2d70370e70af3128d7ed43135225ffebda53
Reviewed-on: https://chromium-review.googlesource.com/609052
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Cr-Commit-Position: refs/heads/master@{#494718}
[modify] https://crrev.com/a752d6bd864234c762b9bb9d5522369fcdec4e9c/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/a752d6bd864234c762b9bb9d5522369fcdec4e9c/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-and-redirects-async.htm
[add] https://crrev.com/a752d6bd864234c762b9bb9d5522369fcdec4e9c/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/pass.txt
[modify] https://crrev.com/a752d6bd864234c762b9bb9d5522369fcdec4e9c/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/redirect-cors.py
[delete] https://crrev.com/b2a524683799e8f092f97b2bc71379f718702f3b/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-async-expected.txt
[delete] https://crrev.com/b2a524683799e8f092f97b2bc71379f718702f3b/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-and-redirects-async.html
Project Member

Comment 17 by bugdroid1@chromium.org, Aug 18 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/43d2fb6a80283559b896dc31382f16df47ecf2a9

commit 43d2fb6a80283559b896dc31382f16df47ecf2a9
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Fri Aug 18 02:53:39 2017

Porting access-control-basic-allow-preflight-cache from LayoutTest to WPT

Bug:  745385 
Change-Id: I9e189f0d3f1e5b88150e4058fb384573753ce41a
Reviewed-on: https://chromium-review.googlesource.com/615320
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Cr-Commit-Position: refs/heads/master@{#495441}
[add] https://crrev.com/43d2fb6a80283559b896dc31382f16df47ecf2a9/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-basic-allow-preflight-cache.htm
[add] https://crrev.com/43d2fb6a80283559b896dc31382f16df47ecf2a9/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-basic-preflight-cache.py
[delete] https://crrev.com/65841dfc9e96ae200a635e3d47fd92b172213ec1/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache-expected.txt
[delete] https://crrev.com/65841dfc9e96ae200a635e3d47fd92b172213ec1/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache.html
[delete] https://crrev.com/65841dfc9e96ae200a635e3d47fd92b172213ec1/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache.php
Project Member

Comment 18 by bugdroid1@chromium.org, Aug 18 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/54cf38c88c1ba497824b82989fb893882b772039

commit 54cf38c88c1ba497824b82989fb893882b772039
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Fri Aug 18 09:06:23 2017

Porting access-control-preflight-headers-sync from LayoutTest to WPT

Bug:  745385 
Change-Id: Ie1c5bbb57bef4dad892a2d43cba90fdb5778882b
Reviewed-on: https://chromium-review.googlesource.com/618534
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Cr-Commit-Position: refs/heads/master@{#495507}
[modify] https://crrev.com/54cf38c88c1ba497824b82989fb893882b772039/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/54cf38c88c1ba497824b82989fb893882b772039/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-preflight-headers-sync.htm
[add] https://crrev.com/54cf38c88c1ba497824b82989fb893882b772039/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/no-custom-header-on-preflight.py
[delete] https://crrev.com/7edf3b23bed6ea03309ddc27e65732f7cb00fe20/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-headers-sync-expected.txt
[delete] https://crrev.com/7edf3b23bed6ea03309ddc27e65732f7cb00fe20/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-headers-sync.html
Project Member

Comment 19 by bugdroid1@chromium.org, Aug 18 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4d0cbc3b3e6d6f3133a9b755be49ce21f4b9f716

commit 4d0cbc3b3e6d6f3133a9b755be49ce21f4b9f716
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Fri Aug 18 09:29:43 2017

Porting access-control-basic-post-fail-non-simple-content-type to WPT

Bug:  745385 
Change-Id: Iff8975a41dcace408427192c03087571b14bae53
Reviewed-on: https://chromium-review.googlesource.com/616482
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Cr-Commit-Position: refs/heads/master@{#495509}
[add] https://crrev.com/4d0cbc3b3e6d6f3133a9b755be49ce21f4b9f716/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-basic-post-with-non-cors-safelisted-content-type.htm
[add] https://crrev.com/4d0cbc3b3e6d6f3133a9b755be49ce21f4b9f716/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-basic-options-not-supported.py
[delete] https://crrev.com/194c7232405ac8e866b9192681fde244c9f5b513/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-post-fail-non-simple-content-type-expected.txt
[delete] https://crrev.com/194c7232405ac8e866b9192681fde244c9f5b513/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-post-fail-non-simple-content-type.html
[delete] https://crrev.com/194c7232405ac8e866b9192681fde244c9f5b513/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-options-not-supported.cgi
Project Member

Comment 20 by bugdroid1@chromium.org, Aug 23 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d3602dbcca459ebe4a39a53ff92f70c528be4153

commit d3602dbcca459ebe4a39a53ff92f70c528be4153
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Wed Aug 23 07:29:18 2017

Porting access-control-basic-post-success-no-content-type to WPT

Bug:  745385 
Change-Id: I4a8bdc86087b3c82d9bdbb628bd2e248fd860861
Reviewed-on: https://chromium-review.googlesource.com/616327
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#496614}
[modify] https://crrev.com/d3602dbcca459ebe4a39a53ff92f70c528be4153/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/d3602dbcca459ebe4a39a53ff92f70c528be4153/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-basic-post-success-no-content-type.htm
[delete] https://crrev.com/8dc523fae5a781273a32679806fed8b1c69ed0c1/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-post-success-no-content-type-expected.txt
[delete] https://crrev.com/8dc523fae5a781273a32679806fed8b1c69ed0c1/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-post-success-no-content-type.html
[delete] https://crrev.com/8dc523fae5a781273a32679806fed8b1c69ed0c1/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-post-success-no-content-type.php

Comment 21 by tyoshino@chromium.org, Aug 23 2017

Labels: -Hotlist-Google Hotlist-Interop
Status: Started (was: Assigned)
Project Member

Comment 22 by bugdroid1@chromium.org, Aug 25 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fe8dd81c1717310e34048cb0ac4672f027a11cd5

commit fe8dd81c1717310e34048cb0ac4672f027a11cd5
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Fri Aug 25 03:31:05 2017

Porting access-control-preflight-credential-sync from LayoutTest to WPT

Bug:  745385 
Change-Id: I876c9f3c90d93fcfa4c808d7a36e52c5e17d154d
Reviewed-on: https://chromium-review.googlesource.com/620332
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#497313}
[modify] https://crrev.com/fe8dd81c1717310e34048cb0ac4672f027a11cd5/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/fe8dd81c1717310e34048cb0ac4672f027a11cd5/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-preflight-credential-sync.htm
[add] https://crrev.com/fe8dd81c1717310e34048cb0ac4672f027a11cd5/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-auth-basic.py
[delete] https://crrev.com/07b77df562643f733dfed3609ff8c85894a2e501/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-credential-sync-expected.txt
[delete] https://crrev.com/07b77df562643f733dfed3609ff8c85894a2e501/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-credential-sync.html
Project Member

Comment 23 by bugdroid1@chromium.org, Aug 30 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/951031f81f596a3821a168ddab86460e710b3852

commit 951031f81f596a3821a168ddab86460e710b3852
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Wed Aug 30 08:03:07 2017

Porting access-control-preflight-request-header-lowercase to WPT

Bug:  745385 
Change-Id: Ibbe62d83f0ea45940be1f89f7bfe71ed7c62bf77
Reviewed-on: https://chromium-review.googlesource.com/620448
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Cr-Commit-Position: refs/heads/master@{#498392}
[modify] https://crrev.com/951031f81f596a3821a168ddab86460e710b3852/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/951031f81f596a3821a168ddab86460e710b3852/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-preflight-request-header-lowercase.htm
[add] https://crrev.com/951031f81f596a3821a168ddab86460e710b3852/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-preflight-request-header-lowercase.py
[delete] https://crrev.com/3f40c15481f438cdc515908c12cb2e638319a742/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-request-header-lowercase-expected.txt
[delete] https://crrev.com/3f40c15481f438cdc515908c12cb2e638319a742/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-request-header-lowercase.html
[delete] https://crrev.com/3f40c15481f438cdc515908c12cb2e638319a742/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-preflight-request-header-lowercase.php
Project Member

Comment 26 by bugdroid1@chromium.org, Aug 31 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d9b3775069bfef9ea3a44180151dff7449b59be2

commit d9b3775069bfef9ea3a44180151dff7449b59be2
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Aug 31 02:12:31 2017

Porting access-control-preflight-request-invalid-status-301 to WPT

Bug:  745385 
Change-Id: Ia6b56943033b0fcb6ed683a7cb60b9a2c3f86f20
Reviewed-on: https://chromium-review.googlesource.com/620449
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#498722}
[modify] https://crrev.com/d9b3775069bfef9ea3a44180151dff7449b59be2/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/d9b3775069bfef9ea3a44180151dff7449b59be2/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-preflight-request-invalid-status-301.htm
[add] https://crrev.com/d9b3775069bfef9ea3a44180151dff7449b59be2/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-preflight-request-invalid-status.py
[delete] https://crrev.com/80716088aa9282e7e9ad71411e115fa6f1613f50/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-request-invalid-status-301-expected.txt
[delete] https://crrev.com/80716088aa9282e7e9ad71411e115fa6f1613f50/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-request-invalid-status-301.html
Project Member

Comment 28 by bugdroid1@chromium.org, Aug 31 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5836a2970de5d1de7f1593cc89ecf2fa37477f67

commit 5836a2970de5d1de7f1593cc89ecf2fa37477f67
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Aug 31 05:00:22 2017

Porting access-control-basic-denied-preflight-cache to WPT

Note that in the server-side file,
the "Cache-Control" header is set to "no-store".
This does not relate specifically to this bug,
but is added as a general practice to avoid
unexpected behavior caused by caching.

Bug:  745385 
Change-Id: I0b25edd79cf2bf34c6c3602f74c99c895b2ca490
Reviewed-on: https://chromium-review.googlesource.com/630338
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#498755}
[modify] https://crrev.com/5836a2970de5d1de7f1593cc89ecf2fa37477f67/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/5836a2970de5d1de7f1593cc89ecf2fa37477f67/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-basic-preflight-denied.htm
[modify] https://crrev.com/5836a2970de5d1de7f1593cc89ecf2fa37477f67/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-basic-denied.py
[delete] https://crrev.com/ef5974549da8fb92738b689372a6bf35f13156ca/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-denied-preflight-cache-expected.txt
[delete] https://crrev.com/ef5974549da8fb92738b689372a6bf35f13156ca/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-denied-preflight-cache.html
Project Member

Comment 31 by bugdroid1@chromium.org, Sep 1 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9f6322feaaa70f698737ef53715f3d9160e8f828

commit 9f6322feaaa70f698737ef53715f3d9160e8f828
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Fri Sep 01 21:20:32 2017

Reland "Port XHR access-control LayoutTests to Web Platform Tests"

This is a reland of a03f32888a033a56d15b552f888bad33f5a71ae5.
In the original CL, wpt-exporter merged the provisional PR on GitHub
prematurely at patchset 3, even though the CL had not landed.
The final checked-in patchset 12 was therefore not exported, and the
change between PS3 and PS12 was eventually clobbered by the importer:
https://chromium.googlesource.com/chromium/src/+log/master/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/allow-lists-starting-with-comma.htm

Original change's description:
> Port XHR access-control LayoutTests to Web Platform Tests
> 
> Porting access-control-allow-lists-starting-with-comma.html
> from LayoutTests to Web Platform Tests
> 
> Bug:  745385 
> Change-Id: I035fe6b26463ff9e2bcf70af2be80464849766d4
> Reviewed-on: https://chromium-review.googlesource.com/571249
> Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
> Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
> Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#489850}

Bug: 754619,  745385 
Change-Id: I5dfdb8c9ccb4fe4fcae94c237eaee3196103e7f8
Reviewed-on: https://chromium-review.googlesource.com/648046
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#499302}
[modify] https://crrev.com/9f6322feaaa70f698737ef53715f3d9160e8f828/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/allow-lists-starting-with-comma.htm
[modify] https://crrev.com/9f6322feaaa70f698737ef53715f3d9160e8f828/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-allow-lists.py
Project Member

Comment 35 by bugdroid1@chromium.org, Sep 5 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e241e5c30d5954927735fe31b39ced2aa1859434

commit e241e5c30d5954927735fe31b39ced2aa1859434
Author: Yutaka Hirano <yhirano@chromium.org>
Date: Tue Sep 05 03:09:44 2017

Port access-control-preflight-request-header-sorted to wpt

Bug:  745385 
Change-Id: Id15bc7d09e55b1300edd618742efc2187c2ea577
Reviewed-on: https://chromium-review.googlesource.com/640192
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#499545}
[modify] https://crrev.com/e241e5c30d5954927735fe31b39ced2aa1859434/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/e241e5c30d5954927735fe31b39ced2aa1859434/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-preflight-request-header-sorted.htm
[add] https://crrev.com/e241e5c30d5954927735fe31b39ced2aa1859434/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-preflight-request-header-sorted.py
[delete] https://crrev.com/8f0604da06e730fc0c38a318577ddb1a64f6827e/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-request-header-sorted-expected.txt
[delete] https://crrev.com/8f0604da06e730fc0c38a318577ddb1a64f6827e/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-request-header-sorted.html
[delete] https://crrev.com/8f0604da06e730fc0c38a318577ddb1a64f6827e/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-preflight-request-header-sorted.php
Project Member

Comment 39 by bugdroid1@chromium.org, Sep 7 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c0161cc09bab97be77bb1ae5ce2878d858db9387

commit c0161cc09bab97be77bb1ae5ce2878d858db9387
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Sep 07 02:31:17 2017

Marking access-control-origin-header-in-isolated-world layout test as Chromium-specific

Bug:  745385 
Change-Id: I69acd351e612b4abe50122d4b66836b2d644bcbd
Reviewed-on: https://chromium-review.googlesource.com/640210
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500190}
[rename] https://crrev.com/c0161cc09bab97be77bb1ae5ce2878d858db9387/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/chromium/access-control-origin-header-in-isolated-world.html
[rename] https://crrev.com/c0161cc09bab97be77bb1ae5ce2878d858db9387/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/chromium/resources/access-control-origin-header-in-isolated-world.php
Project Member

Comment 40 by bugdroid1@chromium.org, Sep 7 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/121450d37cd747d6dae44671ffc0409473443ee2

commit 121450d37cd747d6dae44671ffc0409473443ee2
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Sep 07 02:39:16 2017

Porting access-control-preflight-request-headers-origin to WPT

Bug:  745385 
Change-Id: I446247cf8f980e17f7aadb4c922e2141ed97faaa
Reviewed-on: https://chromium-review.googlesource.com/642152
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500194}
[modify] https://crrev.com/121450d37cd747d6dae44671ffc0409473443ee2/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/121450d37cd747d6dae44671ffc0409473443ee2/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-preflight-request-headers-origin.htm
[add] https://crrev.com/121450d37cd747d6dae44671ffc0409473443ee2/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-preflight-request-headers-origin.py
[delete] https://crrev.com/7a933043a238e4c89d1e1aaeec6f575008769205/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-request-headers-origin-expected.txt
[delete] https://crrev.com/7a933043a238e4c89d1e1aaeec6f575008769205/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-request-headers-origin.html
[delete] https://crrev.com/7a933043a238e4c89d1e1aaeec6f575008769205/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-preflight-request-headers-origin.php
Project Member

Comment 41 by bugdroid1@chromium.org, Sep 7 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fda432b111b7d9b05956e8f57d9fa97b5176c052

commit fda432b111b7d9b05956e8f57d9fa97b5176c052
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Sep 07 02:46:08 2017

Porting access-control-response-with-body from LayoutTest to WPT

Bug:  745385 
Change-Id: I90461ac5d10414180198df10ca55a05be8ca1d66
Reviewed-on: https://chromium-review.googlesource.com/644830
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500197}
[modify] https://crrev.com/fda432b111b7d9b05956e8f57d9fa97b5176c052/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/fda432b111b7d9b05956e8f57d9fa97b5176c052/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-response-with-body.htm
[add] https://crrev.com/fda432b111b7d9b05956e8f57d9fa97b5176c052/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-allow-with-body.py
[delete] https://crrev.com/3ecad9bc91dbd2284e6ec0afab22f1b1186cae36/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-response-with-body-expected.txt
[delete] https://crrev.com/3ecad9bc91dbd2284e6ec0afab22f1b1186cae36/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-response-with-body.html
Project Member

Comment 43 by bugdroid1@chromium.org, Sep 11 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/fb0fa3d8fd203598e589fe21acb25789564fc759

commit fb0fa3d8fd203598e589fe21acb25789564fc759
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Mon Sep 11 05:48:13 2017

Porting access-control-response-with-expose-headers to WPT

Bug:  745385 
Change-Id: I4f66282879b301c8dcd02bb94f3d1f76fc246b7e
Reviewed-on: https://chromium-review.googlesource.com/645005
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#500845}
[modify] https://crrev.com/fb0fa3d8fd203598e589fe21acb25789564fc759/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/fb0fa3d8fd203598e589fe21acb25789564fc759/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-response-with-exposed-headers.htm
[delete] https://crrev.com/acdcf556ed4b49488d3bcf454aa5e7b6d4f9c03f/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-response-with-expose-headers-expected.txt
[delete] https://crrev.com/acdcf556ed4b49488d3bcf454aa5e7b6d4f9c03f/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-response-with-expose-headers.html
[delete] https://crrev.com/acdcf556ed4b49488d3bcf454aa5e7b6d4f9c03f/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-response-with-expose-headers.php
Project Member

Comment 44 by bugdroid1@chromium.org, Sep 12 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6779aa90cfda151418267d09551dcb574aa24017

commit 6779aa90cfda151418267d09551dcb574aa24017
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Tue Sep 12 03:12:30 2017

Porting access-control-sandboxed-iframe-allow from LayoutTest to WPT

Bug:  745385 
Change-Id: Iea2049e1012303d0655d514a6ba4b1406acfbc41
Reviewed-on: https://chromium-review.googlesource.com/644673
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Cr-Commit-Position: refs/heads/master@{#501164}
[modify] https://crrev.com/6779aa90cfda151418267d09551dcb574aa24017/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/6779aa90cfda151418267d09551dcb574aa24017/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-sandboxed-iframe-allow.htm
[add] https://crrev.com/6779aa90cfda151418267d09551dcb574aa24017/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-sandboxed-iframe.html
[delete] https://crrev.com/ad2bc45b4559ad65b3bb2227c1cc064f8b49b0aa/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-sandboxed-iframe-allow-expected.txt
[delete] https://crrev.com/ad2bc45b4559ad65b3bb2227c1cc064f8b49b0aa/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-sandboxed-iframe-allow.html
[delete] https://crrev.com/ad2bc45b4559ad65b3bb2227c1cc064f8b49b0aa/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-sandboxed-iframe-allow-iframe.html
[delete] https://crrev.com/ad2bc45b4559ad65b3bb2227c1cc064f8b49b0aa/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-sandboxed-iframe-allow.cgi
Project Member

Comment 45 by bugdroid1@chromium.org, Sep 12 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6dc789ca1ec1547a45e9c12615e4ed4ce2a515ab

commit 6dc789ca1ec1547a45e9c12615e4ed4ce2a515ab
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Tue Sep 12 05:08:15 2017

Porting access-control-sandboxed-iframe-allow-origin-null to WPT

Bug:  745385 
Change-Id: I5a70b0b71dc56c74c9786892e87befedb5b3a6a6
Reviewed-on: https://chromium-review.googlesource.com/648906
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501186}
[modify] https://crrev.com/6dc789ca1ec1547a45e9c12615e4ed4ce2a515ab/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/6dc789ca1ec1547a45e9c12615e4ed4ce2a515ab/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-sandboxed-iframe-allow-origin-null.htm
[delete] https://crrev.com/357487087b68fb14676e94c725a9c3abbb44d2d7/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-sandboxed-iframe-allow-origin-null-expected.txt
[delete] https://crrev.com/357487087b68fb14676e94c725a9c3abbb44d2d7/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-sandboxed-iframe-allow-origin-null.html
[delete] https://crrev.com/357487087b68fb14676e94c725a9c3abbb44d2d7/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-sandboxed-iframe-allow-origin-null-iframe.html
[delete] https://crrev.com/357487087b68fb14676e94c725a9c3abbb44d2d7/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-sandboxed-iframe-allow-origin-null.cgi
Project Member

Comment 46 by bugdroid1@chromium.org, Sep 12 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6ac9b6d9f39cfc14c488d42618e4db1a761635d9

commit 6ac9b6d9f39cfc14c488d42618e4db1a761635d9
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Tue Sep 12 05:22:22 2017

Porting access-control-sandboxed-iframe-denied from LayoutTest to WPT

Bug:  745385 
Change-Id: I6592737458f684e65738ca45aa34a58d5e2ed8bd
Reviewed-on: https://chromium-review.googlesource.com/647955
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501193}
[modify] https://crrev.com/6ac9b6d9f39cfc14c488d42618e4db1a761635d9/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/6ac9b6d9f39cfc14c488d42618e4db1a761635d9/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-sandboxed-iframe-denied.htm
[delete] https://crrev.com/8e88b315858e76683589c4e70359cbf9df6239af/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-sandboxed-iframe-denied-expected.txt
[delete] https://crrev.com/8e88b315858e76683589c4e70359cbf9df6239af/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-sandboxed-iframe-denied.html
[delete] https://crrev.com/8e88b315858e76683589c4e70359cbf9df6239af/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-sandboxed-iframe-denied-iframe.html
[delete] https://crrev.com/8e88b315858e76683589c4e70359cbf9df6239af/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-sandboxed-iframe-denied.cgi
Project Member

Comment 47 by bugdroid1@chromium.org, Sep 12 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5d91a717baf9da4a4a29b2033fdf6c2c8e789b27

commit 5d91a717baf9da4a4a29b2033fdf6c2c8e789b27
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Tue Sep 12 09:11:28 2017

Porting access-control-sandboxed-iframe-denied-without-wildcard to WPT

Bug:  745385 
Change-Id: I4eb69d0acfdeafecff59251f53179bfd5241d8cd
Reviewed-on: https://chromium-review.googlesource.com/647392
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501222}
[modify] https://crrev.com/5d91a717baf9da4a4a29b2033fdf6c2c8e789b27/third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
[add] https://crrev.com/5d91a717baf9da4a4a29b2033fdf6c2c8e789b27/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-sandboxed-iframe-denied-without-wildcard.htm
[delete] https://crrev.com/70772ae7f82dd7fac8a3e67280962624a97e2edc/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-sandboxed-iframe-denied-without-wildcard-expected.txt
[delete] https://crrev.com/70772ae7f82dd7fac8a3e67280962624a97e2edc/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-sandboxed-iframe-denied-without-wildcard.html
[delete] https://crrev.com/70772ae7f82dd7fac8a3e67280962624a97e2edc/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-sandboxed-iframe-denied-without-wildcard-iframe.html
[delete] https://crrev.com/70772ae7f82dd7fac8a3e67280962624a97e2edc/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-sandboxed-iframe-denied-without-wildcard.cgi
Project Member

Comment 51 by bugdroid1@chromium.org, Sep 14 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/386a4246f63951b406b1fa6524633f268b292d2a

commit 386a4246f63951b406b1fa6524633f268b292d2a
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Sep 14 06:24:00 2017

Quick fix on access-control-basic-post-with-non-cors-safelisted-content-type

I failed to implement a suggested change on this issue in another CL.
This CL fixes the issue. The relevant link is listed below:
https://chromium-review.googlesource.com/616482

Bug:  745385 
Change-Id: I3b2743895193ad4092d1178f3d0e1eb7d40c6766
Reviewed-on: https://chromium-review.googlesource.com/630976
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Cr-Commit-Position: refs/heads/master@{#501882}
[modify] https://crrev.com/386a4246f63951b406b1fa6524633f268b292d2a/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-basic-post-with-non-cors-safelisted-content-type.htm
Project Member

Comment 53 by bugdroid1@chromium.org, Sep 14 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ba0ffe3e582d222f320f154985ad4287fa619902

commit ba0ffe3e582d222f320f154985ad4287fa619902
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Sep 14 07:36:04 2017

Porting access-control-basic-whitelist-request-headers to WPT

Bug:  745385 
Change-Id: I8fc00478884e1ca5bcbf49bcc77cbb39dc8e2577
Reviewed-on: https://chromium-review.googlesource.com/616326
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501891}
[add] https://crrev.com/ba0ffe3e582d222f320f154985ad4287fa619902/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-basic-cors-safelisted-request-headers.htm
[add] https://crrev.com/ba0ffe3e582d222f320f154985ad4287fa619902/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-basic-cors-safelisted-request-headers.py
[delete] https://crrev.com/35951e431a3b20a91b0102173872b60e548f80b4/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-whitelist-request-headers-expected.txt
[delete] https://crrev.com/35951e431a3b20a91b0102173872b60e548f80b4/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-whitelist-request-headers.html
[delete] https://crrev.com/35951e431a3b20a91b0102173872b60e548f80b4/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-whitelist-request-headers.cgi
Project Member

Comment 54 by bugdroid1@chromium.org, Sep 14 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5b053693c3aa1dba3174a1262710e879cf19180e

commit 5b053693c3aa1dba3174a1262710e879cf19180e
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Sep 14 07:44:44 2017

Porting access-control-preflight-request-must-not-contain-cookie to WPT

Bug:  745385 
Change-Id: I83212ba1afd5ba11e0bbf59e8446f3b8a0d833d9
Reviewed-on: https://chromium-review.googlesource.com/656868
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501893}
[modify] https://crrev.com/5b053693c3aa1dba3174a1262710e879cf19180e/third_party/WebKit/LayoutTests/FlagExpectations/enable-features=NetworkService
[add] https://crrev.com/5b053693c3aa1dba3174a1262710e879cf19180e/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-preflight-request-must-not-contain-cookie.htm
[add] https://crrev.com/5b053693c3aa1dba3174a1262710e879cf19180e/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-cookie.py
[add] https://crrev.com/5b053693c3aa1dba3174a1262710e879cf19180e/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-preflight-request-must-not-contain-cookie.py
[delete] https://crrev.com/a314d1ecacb5d7fff24b94b27fe75e796653cc9b/third_party/WebKit/LayoutTests/http/tests/cookies/resources/third-party-cookie-relaxing-iframe.html
[delete] https://crrev.com/a314d1ecacb5d7fff24b94b27fe75e796653cc9b/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-request-must-not-contain-cookie-expected.txt
[delete] https://crrev.com/a314d1ecacb5d7fff24b94b27fe75e796653cc9b/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-preflight-request-must-not-contain-cookie.html
[delete] https://crrev.com/a314d1ecacb5d7fff24b94b27fe75e796653cc9b/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-preflight-request-must-not-contain-cookie.php
Project Member

Comment 55 by bugdroid1@chromium.org, Sep 14 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/db079f49d37b442020a63ede10748dc1141ab4c8

commit db079f49d37b442020a63ede10748dc1141ab4c8
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Thu Sep 14 09:05:27 2017

Porting access-control-basic-whitelist-response-headers to WPT

Bug:  745385 
Change-Id: I21ebfbb2fb0c22ddb2a3872119c87a1dcd36d90a
Reviewed-on: https://chromium-review.googlesource.com/634025
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501906}
[add] https://crrev.com/db079f49d37b442020a63ede10748dc1141ab4c8/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/access-control-basic-cors-safelisted-response-headers.htm
[add] https://crrev.com/db079f49d37b442020a63ede10748dc1141ab4c8/third_party/WebKit/LayoutTests/external/wpt/XMLHttpRequest/resources/access-control-basic-whitelist-response-headers.py
[delete] https://crrev.com/85b7fde46c7452381db2e1aee039746335069d6d/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-whitelist-response-headers-expected.txt
[delete] https://crrev.com/85b7fde46c7452381db2e1aee039746335069d6d/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/access-control-basic-whitelist-response-headers.html
[delete] https://crrev.com/85b7fde46c7452381db2e1aee039746335069d6d/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-whitelist-response-headers.cgi
Project Member

Comment 56 by bugdroid1@chromium.org, Sep 15 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/dec77345299abd82fc31b28d3233c0053ec6a461

commit dec77345299abd82fc31b28d3233c0053ec6a461
Author: Austin James Ahlstrom <aahlstrom@google.com>
Date: Fri Sep 15 02:39:27 2017

Removing unused server-side files

Bug:  745385 
Change-Id: If076664eb6fbacc48a08d91d6917df428618c9ba
Reviewed-on: https://chromium-review.googlesource.com/666368
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
Commit-Queue: Austin James Ahlstrom <aahlstrom@google.com>
Cr-Commit-Position: refs/heads/master@{#502140}
[delete] https://crrev.com/96f4437916860e307ec9b8d02b098cb784430c79/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-allow-with-body.php
[delete] https://crrev.com/96f4437916860e307ec9b8d02b098cb784430c79/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-allow-print-headers.cgi
[delete] https://crrev.com/96f4437916860e307ec9b8d02b098cb784430c79/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-allow-star.cgi
[delete] https://crrev.com/96f4437916860e307ec9b8d02b098cb784430c79/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-non-get-allow.cgi
[delete] https://crrev.com/96f4437916860e307ec9b8d02b098cb784430c79/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache-invalidation.php

Comment 57 by yhirano@chromium.org, Sep 20 2017

Status: Fixed (was: Started)

Comment 58 by phistuck@gmail.com, Sep 20 2017 

This was a wonderful initiative. I really appreciate it! Thank you, guys.

Comment 59 by yhirano@chromium.org, Sep 20 2017

Description: Show this description

Comment 60 by yhirano@chromium.org, Sep 20 2017

Summary: Porting CORS related XHR Layout Tests to Web Platform Tests (was: Porting XHR-related Layout Tests to Web Platform Tests)
Sorry, I forgot limiting the scope.
►

Sign in to add a comment