Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/exported

Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/exported

Replace ASSERT, RELEASE_ASSERT, and ASSERT_NOT_REACHED
with DCHECK_op, CHECK_op, and NOTREACHED respectively in
third_party/WebKit/Source/platform/exported

BUG=707663
Review-Url: https://codereview.chromium.org/2811463002
Cr-Commit-Position: refs/heads/master@{#463818}
Committed: https://chromium.googlesource.com/chromium/src/+/8c2573026e78bc68e90b1d86aff58649ad855acf

[Hwanseung Lee, 2017-04-08 08:55:06]

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

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

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-08 10:11:43]

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

[commit-bot: I haz the power, 2017-04-08 10:11:44]

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-08 10:15:39]

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

[Hwanseung Lee, 2017-04-08 10:15:40]

PTAL, thank you.

[Hwanseung Lee, 2017-04-08 10:18:53]

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

[commit-bot: I haz the power, 2017-04-08 10:19:03]

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-08 12:36:21]

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

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

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-09 09:56:22]

Description was changed from

==========
Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/exported

Replace ASSERT, RELEASE_ASSERT, and ASSERT_NOT_REACHED
with DCHECK_op, CHECK_op, and NOTREACHED respectively in
third_party/WebKit/Source/platform/exported

BUG=707663
==========

to

==========
Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/exported

Replace ASSERT, RELEASE_ASSERT, and ASSERT_NOT_REACHED
with DCHECK_op, CHECK_op, and NOTREACHED respectively in
third_party/WebKit/Source/platform/exported

BUG=707663
==========

[Hwanseung Lee, 2017-04-09 09:56:22]

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

[tkent, 2017-04-09 22:57:41]

Please rebase.

[tzik, 2017-04-10 02:31:48]

lgtm

[Hwanseung Lee, 2017-04-11 14:52:31]

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

[commit-bot: I haz the power, 2017-04-11 14:52:45]

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-11 16:12:08]

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

[commit-bot: I haz the power, 2017-04-11 16:12:09]

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-11 16:29:15]

@tkent
PTAL, thank you.

[tkent, 2017-04-11 22:39:47]

lgtm

[Hwanseung Lee, 2017-04-11 22:43:08]

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

[Hwanseung Lee, 2017-04-11 22:43:09]

The patchset sent to the CQ was uploaded after l-g-t-m from tzik@chromium.org
Link to the patchset: https://codereview.chromium.org/2811463002/#ps40001
(title: "rebase")

[commit-bot: I haz the power, 2017-04-11 22:44:03]

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-11 22:54:11]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1491950588786980,
"parent_rev": "88b13e159698184c646a6b522a5313100bfcdf1b", "commit_rev":
"8c2573026e78bc68e90b1d86aff58649ad855acf"}

[commit-bot: I haz the power, 2017-04-11 22:55:10]

Description was changed from

==========
Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/exported

Replace ASSERT, RELEASE_ASSERT, and ASSERT_NOT_REACHED
with DCHECK_op, CHECK_op, and NOTREACHED respectively in
third_party/WebKit/Source/platform/exported

BUG=707663
==========

to

==========
Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/exported

Replace ASSERT, RELEASE_ASSERT, and ASSERT_NOT_REACHED
with DCHECK_op, CHECK_op, and NOTREACHED respectively in
third_party/WebKit/Source/platform/exported

BUG=707663

Review-Url: https://codereview.chromium.org/2811463002
Cr-Commit-Position: refs/heads/master@{#463818}
Committed:
https://chromium.googlesource.com/chromium/src/+/8c2573026e78bc68e90b1d86aff5...
==========

[commit-bot: I haz the power, 2017-04-11 22:55:11]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/8c2573026e78bc68e90b1d86aff5...

[dcheng1, 2017-07-11 03:15:03]

dcheng@google.com changed reviewers:
+ dcheng@google.com

[dcheng1, 2017-07-11 03:15:06]

https://codereview.chromium.org/2811463002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/exported/WebHTTPBody.cpp (right):

https://codereview.chromium.org/2811463002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/exported/WebHTTPBody.cpp:164:
DCHECK(static_cast<WebHTTPBodyPrivate*>(data.LeakRef()));
Just a friendly reminder to be careful with refactoring CLs. This was an
incorrect change (that's being fixed in
https://chromium-review.googlesource.com/c/565077/); luckily, it seems no code
was using this so nothing broke.