Move SetReferrerForRequest method from WebFrame to WebLocalFrame.

Move SetReferrerForRequest method from WebFrame to WebLocalFrame.

The SetReferrerForRequest method needs to access Document, which
doesn't make sense for a remote frame.  Therefore, this CL moves
this method from WebFrame to WebLocalFrame.

The caller in content/renderer/pepper/url_request_info_util.cc already
calls through WebLocalFrame.

The caller in media/blink/resource_multibuffer_data_provider.cc
calls through WebLocalFrame, but doesn't realize it.  This CL
changes s/WebFrame/WebLocalFrame/ in a few places under media/blink
to make this explicit.

No intended behavior change.

BUG=416660
Review-Url: https://codereview.chromium.org/2962573002
Cr-Commit-Position: refs/heads/master@{#482885}
Committed: https://chromium.googlesource.com/chromium/src/+/3034ad79e9d6a0ccdd067608b172b7ff126b3689

[Łukasz Anforowicz, 2017-06-26 19:56:50]

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

[commit-bot: I haz the power, 2017-06-26 19:57:08]

Dry run: CQ is trying da patch.

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

[Łukasz Anforowicz, 2017-06-26 20:51:03]

Description was changed from

==========
Move SetReferrerForRequest method from WebFrame to WebLocalFrame.

The caller in content/renderer/pepper/url_request_info_util.cc already
calls through WebLocalFrame.

The caller in media/blink/resource_multibuffer_data_provider.cc
calls through WebLocalFrame, but doesn't realize it.  This CL
changes s/WebFrame/WebLocalFrame/ in a few places under media/blink
to make this explicit.

No intended behavior change.

BUG=416660
==========

to

==========
Move SetReferrerForRequest method from WebFrame to WebLocalFrame.

The SetReferrerForRequest method needs to access Document, which
doesn't make sense for a remote frame.  Therefore, this CL moves
this method from WebFrame to WebLocalFrame.

The caller in content/renderer/pepper/url_request_info_util.cc already
calls through WebLocalFrame.

The caller in media/blink/resource_multibuffer_data_provider.cc
calls through WebLocalFrame, but doesn't realize it.  This CL
changes s/WebFrame/WebLocalFrame/ in a few places under media/blink
to make this explicit.

No intended behavior change.

BUG=416660
==========

[Łukasz Anforowicz, 2017-06-26 20:52:11]

lukasza@chromium.org changed reviewers:
+ watk@chromium.org

[Łukasz Anforowicz, 2017-06-26 20:52:11]

watk@, could you PTAL?

[commit-bot: I haz the power, 2017-06-26 21:58:27]

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

[commit-bot: I haz the power, 2017-06-26 21:58:29]

Dry run: This issue passed the CQ dry run.

[watk, 2017-06-27 17:54:21]

lgtm

[Łukasz Anforowicz, 2017-06-27 18:06:30]

lukasza@chromium.org changed reviewers:
+ chrishtr@chromium.org

[Łukasz Anforowicz, 2017-06-27 18:06:30]

chrishtr@, could you PTAL?  (I used to send CLs related to
https://crbug.com/416660 to dcheng@, but he is OOO for a while, so I picked you
as a random owner common across third_party/WebKit/Source/core/OWNERS and
third_party/WebKit/public/OWNERS;  I hope you don't mind :-).

[chrishtr, 2017-06-27 21:36:16]

Why doesn't it make sense for a remote frame? Shouldn't setting referrers work
for
all iframes?

[Łukasz Anforowicz, 2017-06-27 22:09:28]

On 2017/06/27 21:36:16, chrishtr wrote:
> Why doesn't it make sense for a remote frame? Shouldn't setting referrers work
> for
> all iframes?

Because a renderer doesn't know the URL of a remote frame (only its origin).

[chrishtr, 2017-06-27 22:10:35]

lgtm

[Łukasz Anforowicz, 2017-06-27 22:25:18]

The CQ bit was checked by lukasza@chromium.org

[Łukasz Anforowicz, 2017-06-27 22:25:36]

Thanks for the reviews!

[commit-bot: I haz the power, 2017-06-27 22:25: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, 2017-06-28 00:38:25]

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

[commit-bot: I haz the power, 2017-06-28 00:38:26]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Łukasz Anforowicz, 2017-06-28 04:05:00]

The CQ bit was checked by lukasza@chromium.org

[commit-bot: I haz the power, 2017-06-28 04:05:12]

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, 2017-06-28 04:53:37]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498622700030620, "parent_rev":
"8cf9ccb3cb8f650ab385ec2d359fcf27e76abf77", "commit_rev":
"3034ad79e9d6a0ccdd067608b172b7ff126b3689"}

[commit-bot: I haz the power, 2017-06-28 04:53:49]

Description was changed from

==========
Move SetReferrerForRequest method from WebFrame to WebLocalFrame.

The SetReferrerForRequest method needs to access Document, which
doesn't make sense for a remote frame.  Therefore, this CL moves
this method from WebFrame to WebLocalFrame.

The caller in content/renderer/pepper/url_request_info_util.cc already
calls through WebLocalFrame.

The caller in media/blink/resource_multibuffer_data_provider.cc
calls through WebLocalFrame, but doesn't realize it.  This CL
changes s/WebFrame/WebLocalFrame/ in a few places under media/blink
to make this explicit.

No intended behavior change.

BUG=416660
==========

to

==========
Move SetReferrerForRequest method from WebFrame to WebLocalFrame.

The SetReferrerForRequest method needs to access Document, which
doesn't make sense for a remote frame.  Therefore, this CL moves
this method from WebFrame to WebLocalFrame.

The caller in content/renderer/pepper/url_request_info_util.cc already
calls through WebLocalFrame.

The caller in media/blink/resource_multibuffer_data_provider.cc
calls through WebLocalFrame, but doesn't realize it.  This CL
changes s/WebFrame/WebLocalFrame/ in a few places under media/blink
to make this explicit.

No intended behavior change.

BUG=416660

Review-Url: https://codereview.chromium.org/2962573002
Cr-Commit-Position: refs/heads/master@{#482885}
Committed:
https://chromium.googlesource.com/chromium/src/+/3034ad79e9d6a0ccdd067608b172...
==========

[commit-bot: I haz the power, 2017-06-28 04:53:50]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/3034ad79e9d6a0ccdd067608b172...