Replace remaining ASSERT with DCHECK|DCHECK_FOO in core/frame

Replace remaining ASSERT with DCHECK|DCHECK_FOO in core/frame

As a step to reduce uses of wtf/Assertions.h, This CL changes uses of ASSERT with
DCHECK() or DCHECK_FOO() in core/frame.

BUG=596760
Review-Url: https://codereview.chromium.org/2891843004
Cr-Commit-Position: refs/heads/master@{#473115}
Committed: https://chromium.googlesource.com/chromium/src/+/1dc16ecb4daf2aca6cb20a7c7bc29f2f22a8948e

[gyuyoung, 2017-05-19 02:01:09]

gyuyoung@igalia.com changed reviewers:
+ haraken@chromium.org, tkent@chromium.org

[gyuyoung, 2017-05-19 02:01:10]

PTAL.

[tkent, 2017-05-19 02:01:48]

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

[haraken, 2017-05-19 02:03:02]

LGTM

[tkent, 2017-05-19 02:04:07]

Please rebase.

> BUG=707638

The bug number is incorrect.  707638 is for Source/platform/.

[commit-bot: I haz the power, 2017-05-19 02:04:21]

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-19 02:04:22]

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

[commit-bot: I haz the power, 2017-05-19 02:04:22]

Dry run: The author gyuyoung@igalia.com has not signed Google Contributor
License Agreement. Please visit https://cla.developers.google.com to sign and
manage CLA.

[gyuyoung, 2017-05-19 02:07:13]

Description was changed from

==========
Replace remaining ASSERT with DCHECK|DCHECK_FOO in core/frame

As a step to reduce uses of wtf/Assertions.h, This CL changes uses of ASSERT
with
DCHECK() or DCHECK_FOO() in core/frame.

BUG=707638
==========

to

==========
Replace remaining ASSERT with DCHECK|DCHECK_FOO in core/frame

As a step to reduce uses of wtf/Assertions.h, This CL changes uses of ASSERT
with
DCHECK() or DCHECK_FOO() in core/frame.

BUG=596760
==========

[gyuyoung, 2017-05-19 02:07:34]

On 2017/05/19 02:04:07, tkent wrote:
> Please rebase.
> 
> > BUG=707638
> 
> The bug number is incorrect.  707638 is for Source/platform/.

Oops. Fixed.

[tkent, 2017-05-19 02:09:22]

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

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

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-19 07:01:12]

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

[commit-bot: I haz the power, 2017-05-19 07:01:13]

Dry run: This issue passed the CQ dry run.

[tkent, 2017-05-19 07:01:52]

The CQ bit was checked by tkent@chromium.org

[tkent, 2017-05-19 07:01:52]

lgtm

[tkent, 2017-05-19 07:01:53]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2891843004/#ps20001
(title: "Replace remaining ASSERT with DCHECK|DCHECK_FOO in core/frame")

[commit-bot: I haz the power, 2017-05-19 07:02:41]

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-19 07:07:16]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495177312246470,
"parent_rev": "34da61835c4da3f85eb33448a87a5dc530d6dc61", "commit_rev":
"1dc16ecb4daf2aca6cb20a7c7bc29f2f22a8948e"}

[commit-bot: I haz the power, 2017-05-19 07:07:27]

Description was changed from

==========
Replace remaining ASSERT with DCHECK|DCHECK_FOO in core/frame

As a step to reduce uses of wtf/Assertions.h, This CL changes uses of ASSERT
with
DCHECK() or DCHECK_FOO() in core/frame.

BUG=596760
==========

to

==========
Replace remaining ASSERT with DCHECK|DCHECK_FOO in core/frame

As a step to reduce uses of wtf/Assertions.h, This CL changes uses of ASSERT
with
DCHECK() or DCHECK_FOO() in core/frame.

BUG=596760

Review-Url: https://codereview.chromium.org/2891843004
Cr-Commit-Position: refs/heads/master@{#473115}
Committed:
https://chromium.googlesource.com/chromium/src/+/1dc16ecb4daf2aca6cb20a7c7bc2...
==========

[commit-bot: I haz the power, 2017-05-19 07:07:28]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/1dc16ecb4daf2aca6cb20a7c7bc2...