Reset CORS status upon re-initiating fetch of a stylesheet.

Reset CORS status upon re-initiating fetch of a stylesheet.

When initiating a fetch of a potentially cross-origin stylesheet that's
already fetching, reset the is-following-CORS flag along with clearing
the loading resource.

R=haraken,mkwst
BUG=544690
Committed: https://crrev.com/53ab90724aca77dcd6ad912d023c282cdf4df117
Cr-Commit-Position: refs/heads/master@{#355306}

[sof, 2015-10-21 11:50:29]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, japhet@chromium.org, mkwst@chromium.org

[sof, 2015-10-21 11:50:30]

please take a look.

[haraken, 2015-10-21 13:47:41]

LGTM, but I want to have japhet confirm.

[Mike West, 2015-10-21 14:14:04]

LGTM % nit.

https://codereview.chromium.org/1413563005/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/security/link-crossorigin-stylesheet-reinserted.html
(right):

https://codereview.chromium.org/1413563005/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/security/link-crossorigin-stylesheet-reinserted.html:26:
div1.appendChild(link1);
Nit: Can you actually grab the element before appending it? I know the magical
ID creation thing works, but it confused me on the first pass through this file.

[sof, 2015-10-21 15:17:28]

Description was changed from

==========
Reset CORS status upon re-initiating fetch of a stylesheet.

When initiating a fetch of a potentially cross-origin stylesheet that's
already fetching, reset the is-following-CORS flag along with clearing
the loading resource.

R=
BUG=544690
==========

to

==========
Reset CORS status upon re-initiating fetch of a stylesheet.

When initiating a fetch of a potentially cross-origin stylesheet that's
already fetching, reset the is-following-CORS flag along with clearing
the loading resource.

R=haraken,mkwst
BUG=544690
==========

[sof, 2015-10-21 15:17:34]

https://codereview.chromium.org/1413563005/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/security/link-crossorigin-stylesheet-reinserted.html
(right):

https://codereview.chromium.org/1413563005/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/security/link-crossorigin-stylesheet-reinserted.html:26:
div1.appendChild(link1);
On 2015/10/21 14:14:04, Mike West (slow) wrote:
> Nit: Can you actually grab the element before appending it? I know the magical
> ID creation thing works, but it confused me on the first pass through this
file.

Can do that, no prizes for brevity in tests; done.

[sof, 2015-10-21 15:17:41]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2015-10-21 15:17:42]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, mkwst@chromium.org
Link to the patchset: https://codereview.chromium.org/1413563005/#ps20001
(title: "adjust test a bit")

[commit-bot: I haz the power, 2015-10-21 15:17:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1413563005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1413563005/20001

[commit-bot: I haz the power, 2015-10-21 16:18:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-21 16:19:42]

Patchset 2 (id:??) landed as
https://crrev.com/53ab90724aca77dcd6ad912d023c282cdf4df117
Cr-Commit-Position: refs/heads/master@{#355306}