Remove rendundant WebLocalFrame parameter in various plugin code.

Remove rendundant WebLocalFrame parameter in various plugin code.

Plugin code all over pass around a RenderFrame and a WebLocalFrame, but
the RenderFrame parameter already knows about the associated
WebLocalFrame.

Review-Url: https://codereview.chromium.org/2855123003
Cr-Commit-Position: refs/heads/master@{#469746}
Committed: https://chromium.googlesource.com/chromium/src/+/f7d2fa11695bb0cf4a1e1b4a1a62b23c6e154236

[Lei Zhang, 2017-05-04 00:54:20]

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

[commit-bot: I haz the power, 2017-05-04 00:55:09]

Dry run: CQ is trying da patch.

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

[Lei Zhang, 2017-05-04 00:55:40]

Description was changed from

==========
Remove rendundant frame parameter in various plugin code.

The RenderFrame paramter already knows about the associated
WebLocalFrame.
==========

to

==========
Remove rendundant WebLocalFrame parameter in various plugin code.

Plugin code all over pass around a RenderFrame and a WebLocalFrame, but
the RenderFrame parameter already knows about the associated
WebLocalFrame.
==========

[commit-bot: I haz the power, 2017-05-04 01:06:23]

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

[commit-bot: I haz the power, 2017-05-04 01:06:24]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Lei Zhang, 2017-05-04 01:21:25]

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

[commit-bot: I haz the power, 2017-05-04 01:22:06]

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, 2017-05-04 01:35:22]

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

[commit-bot: I haz the power, 2017-05-04 01:35:23]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Lei Zhang, 2017-05-04 01:49:31]

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

[commit-bot: I haz the power, 2017-05-04 01:49:45]

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, 2017-05-04 02:03:10]

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

[commit-bot: I haz the power, 2017-05-04 02:03:12]

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

[Lei Zhang, 2017-05-04 02:19:44]

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

[commit-bot: I haz the power, 2017-05-04 02:20:11]

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, 2017-05-04 02:53:06]

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

[commit-bot: I haz the power, 2017-05-04 02:53:07]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)

[Lei Zhang, 2017-05-04 05:06:12]

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

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

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, 2017-05-04 06:52:29]

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

[commit-bot: I haz the power, 2017-05-04 06:52:30]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2017-05-04 07:32:53]

thestig@chromium.org changed reviewers:
+ bauerb@chromium.org

[Bernhard Bauer, 2017-05-04 09:34:08]

lgtm

[Lei Zhang, 2017-05-04 20:49:02]

thestig@chromium.org changed reviewers:
+ jochen@chromium.org, michaelpg@chromium.org

[Lei Zhang, 2017-05-04 20:49:03]

Adding more reviewers for OWNERS:
jochen: content/ and third_party/WebKit/
michaelpg: extensions/

[michaelpg, 2017-05-04 22:15:44]

lgtm

[jochen (gone - plz use gerrit), 2017-05-05 14:32:40]

lgtm

https://codereview.chromium.org/2855123003/diff/80001/third_party/WebKit/publ...
File third_party/WebKit/public/web/WebFrameClient.h (right):

https://codereview.chromium.org/2855123003/diff/80001/third_party/WebKit/publ...
third_party/WebKit/public/web/WebFrameClient.h:129: return nullptr;
nit. adding many but unrelated style fixes like this makes reviewing such CLs a
bit hard :/

[Lei Zhang, 2017-05-05 17:18:29]

https://codereview.chromium.org/2855123003/diff/80001/third_party/WebKit/publ...
File third_party/WebKit/public/web/WebFrameClient.h (right):

https://codereview.chromium.org/2855123003/diff/80001/third_party/WebKit/publ...
third_party/WebKit/public/web/WebFrameClient.h:129: return nullptr;
On 2017/05/05 14:32:40, jochen wrote:
> nit. adding many but unrelated style fixes like this makes reviewing such CLs
a
> bit hard :/

I'll try to make a separate cleanup pass next time.

[Lei Zhang, 2017-05-05 17:18:35]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2017-05-05 17:19:08]

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-05-05 19:11:47]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1494004715227680,
"parent_rev": "595a3733b47f8575411c53bf9de1e4066bc40027", "commit_rev":
"f7d2fa11695bb0cf4a1e1b4a1a62b23c6e154236"}

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

Description was changed from

==========
Remove rendundant WebLocalFrame parameter in various plugin code.

Plugin code all over pass around a RenderFrame and a WebLocalFrame, but
the RenderFrame parameter already knows about the associated
WebLocalFrame.
==========

to

==========
Remove rendundant WebLocalFrame parameter in various plugin code.

Plugin code all over pass around a RenderFrame and a WebLocalFrame, but
the RenderFrame parameter already knows about the associated
WebLocalFrame.

Review-Url: https://codereview.chromium.org/2855123003
Cr-Commit-Position: refs/heads/master@{#469746}
Committed:
https://chromium.googlesource.com/chromium/src/+/f7d2fa11695bb0cf4a1e1b4a1a62...
==========

[commit-bot: I haz the power, 2017-05-05 19:12:11]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/f7d2fa11695bb0cf4a1e1b4a1a62...