Increase coverage for img element's crossOrigin attribute's behavior on redirect

Increase coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

R=yhirano@chromium.org
BUG=646277
Committed: https://crrev.com/131650a98e8d7dc8019b74c6221c8ac3db951c64
Cr-Commit-Position: refs/heads/master@{#425941}

[tyoshino (SeeGerritForStatus), 2016-09-13 08:21:23]

Description was changed from

==========
aaaaa

BUG=
==========

to

==========
Improve coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

BUG=646277
==========

[tyoshino (SeeGerritForStatus), 2016-09-13 08:22:12]

Description was changed from

==========
Improve coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

BUG=646277
==========

to

==========
Increase coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

BUG=646277
==========

[tyoshino (SeeGerritForStatus), 2016-09-13 08:27:02]

tyoshino@chromium.org changed reviewers:
+ hiroshige@chromium.org

[tyoshino (SeeGerritForStatus), 2016-09-13 08:27:33]

The CQ bit was checked by tyoshino@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-13 08:27:50]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-13 10:46:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-13 10:46:09]

Dry run: This issue passed the CQ dry run.

[tyoshino (SeeGerritForStatus), 2016-10-05 11:18:47]

The CQ bit was checked by tyoshino@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-05 11:19:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tyoshino (SeeGerritForStatus), 2016-10-05 11:20:21]

Hi hiroshige,

Could you please review this? If you don't have bandwidth for this or you think
this should be reviewed by someone else, please tell me.

Thanks

[commit-bot: I haz the power, 2016-10-05 12:45:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-05 12:45:20]

Dry run: This issue passed the CQ dry run.

[tyoshino (SeeGerritForStatus), 2016-10-14 04:27:11]

Description was changed from

==========
Increase coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

BUG=646277
==========

to

==========
Increase coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

BUG=646277
==========

[tyoshino (SeeGerritForStatus), 2016-10-14 04:27:12]

tyoshino@chromium.org changed reviewers:
+ yhirano@chromium.org

[tyoshino (SeeGerritForStatus), 2016-10-14 04:28:14]

Hi Yutaka, could you please review this?

[yhirano, 2016-10-14 09:11:10]

https://codereview.chromium.org/2333153002/diff/160001/third_party/WebKit/Lay...
File
third_party/WebKit/LayoutTests/http/tests/security/img-crossorigin-redirect-credentials.html
(right):

https://codereview.chromium.org/2333153002/diff/160001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/http/tests/security/img-crossorigin-redirect-credentials.html:144:
</script>
To be removed?

https://codereview.chromium.org/2333153002/diff/160001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/http/tests/security/img-crossorigin-redirect-credentials.html:153:
</script>
ditto

[tyoshino (SeeGerritForStatus), 2016-10-14 11:36:19]

Description was changed from

==========
Increase coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

BUG=646277
==========

to

==========
Increase coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

R=yhirano@chromium.org
BUG=646277
==========

[tyoshino (SeeGerritForStatus), 2016-10-14 12:09:35]

https://codereview.chromium.org/2333153002/diff/160001/third_party/WebKit/Lay...
File
third_party/WebKit/LayoutTests/http/tests/security/img-crossorigin-redirect-credentials.html
(right):

https://codereview.chromium.org/2333153002/diff/160001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/http/tests/security/img-crossorigin-redirect-credentials.html:144:
</script>
On 2016/10/14 09:11:09, yhirano wrote:
> To be removed?

ohhh! I don't remember why... Removed

https://codereview.chromium.org/2333153002/diff/160001/third_party/WebKit/Lay...
third_party/WebKit/LayoutTests/http/tests/security/img-crossorigin-redirect-credentials.html:153:
</script>
On 2016/10/14 09:11:09, yhirano wrote:
> ditto

Done.

[yhirano, 2016-10-17 12:44:39]

lgtm

[tyoshino (SeeGerritForStatus), 2016-10-18 10:26:44]

The CQ bit was checked by tyoshino@chromium.org

[tyoshino (SeeGerritForStatus), 2016-10-18 10:26:45]

The patchset sent to the CQ was uploaded after l-g-t-m from yhirano@chromium.org
Link to the patchset: https://codereview.chromium.org/2333153002/#ps200001
(title: "Rebase")

[commit-bot: I haz the power, 2016-10-18 10:26:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-18 11:24:15]

Description was changed from

==========
Increase coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

R=yhirano@chromium.org
BUG=646277
==========

to

==========
Increase coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

R=yhirano@chromium.org
BUG=646277
==========

[commit-bot: I haz the power, 2016-10-18 11:24:16]

Committed patchset #11 (id:200001)

[commit-bot: I haz the power, 2016-10-18 11:27:17]

Description was changed from

==========
Increase coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

R=yhirano@chromium.org
BUG=646277
==========

to

==========
Increase coverage for img element's crossOrigin attribute's behavior on redirect

Even after http://crbug.com/563328 was fixed, the behavior is not fully
conforming to the spec. Newly added tests will clarify the issue and
ensure it's working for the other cases.

R=yhirano@chromium.org
BUG=646277

Committed: https://crrev.com/131650a98e8d7dc8019b74c6221c8ac3db951c64
Cr-Commit-Position: refs/heads/master@{#425941}
==========

[commit-bot: I haz the power, 2016-10-18 11:27:18]

Patchset 11 (id:??) landed as
https://crrev.com/131650a98e8d7dc8019b74c6221c8ac3db951c64
Cr-Commit-Position: refs/heads/master@{#425941}