Replace ASSERT family with DCHECK and so on in core/dom/.

Replace ASSERT family with DCHECK and so on in core/dom/.

- ASSERT_UNUSED -> DCHECK*
- ASSERT_NOT_REACHED -> NOTREACHED
- ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK

* core/dom/SpaceSplitString.h
 Remove two ASSERT_WITH_SECURITY_IMPLICATIONs. Vector[] has a RELEASE_ASSERT for
 the same condition. These ASSERT_WITH_SECURITY_IMPLICATIONs were redundant.

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

[tkent, 2016-10-06 03:17:19]

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

[commit-bot: I haz the power, 2016-10-06 03:17:45]

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

[tkent, 2016-10-06 03:30:42]

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

[tkent, 2016-10-06 03:30:50]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-10-06 03:31:00]

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

[tkent, 2016-10-06 03:31:31]

Description was changed from

==========
Replace ASSERT family with DCHECK, etc. in core/dom/.

- ASSERT_UNUSED -> DCHECK*
- ASSERT_NOT_REACHED -> NOTREACHED
- ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK

* core/dom/SpaceSplitString.h
 Remove two ASSERT_WITH_SECURITY_IMPLICATIONs. Vector[] has a RELEASE_ASSERT for
 the same condition. These ASSERT_WITH_SECURITY_IMPLICATIONs were redundant.

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

to

==========
Replace ASSERT family with DCHECK and so on in core/dom/.

- ASSERT_UNUSED -> DCHECK*
- ASSERT_NOT_REACHED -> NOTREACHED
- ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK

* core/dom/SpaceSplitString.h
 Remove two ASSERT_WITH_SECURITY_IMPLICATIONs. Vector[] has a RELEASE_ASSERT for
 the same condition. These ASSERT_WITH_SECURITY_IMPLICATIONs were redundant.

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

[commit-bot: I haz the power, 2016-10-06 03:53:00]

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

[commit-bot: I haz the power, 2016-10-06 03:53:00]

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

[tkent, 2016-10-06 04:20:00]

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

[tkent, 2016-10-06 04:20:13]

Patchset #1 (id:20001) has been deleted

[commit-bot: I haz the power, 2016-10-06 04:20:13]

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

[tkent, 2016-10-06 04:26:48]

Patchset #1 (id:20001) has been deleted

[tkent, 2016-10-06 05:27:08]

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

[tkent, 2016-10-06 05:27:08]

hayato@, would you review this please?

[commit-bot: I haz the power, 2016-10-06 05:56:56]

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

[commit-bot: I haz the power, 2016-10-06 05:56:57]

Dry run: This issue passed the CQ dry run.

[hayato, 2016-10-06 06:30:10]

lgtm

[tkent, 2016-10-06 06:44:03]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-10-06 06:44:12]

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, 2016-10-06 06:52:22]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-10-06 06:54:37]

Description was changed from

==========
Replace ASSERT family with DCHECK and so on in core/dom/.

- ASSERT_UNUSED -> DCHECK*
- ASSERT_NOT_REACHED -> NOTREACHED
- ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK

* core/dom/SpaceSplitString.h
 Remove two ASSERT_WITH_SECURITY_IMPLICATIONs. Vector[] has a RELEASE_ASSERT for
 the same condition. These ASSERT_WITH_SECURITY_IMPLICATIONs were redundant.

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

to

==========
Replace ASSERT family with DCHECK and so on in core/dom/.

- ASSERT_UNUSED -> DCHECK*
- ASSERT_NOT_REACHED -> NOTREACHED
- ASSERT_WITH_SECURITY_IMPLICATION -> SECURITY_DCHECK

* core/dom/SpaceSplitString.h
 Remove two ASSERT_WITH_SECURITY_IMPLICATIONs. Vector[] has a RELEASE_ASSERT for
 the same condition. These ASSERT_WITH_SECURITY_IMPLICATIONs were redundant.

BUG=596760

Committed: https://crrev.com/241ea4031fccfda293385b4ce806f2596b3c4d97
Cr-Commit-Position: refs/heads/master@{#423455}
==========

[commit-bot: I haz the power, 2016-10-06 06:54:38]

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