Add DCHECK to WebViewPlugin helper to assert WebFrameClient params.

Add DCHECK to WebViewPlugin helper to assert WebFrameClient params.

Since the WebLocalFrame* parameter being passed is already available
from WebViewPlugin::WebViewHelper, these should always match. This is
pre-work to remove the (now redundant) parameter since it's already
available from inside the method.

BUG=361765
Review-Url: https://codereview.chromium.org/2778393002
Cr-Commit-Position: refs/heads/master@{#460624}
Committed: https://chromium.googlesource.com/chromium/src/+/72687bdd262209de03d258b03d14f9eff4554a8d

[sashab, 2017-03-29 03:18:45]

sashab@chromium.org changed reviewers:
+ joelhockey@chromium.org

[joelhockey, 2017-03-29 03:22:07]

lgtm

[sashab, 2017-03-29 03:23:08]

sashab@chromium.org changed reviewers:
+ jochen@chromium.org

[jochen (gone - plz use gerrit), 2017-03-29 06:31:24]

lgtm

[sashab, 2017-03-29 06:31:43]

The CQ bit was checked by sashab@chromium.org

[commit-bot: I haz the power, 2017-03-29 06:32:30]

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-03-29 08:15:37]

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

[commit-bot: I haz the power, 2017-03-29 08:15:38]

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_...)

[sashab, 2017-03-30 01:36:48]

The CQ bit was checked by sashab@chromium.org

[commit-bot: I haz the power, 2017-03-30 01:38:02]

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-03-30 01:59:46]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490837808636350, "parent_rev":
"bc288f4389ba798aa9c3ad2f593a2b008d0b1a6a", "commit_rev":
"72687bdd262209de03d258b03d14f9eff4554a8d"}

[commit-bot: I haz the power, 2017-03-30 02:00:37]

Description was changed from

==========
Add DCHECK to WebViewPlugin helper to assert WebFrameClient params.

Since the WebLocalFrame* parameter being passed is already available
from WebViewPlugin::WebViewHelper, these should always match. This is
pre-work to remove the (now redundant) parameter since it's already
available from inside the method.

BUG=361765
==========

to

==========
Add DCHECK to WebViewPlugin helper to assert WebFrameClient params.

Since the WebLocalFrame* parameter being passed is already available
from WebViewPlugin::WebViewHelper, these should always match. This is
pre-work to remove the (now redundant) parameter since it's already
available from inside the method.

BUG=361765

Review-Url: https://codereview.chromium.org/2778393002
Cr-Commit-Position: refs/heads/master@{#460624}
Committed:
https://chromium.googlesource.com/chromium/src/+/72687bdd262209de03d258b03d14...
==========

[commit-bot: I haz the power, 2017-03-30 02:00:38]

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