Add DCHECK to WebEmbeddedWorkerImpl to assert WebFrameClient params.

Add DCHECK to WebEmbeddedWorkerImpl to assert WebFrameClient params.

Since the WebLocalFrame* parameter being passed is already available
from WebEmbeddedWorkerImpl, 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/2779543007
Cr-Commit-Position: refs/heads/master@{#460302}
Committed: https://chromium.googlesource.com/chromium/src/+/0611b01da7b4979fada8de6369f948cc3ba53dd2

[sashab, 2017-03-28 23:21:50]

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

[sashab, 2017-03-28 23:22:03]

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

[commit-bot: I haz the power, 2017-03-28 23:22:36]

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

[sashab, 2017-03-28 23:24:59]

Description was changed from

==========
Add DCHECKs to WebEmbeddedWorkerImpl to assert WebFrameClient params.

Since the WebLocalFrame* parameter being passed is already available
from WebEmbeddedWorkerImpl, 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 WebEmbeddedWorkerImpl to assert WebFrameClient params.

Since the WebLocalFrame* parameter being passed is already available
from WebEmbeddedWorkerImpl, 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
==========

[joelhockey, 2017-03-29 00:28:14]

lgtm

[sashab, 2017-03-29 00:28:53]

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

[commit-bot: I haz the power, 2017-03-29 01:10:22]

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

[commit-bot: I haz the power, 2017-03-29 01:10:22]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[sashab, 2017-03-29 03:27:29]

sashab@chromium.org changed reviewers:
+ dcheng@chromium.org
- haraken@chromium.org

[sashab, 2017-03-29 03:27:29]

-haraken +dcheng

[dcheng, 2017-03-29 03:58:47]

rs lgtm

[kinuko, 2017-03-29 04:52:35]

lgtm (and ditto)

[sashab, 2017-03-29 05:03:53]

The CQ bit was checked by sashab@chromium.org

[commit-bot: I haz the power, 2017-03-29 05:04:19]

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

[haraken, 2017-03-29 06:20:09]

LGTM

[commit-bot: I haz the power, 2017-03-29 06:35:16]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490763833673630, "parent_rev":
"f244469aa781823cdde22de772e1ca1944f55b2f", "commit_rev":
"0611b01da7b4979fada8de6369f948cc3ba53dd2"}

[commit-bot: I haz the power, 2017-03-29 06:36:10]

Description was changed from

==========
Add DCHECK to WebEmbeddedWorkerImpl to assert WebFrameClient params.

Since the WebLocalFrame* parameter being passed is already available
from WebEmbeddedWorkerImpl, 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 WebEmbeddedWorkerImpl to assert WebFrameClient params.

Since the WebLocalFrame* parameter being passed is already available
from WebEmbeddedWorkerImpl, 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/2779543007
Cr-Commit-Position: refs/heads/master@{#460302}
Committed:
https://chromium.googlesource.com/chromium/src/+/0611b01da7b4979fada8de6369f9...
==========

[commit-bot: I haz the power, 2017-03-29 06:36:11]

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