Replace ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/wtf

Replace ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/wtf

Replace RELEASE_ASSERT, and ASSERT_NOT_REACHED
with CHECK_op, and NOTREACHED respectively in
third_party/WebKit/Source/platform/wtf.

BUG=707661
Review-Url: https://codereview.chromium.org/2833123002
Cr-Commit-Position: refs/heads/master@{#466672}
Committed: https://chromium.googlesource.com/chromium/src/+/177d44c2b478ca96d5cff47940509b578e4af697

[Hwanseung Lee, 2017-04-21 14:25:38]

The CQ bit was checked by hs1217.lee@samsung.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-21 14:25:59]

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-04-21 14:35:12]

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

[commit-bot: I haz the power, 2017-04-21 14:35:13]

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

[Hwanseung Lee, 2017-04-21 15:46:43]

The CQ bit was checked by hs1217.lee@samsung.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-21 15:47:17]

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-04-21 16:35:21]

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

[commit-bot: I haz the power, 2017-04-21 16:35:22]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Hwanseung Lee, 2017-04-22 02:09:49]

The CQ bit was checked by hs1217.lee@samsung.com to run a CQ dry run

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

Dry run: CQ is trying da patch.

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

[Hwanseung Lee, 2017-04-22 03:39:12]

The CQ bit was checked by hs1217.lee@samsung.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-22 03:39: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-04-22 05:38:08]

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

[commit-bot: I haz the power, 2017-04-22 05:38:09]

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-22 06:21:40]

Description was changed from

==========
wtf

BUG=
==========

to

==========
Replace ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/wtf

Replace RELEASE_ASSERT, and ASSERT_NOT_REACHED
with CHECK_op, and NOTREACHED respectively in
third_party/WebKit/Source/platform/wtf.

BUG=707661
==========

[Hwanseung Lee, 2017-04-22 06:29:54]

The CQ bit was checked by hs1217.lee@samsung.com to run a CQ dry run

[commit-bot: I haz the power, 2017-04-22 06:30:07]

Dry run: CQ is trying da patch.

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

[Hwanseung Lee, 2017-04-22 06:30:10]

Patchset #1 (id:1) has been deleted

[Hwanseung Lee, 2017-04-22 06:30:21]

Patchset #1 (id:20001) has been deleted

[Hwanseung Lee, 2017-04-22 06:31:03]

Patchset #1 (id:40001) has been deleted

[commit-bot: I haz the power, 2017-04-22 08:14:08]

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

[commit-bot: I haz the power, 2017-04-22 08:14:09]

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-22 08:14:30]

hs1217.lee@samsung.com changed reviewers:
+ tkent@chromium.org

[Hwanseung Lee, 2017-04-22 08:14:31]

PTAL, thank you.

[tkent, 2017-04-23 23:00:45]

lgtm if one comment is addressed.

https://codereview.chromium.org/2833123002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/wtf/StackUtil.cpp (right):

https://codereview.chromium.org/2833123002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/wtf/StackUtil.cpp:190:
CHECK_GT(thread_stack_size, (unsigned) 4 * 0x1000);
Do not use C-style cast.  I think |4u * 0x1000| would work.

[Hwanseung Lee, 2017-04-24 15:19:36]

The CQ bit was checked by hs1217.lee@samsung.com

[Hwanseung Lee, 2017-04-24 15:19:36]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2833123002/#ps100001
(title: "wtf")

[commit-bot: I haz the power, 2017-04-24 15:20: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, 2017-04-24 17:38:03]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1493047176405740,
"parent_rev": "29488693e08129987436199a7adf060b5a83d38f", "commit_rev":
"177d44c2b478ca96d5cff47940509b578e4af697"}

[commit-bot: I haz the power, 2017-04-24 17:38:12]

Description was changed from

==========
Replace ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/wtf

Replace RELEASE_ASSERT, and ASSERT_NOT_REACHED
with CHECK_op, and NOTREACHED respectively in
third_party/WebKit/Source/platform/wtf.

BUG=707661
==========

to

==========
Replace ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/wtf

Replace RELEASE_ASSERT, and ASSERT_NOT_REACHED
with CHECK_op, and NOTREACHED respectively in
third_party/WebKit/Source/platform/wtf.

BUG=707661

Review-Url: https://codereview.chromium.org/2833123002
Cr-Commit-Position: refs/heads/master@{#466672}
Committed:
https://chromium.googlesource.com/chromium/src/+/177d44c2b478ca96d5cff4794050...
==========

[commit-bot: I haz the power, 2017-04-24 17:38:13]

Committed patchset #3 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/177d44c2b478ca96d5cff4794050...