Avoid gurl -> origin -> gurl conversion in AddCookieHeaderAndStart

Avoid gurl -> origin -> gurl conversion in AddCookieHeaderAndStart

SameDomainOrHost does not require Origins. In fact, we just convert the
origins back to GURLs to do the check, which just looks at the host
piece of the url.

This conversion is not free in either direction, as we need to do
allocations, copies, and in some cases, url-related policies
(e.g. in constructing the underlying SchemeHostPort of Origin).

BUG=664174
Committed: https://crrev.com/d5c68c496fa9111fb45e482d4feea0f1a2cdcb9e
Cr-Commit-Position: refs/heads/master@{#436092}

[Charlie Harrison, 2016-12-02 21:14:22]

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

[commit-bot: I haz the power, 2016-12-02 21:15:27]

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

[Charlie Harrison, 2016-12-02 22:03:13]

Description was changed from

==========
Avoid origin -> gurl -> origin conversion in AddCookieHeaderAndStart

SameDomainOrHost does not require Origins. In fact, we just convert the
origins back to GURLs to do the check, which just looks at the host
piece of the url.

This conversion is not free in either direction, as we need to do
allocations, copies, and in some cases, url-related policies
(e.g. in constructing the underlying SchemeHostPort of Origin).

BUG=664174
==========

to

==========
Avoid gurl -> origin -> gurl conversion in AddCookieHeaderAndStart

SameDomainOrHost does not require Origins. In fact, we just convert the
origins back to GURLs to do the check, which just looks at the host
piece of the url.

This conversion is not free in either direction, as we need to do
allocations, copies, and in some cases, url-related policies
(e.g. in constructing the underlying SchemeHostPort of Origin).

BUG=664174
==========

[Charlie Harrison, 2016-12-02 22:03:30]

csharrison@chromium.org changed reviewers:
+ mmenke@chromium.org

[Charlie Harrison, 2016-12-02 22:03:30]

mmenke@ PTAL (trivial)?

[mmenke, 2016-12-02 22:17:42]

On 2016/12/02 22:03:30, Charlie Harrison wrote:
> mmenke@ PTAL (trivial)?

LGTM

[Charlie Harrison, 2016-12-02 22:28:25]

The CQ bit was unchecked by csharrison@chromium.org

[Charlie Harrison, 2016-12-02 22:28:25]

The CQ bit was checked by csharrison@chromium.org

[commit-bot: I haz the power, 2016-12-02 22:28:46]

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-12-03 00:15:30]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480717705529270, "parent_rev":
"9b598f3291c0baaa7e6f609891d67421f1ea0c0b", "commit_rev":
"e55ed9882b2397919385f6a4ab434af46c8e95ae"}

[commit-bot: I haz the power, 2016-12-03 00:15:56]

Description was changed from

==========
Avoid gurl -> origin -> gurl conversion in AddCookieHeaderAndStart

SameDomainOrHost does not require Origins. In fact, we just convert the
origins back to GURLs to do the check, which just looks at the host
piece of the url.

This conversion is not free in either direction, as we need to do
allocations, copies, and in some cases, url-related policies
(e.g. in constructing the underlying SchemeHostPort of Origin).

BUG=664174
==========

to

==========
Avoid gurl -> origin -> gurl conversion in AddCookieHeaderAndStart

SameDomainOrHost does not require Origins. In fact, we just convert the
origins back to GURLs to do the check, which just looks at the host
piece of the url.

This conversion is not free in either direction, as we need to do
allocations, copies, and in some cases, url-related policies
(e.g. in constructing the underlying SchemeHostPort of Origin).

BUG=664174
==========

[commit-bot: I haz the power, 2016-12-03 00:15:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-03 00:20:19]

Description was changed from

==========
Avoid gurl -> origin -> gurl conversion in AddCookieHeaderAndStart

SameDomainOrHost does not require Origins. In fact, we just convert the
origins back to GURLs to do the check, which just looks at the host
piece of the url.

This conversion is not free in either direction, as we need to do
allocations, copies, and in some cases, url-related policies
(e.g. in constructing the underlying SchemeHostPort of Origin).

BUG=664174
==========

to

==========
Avoid gurl -> origin -> gurl conversion in AddCookieHeaderAndStart

SameDomainOrHost does not require Origins. In fact, we just convert the
origins back to GURLs to do the check, which just looks at the host
piece of the url.

This conversion is not free in either direction, as we need to do
allocations, copies, and in some cases, url-related policies
(e.g. in constructing the underlying SchemeHostPort of Origin).

BUG=664174

Committed: https://crrev.com/d5c68c496fa9111fb45e482d4feea0f1a2cdcb9e
Cr-Commit-Position: refs/heads/master@{#436092}
==========

[commit-bot: I haz the power, 2016-12-03 00:20:20]

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