Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/text

Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/text

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

BUG=707660
Review-Url: https://codereview.chromium.org/2811453002
Cr-Commit-Position: refs/heads/master@{#463569}
Committed: https://chromium.googlesource.com/chromium/src/+/28d7e2fdd0bb11a7d0061b60ee1664728ce869d5

[Hwanseung Lee, 2017-04-08 02:48:00]

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 02:48:10]

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 03:01:16]

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

[commit-bot: I haz the power, 2017-04-08 03:01:17]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Hwanseung Lee, 2017-04-08 03:34:45]

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 03:34:56]

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

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

[commit-bot: I haz the power, 2017-04-08 04:02:59]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

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

Patchset #2 (id:20001) has been deleted

[Hwanseung Lee, 2017-04-08 04:51:34]

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 04:51:42]

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 05:28:22]

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

[commit-bot: I haz the power, 2017-04-08 05:28:23]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[Hwanseung Lee, 2017-04-08 06:00: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-08 06:00:57]

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 06:26:35]

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

[commit-bot: I haz the power, 2017-04-08 06:26:36]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Hwanseung Lee, 2017-04-08 08:33:53]

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

[Hwanseung Lee, 2017-04-08 08:34:01]

Patchset #2 (id:40001) has been deleted

[commit-bot: I haz the power, 2017-04-08 08:34:06]

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-08 08:34:18]

Patchset #2 (id:60001) has been deleted

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

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

[commit-bot: I haz the power, 2017-04-08 09:52:10]

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-08 10:04:21]

Description was changed from

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

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

BUG=707660
==========

to

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

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

BUG=707660
==========

[Hwanseung Lee, 2017-04-08 10:04:21]

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

[Hwanseung Lee, 2017-04-08 10:04:34]

PTAL, thank you.

[eae, 2017-04-09 07:32:02]

LGTM

Thank you

[Hwanseung Lee, 2017-04-09 08:16:45]

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

[commit-bot: I haz the power, 2017-04-09 08:16:53]

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-09 10:07:06]

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

[commit-bot: I haz the power, 2017-04-09 10:07:08]

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

[Hwanseung Lee, 2017-04-09 10:36:44]

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

[commit-bot: I haz the power, 2017-04-09 10:36: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-09 11:07:47]

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

[commit-bot: I haz the power, 2017-04-09 11:07:50]

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-09 11:08:44]

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

[commit-bot: I haz the power, 2017-04-09 11:08:56]

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-09 22:05:10]

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

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

Failed to apply patch for
third_party/WebKit/Source/platform/text/BidiCharacterRun.h:
While running git apply --index -3 -p1;
  error: patch failed:
third_party/WebKit/Source/platform/text/BidiCharacterRun.h:39
  Falling back to three-way merge...
  Applied patch to 'third_party/WebKit/Source/platform/text/BidiCharacterRun.h'
with conflicts.
  U third_party/WebKit/Source/platform/text/BidiCharacterRun.h

