Use request initiator to check for cross origin requests.

Use request initiator to check if a request is cross origin.

With initiator() added to URLRequest there is no need to have different code
for regular and foreign fetch for this check, making the code simpler.

Committed: https://crrev.com/5ead27709174b374fb160b71b4702d6e99c07edb
Cr-Commit-Position: refs/heads/master@{#372711}

[Marijn Kruisselbrink, 2016-01-29 21:27:51]

Patchset #1 (id:1) has been deleted

[Marijn Kruisselbrink, 2016-01-29 21:37:06]

Description was changed from

==========
Use request initiator to check for cross origin requests.

With initiator() added to URLRequest this code can be simpler, using
the same implementation for regular and foreign fetch.
==========

to

==========
Use request initiator to check if a request is cross origin.

With initiator() added to URLRequest there is no need to have different code
for regular and foreign fetch for this check, making the code simpler.
==========

[Marijn Kruisselbrink, 2016-01-29 22:55:11]

mek@chromium.org changed reviewers:
+ horo@chromium.org, mkwst@chromium.org

[Marijn Kruisselbrink, 2016-01-29 22:55:11]

mkwst: I'm pretty sure initiator() is the right thing to use here, but since you
added URLRequest::initiator(), does this seem like the correct origin to use in
this check?
horo: as OWNER, and the person that wrote this check originally

[horo, 2016-02-01 03:47:29]

lgtm

[Mike West, 2016-02-01 06:32:07]

With the caveat that I've only tested the initiator code on the subset of
requests that actually hits the network, this LGTM. I think it should be a safe
assumption that's it's set in the cases you care about, but be careful. :)

[Marijn Kruisselbrink, 2016-02-01 17:53:29]

The CQ bit was checked by mek@chromium.org

[commit-bot: I haz the power, 2016-02-01 17:54:19]

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

[commit-bot: I haz the power, 2016-02-01 19:06:23]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-02-01 19:07:29]

Description was changed from

==========
Use request initiator to check if a request is cross origin.

With initiator() added to URLRequest there is no need to have different code
for regular and foreign fetch for this check, making the code simpler.
==========

to

==========
Use request initiator to check if a request is cross origin.

With initiator() added to URLRequest there is no need to have different code
for regular and foreign fetch for this check, making the code simpler.

Committed: https://crrev.com/5ead27709174b374fb160b71b4702d6e99c07edb
Cr-Commit-Position: refs/heads/master@{#372711}
==========

[commit-bot: I haz the power, 2016-02-01 19:07:30]

Patchset 1 (id:??) landed as
https://crrev.com/5ead27709174b374fb160b71b4702d6e99c07edb
Cr-Commit-Position: refs/heads/master@{#372711}