Worker: Replace ASSERT macros with DCHECK macros

Worker: Replace ASSERT macros with DCHECK macros

  * ASSERT -> DCHECK
  * ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK
  * ASSERT_NOT_REACHED -> NOTREACHED
  * ENABLE(ASSERT) -> DCHECK_IS_ON()

NOTE: RELEASE_ASSERT is not replaced with CHECK in this CL because CHECK has a
performance problem. See the following link for details.

The background of this change:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/TL0NkNXIT1w

BUG=596760
Committed: https://crrev.com/faf5a858a3d7d507ad4732abc25f8210ad0f79ce
Cr-Commit-Position: refs/heads/master@{#388705}

[nhiroki, 2016-04-21 01:54:54]

Description was changed from

==========
Worker: Replace ASSERT macros with DCHECK macros

  * ASSERT -> DCHECK
  * ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK
  * ASSERT_NOT_REACHED -> NOTREACHED
  * ENABLE(ASSERT) -> DCHECK_IS_ON()

NOTE: RELEASE_ASSERT is not replaced with CHECK in this CL because CHECK has a
performance problem. See the following link for details.

The background of this change is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/TL0NkNXIT1w

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

to

==========
Worker: Replace ASSERT macros with DCHECK macros

  * ASSERT -> DCHECK
  * ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK
  * ASSERT_NOT_REACHED -> NOTREACHED
  * ENABLE(ASSERT) -> DCHECK_IS_ON()

NOTE: RELEASE_ASSERT is not replaced with CHECK in this CL because CHECK has a
performance problem. See the following link for details.

The background of this change:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/TL0NkNXIT1w

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

[nhiroki, 2016-04-21 02:08:35]

nhiroki@chromium.org changed reviewers:
+ tkent@chromium.org

[nhiroki, 2016-04-21 02:08:35]

Kent-san, can you review this? Thanks!

AFAICS, compile failures on bots are being caused by other changes.

[tkent, 2016-04-21 03:58:06]

lgtm

[nhiroki, 2016-04-21 05:14:59]

The CQ bit was checked by nhiroki@chromium.org

[commit-bot: I haz the power, 2016-04-21 05:15:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1902683005/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1902683005/1

[commit-bot: I haz the power, 2016-04-21 06:39:30]

Description was changed from

==========
Worker: Replace ASSERT macros with DCHECK macros

  * ASSERT -> DCHECK
  * ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK
  * ASSERT_NOT_REACHED -> NOTREACHED
  * ENABLE(ASSERT) -> DCHECK_IS_ON()

NOTE: RELEASE_ASSERT is not replaced with CHECK in this CL because CHECK has a
performance problem. See the following link for details.

The background of this change:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/TL0NkNXIT1w

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

to

==========
Worker: Replace ASSERT macros with DCHECK macros

  * ASSERT -> DCHECK
  * ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK
  * ASSERT_NOT_REACHED -> NOTREACHED
  * ENABLE(ASSERT) -> DCHECK_IS_ON()

NOTE: RELEASE_ASSERT is not replaced with CHECK in this CL because CHECK has a
performance problem. See the following link for details.

The background of this change:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/TL0NkNXIT1w

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

[commit-bot: I haz the power, 2016-04-21 06:39:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:31:59]

Description was changed from

==========
Worker: Replace ASSERT macros with DCHECK macros

  * ASSERT -> DCHECK
  * ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK
  * ASSERT_NOT_REACHED -> NOTREACHED
  * ENABLE(ASSERT) -> DCHECK_IS_ON()

NOTE: RELEASE_ASSERT is not replaced with CHECK in this CL because CHECK has a
performance problem. See the following link for details.

The background of this change:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/TL0NkNXIT1w

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

to

==========
Worker: Replace ASSERT macros with DCHECK macros

  * ASSERT -> DCHECK
  * ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK
  * ASSERT_NOT_REACHED -> NOTREACHED
  * ENABLE(ASSERT) -> DCHECK_IS_ON()

NOTE: RELEASE_ASSERT is not replaced with CHECK in this CL because CHECK has a
performance problem. See the following link for details.

The background of this change:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/TL0NkNXIT1w

BUG=596760

Committed: https://crrev.com/faf5a858a3d7d507ad4732abc25f8210ad0f79ce
Cr-Commit-Position: refs/heads/master@{#388705}
==========

[commit-bot: I haz the power, 2016-04-22 19:32:01]

Patchset 1 (id:??) landed as
https://crrev.com/faf5a858a3d7d507ad4732abc25f8210ad0f79ce
Cr-Commit-Position: refs/heads/master@{#388705}