Patch:       third_party/WebKit/Source/platform/text/BidiCharacterRun.h
Index: third_party/WebKit/Source/platform/text/BidiCharacterRun.h
diff --git a/third_party/WebKit/Source/platform/text/BidiCharacterRun.h
b/third_party/WebKit/Source/platform/text/BidiCharacterRun.h
index
f8f839c7c6c4bd9c828d858405c1c6055817465b..e8c17e5c1e5eeaa14e3eed11acadadee8817ea17
100644
--- a/third_party/WebKit/Source/platform/text/BidiCharacterRun.h
+++ b/third_party/WebKit/Source/platform/text/BidiCharacterRun.h
@@ -39,7 +39,7 @@ struct BidiCharacterRun {
         m_next(0),
         m_start(start),
         m_stop(stop) {
-    ASSERT(m_start <= m_stop);
+    DCHECK_LE(m_start, m_stop);
     if (dir == WTF::Unicode::OtherNeutral)
       dir = overrideDir;

[tkent, 2017-04-09 23:07:02]

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/BidiContext.h (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/BidiContext.h:74: DCHECK(level <=
kMaxLevel);
Use DCHECK_LE if it doesn't cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/BidiResolver.h (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/BidiResolver.h:451: DCHECK(m_status.last
!= NonSpacingMark && m_status.last != BoundaryNeutral &&
Split this into 7 DCHECKs.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/LocaleMac.mm (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleMac.mm:319: DCHECK(symbols.size()
== DecimalSeparatorIndex);
Use DCHECK_EQ if it doesn't cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleMac.mm:321: DCHECK(symbols.size()
== GroupSeparatorIndex);
Use DCHECK_EQ if it doesn't cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleMac.mm:323: DCHECK(symbols.size()
== DecimalSymbolsSize);
Use DCHECK_EQ if it doesn't cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/LocaleWin.cpp (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleWin.cpp:457: DCHECK(symbols.size()
== DecimalSeparatorIndex);
Use DCHECK_EQ if it doesn't cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleWin.cpp:459: DCHECK(symbols.size()
== GroupSeparatorIndex);
Use DCHECK_EQ if it doesn't cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleWin.cpp:461: DCHECK(symbols.size()
== DecimalSymbolsSize);
Use DCHECK_EQ if it doesn't cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/SegmentedString.cpp (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/SegmentedString.cpp:263:
DCHECK(!m_currentString.length() && !isComposite());
Split this into two DCHECKs

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/TextBreakIterator.h (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/TextBreakIterator.h:152:
DCHECK(priorContextLength <= priorContextCapacity);
Use DCHECK_LE if it doesn't cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp:226:
DCHECK(nativeIndex >= text->b && nativeIndex < nativeLength);
Split this into two DCHECKs.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp:232:
DCHECK(nativeIndex > text->b && nativeIndex <= nativeLength);
Split this into two DCHECKs

[Hwanseung Lee, 2017-04-10 23:13:05]

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

[commit-bot: I haz the power, 2017-04-10 23:13: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-10 23:33:19]

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

[commit-bot: I haz the power, 2017-04-10 23:33:20]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  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-11 01:41:37]

Patchset #3 (id:90001) has been deleted

[Hwanseung Lee, 2017-04-11 01:41: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-11 01:42:19]

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 03:26:09]

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

[commit-bot: I haz the power, 2017-04-11 03:26:09]

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-11 03:29:39]

tkent@
PTAL, thank you.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/BidiContext.h (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/BidiContext.h:74: DCHECK(level <=
kMaxLevel);
On 2017/04/09 23:07:01, tkent wrote:
> Use DCHECK_LE if it doesn't cause a compile failure.

it was cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/BidiResolver.h (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/BidiResolver.h:451: DCHECK(m_status.last
!= NonSpacingMark && m_status.last != BoundaryNeutral &&
On 2017/04/09 23:07:01, tkent wrote:
> Split this into 7 DCHECKs.

Done.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/LocaleMac.mm (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleMac.mm:319: DCHECK(symbols.size()
== DecimalSeparatorIndex);
On 2017/04/09 23:07:01, tkent wrote:
> Use DCHECK_EQ if it doesn't cause a compile failure.

it was cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleMac.mm:321: DCHECK(symbols.size()
== GroupSeparatorIndex);
On 2017/04/09 23:07:01, tkent wrote:
> Use DCHECK_EQ if it doesn't cause a compile failure.

it was cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleMac.mm:323: DCHECK(symbols.size()
== DecimalSymbolsSize);
On 2017/04/09 23:07:01, tkent wrote:
> Use DCHECK_EQ if it doesn't cause a compile failure.

it was cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/LocaleWin.cpp (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleWin.cpp:457: DCHECK(symbols.size()
== DecimalSeparatorIndex);
On 2017/04/09 23:07:01, tkent wrote:
> Use DCHECK_EQ if it doesn't cause a compile failure.

it was cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleWin.cpp:459: DCHECK(symbols.size()
== GroupSeparatorIndex);
On 2017/04/09 23:07:01, tkent wrote:
> Use DCHECK_EQ if it doesn't cause a compile failure.

it was cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/LocaleWin.cpp:461: DCHECK(symbols.size()
== DecimalSymbolsSize);
On 2017/04/09 23:07:01, tkent wrote:
> Use DCHECK_EQ if it doesn't cause a compile failure.

it was cause a compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/SegmentedString.cpp (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/SegmentedString.cpp:263:
DCHECK(!m_currentString.length() && !isComposite());
On 2017/04/09 23:07:01, tkent wrote:
> Split this into two DCHECKs

Done.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/TextBreakIterator.h (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/TextBreakIterator.h:152:
DCHECK(priorContextLength <= priorContextCapacity);
On 2017/04/09 23:07:01, tkent wrote:
> Use DCHECK_LE if it doesn't cause a compile failure.

it was cause compile failure.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp (right):

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp:226:
DCHECK(nativeIndex >= text->b && nativeIndex < nativeLength);
On 2017/04/09 23:07:02, tkent wrote:
> Split this into two DCHECKs.

Done.

https://codereview.chromium.org/2811453002/diff/80001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp:232:
DCHECK(nativeIndex > text->b && nativeIndex <= nativeLength);
On 2017/04/09 23:07:02, tkent wrote:
> Split this into two DCHECKs

Done.

[tkent, 2017-04-11 03:38:56]

lgtm with comments

https://codereview.chromium.org/2811453002/diff/110001/third_party/WebKit/Sou...
File third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp (right):

https://codereview.chromium.org/2811453002/diff/110001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp:243:
DCHECK(length <= std::numeric_limits<int32_t>::max());
Use DCHECK_LE

https://codereview.chromium.org/2811453002/diff/110001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp:281:
DCHECK(offset <= std::numeric_limits<int32_t>::max());
Use DCHECK_LE

https://codereview.chromium.org/2811453002/diff/110001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp:439:
DCHECK(length <= std::numeric_limits<int32_t>::max());
Use DCHECK_LE

[Hwanseung Lee, 2017-04-11 04:00:10]

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 04:00:46]

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 07:10:46]

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

[commit-bot: I haz the power, 2017-04-11 07:10:47]

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-11 07:49:09]

https://codereview.chromium.org/2811453002/diff/110001/third_party/WebKit/Sou...
File third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp (right):

https://codereview.chromium.org/2811453002/diff/110001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp:243:
DCHECK(length <= std::numeric_limits<int32_t>::max());
On 2017/04/11 03:38:56, tkent wrote:
> Use DCHECK_LE

Done.

https://codereview.chromium.org/2811453002/diff/110001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp:281:
DCHECK(offset <= std::numeric_limits<int32_t>::max());
On 2017/04/11 03:38:56, tkent wrote:
> Use DCHECK_LE

Done.

https://codereview.chromium.org/2811453002/diff/110001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/text/TextBreakIteratorICU.cpp:439:
DCHECK(length <= std::numeric_limits<int32_t>::max());
On 2017/04/11 03:38:56, tkent wrote:
> Use DCHECK_LE

Done.

[Hwanseung Lee, 2017-04-11 07:49:17]

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

[Hwanseung Lee, 2017-04-11 07:49:18]

The patchset sent to the CQ was uploaded after l-g-t-m from eae@chromium.org,
tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2811453002/#ps130001
(title: "fix")

[commit-bot: I haz the power, 2017-04-11 07:49:33]

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 07:54:44]

CQ is committing da patch.

Bot data: {"patchset_id": 130001, "attempt_start_ts": 1491896957742910,
"parent_rev": "14b25da4874f20bb712e927b56d66593b5280164", "commit_rev":
"28d7e2fdd0bb11a7d0061b60ee1664728ce869d5"}

[commit-bot: I haz the power, 2017-04-11 07:56:05]

Description was changed from

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

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

BUG=707660
==========

to

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

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

BUG=707660

Review-Url: https://codereview.chromium.org/2811453002
Cr-Commit-Position: refs/heads/master@{#463569}
Committed:
https://chromium.googlesource.com/chromium/src/+/28d7e2fdd0bb11a7d0061b60ee16...
==========

[commit-bot: I haz the power, 2017-04-11 07:56:06]

Committed patchset #4 (id:130001) as
https://chromium.googlesource.com/chromium/src/+/28d7e2fdd0bb11a7d0061b60ee16...