Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/graphics

Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/graphics

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

BUG=707658
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2807923002
Cr-Commit-Position: refs/heads/master@{#463903}
Committed: https://chromium.googlesource.com/chromium/src/+/94e9d9cf0302323949e90805222c8aa342b53641

[Hwanseung Lee, 2017-04-08 11:39:27]

Description was changed from

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

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

BUG=707658
==========

to

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

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

BUG=707658
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Hwanseung Lee, 2017-04-08 11:40:07]

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 11:40:13]

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 11:40:49]

Description was changed from

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

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

BUG=707658
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

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

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

BUG=707658
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Hwanseung Lee, 2017-04-08 11:40:49]

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

[commit-bot: I haz the power, 2017-04-08 11:43:01]

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

[commit-bot: I haz the power, 2017-04-08 11:43:01]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Hwanseung Lee, 2017-04-08 11:50: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-08 11:50:51]

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:12:24]

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

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

Dry run: Try jobs failed on following builders:
  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-...)

[Hwanseung Lee, 2017-04-08 12:27:30]

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 12:27:38]

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:42:21]

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

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

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

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

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 14:19:48]

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

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

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

[Hwanseung Lee, 2017-04-08 14:29:07]

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 14:29:16]

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 17:16:54]

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

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

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

[chrishtr, 2017-04-08 18:26:52]

lgtm

[Hwanseung Lee, 2017-04-08 22:58:16]

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

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

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 01:51:10]

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

[commit-bot: I haz the power, 2017-04-09 01:51:12]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Hwanseung Lee, 2017-04-09 02:35: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-09 02:35:54]

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 05:19:06]

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

[commit-bot: I haz the power, 2017-04-09 05:19:07]

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

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

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 07:11:28]

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

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

[commit-bot: I haz the power, 2017-04-09 10:00:06]

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

[Hwanseung Lee, 2017-04-09 10:05: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-09 10:06:06]

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 10:35:27]

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

[commit-bot: I haz the power, 2017-04-09 10:35:27]

Dry run: This issue passed the CQ dry run.

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

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

[commit-bot: I haz the power, 2017-04-09 10:36:28]

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

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

[commit-bot: I haz the power, 2017-04-09 22:03:58]

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

Patch:      
third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridgeTest.cpp
Index: third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridgeTest.cpp
diff --git
a/third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridgeTest.cpp
b/third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridgeTest.cpp
index
58cf4e4a5598d2c34390529ea1bd7b7774d31bbf..784f0e5f5af4712910b6d6ca69b1513ed41c2d9b
100644
--- a/third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridgeTest.cpp
+++ b/third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridgeTest.cpp
@@ -81,7 +81,7 @@ class Canvas2DLayerBridgePtr {
   }
 
   void operator=(PassRefPtr<Canvas2DLayerBridge> layerBridge) {
-    ASSERT(!m_layerBridge);
+    DCHECK(!m_layerBridge);
     m_layerBridge = layerBridge;
   }

[tkent, 2017-04-09 23:17:28]

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/ColorSpaceProfileData.cpp
(right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/ColorSpaceProfileData.cpp:98:
DCHECK(WTF::isMainThread() && data.isEmpty());
Split this into two DCHECKs.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/ColorSpaceProfileData.cpp:104:
DCHECK(WTF::isMainThread() && data.isEmpty());
Split this into two DCHECKs.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/ContiguousContainer.cpp
(right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/ContiguousContainer.cpp:52:
CHECK(m_begin <= object && object < m_end);
Split this into two CHECKs.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.cpp
(right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.cpp:123:
DCHECK(m_canYUVDecode && m_allDataReceived);
Split this into two DCHECKs.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp
(right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp:134:
DCHECK(m_fullSize == scaledSize);
Use DCHECK_EQ if it doesn't cause a compile failure.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp:355:
DCHECK(fullSizeBitmap.width() == m_fullSize.width() &&
Split this into two DCHECK_EQs.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/Path.cpp (right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/Path.cpp:370: DCHECK(startAngle >= 0
&& startAngle < twoPiFloat);
Split this into two DCHECKs.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/Path.cpp:433: DCHECK(startAngle >= 0
&& startAngle < twoPiFloat);
Split this into two DCHECKs.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/gpu/WebGLImageConversion.cpp
(right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/gpu/WebGLImageConversion.cpp:2624:
DCHECK(params.rowLength >= 0 && params.imageHeight >= 0);
Split this into two DCHECKs.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/gpu/WebGLImageConversion.cpp:2625:
DCHECK(params.skipPixels >= 0 && params.skipRows >= 0 &&
Split this into three DCHECKs.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/gpu/WebGLImageConversion.cpp:2793:
DCHECK(skiaImage->width() && skiaImage->height());
Split this into two DCHECKs.

[Hwanseung Lee, 2017-04-11 15:50:13]

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 15:50: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 16:09:12]

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

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

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...)
  linux_layout_tests_slimming_paint_v2 on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_layout_te...)

[Hwanseung Lee, 2017-04-11 16:28:14]

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 16:28:41]

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-11 16:41:14]

Patchset #5 (id:80001) has been deleted

[Hwanseung Lee, 2017-04-11 16:41:20]

Patchset #5 (id:100001) has been deleted

[Hwanseung Lee, 2017-04-11 16:41:56]

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 16:42:29]

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 18:05:35]

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

[commit-bot: I haz the power, 2017-04-11 18:05:36]

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-11 22:24:10]

@tkent
PTAL, thank you.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/ColorSpaceProfileData.cpp
(right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/ColorSpaceProfileData.cpp:98:
DCHECK(WTF::isMainThread() && data.isEmpty());
On 2017/04/09 23:17:27, tkent wrote:
> Split this into two DCHECKs.

Done.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/ColorSpaceProfileData.cpp:104:
DCHECK(WTF::isMainThread() && data.isEmpty());
On 2017/04/09 23:17:27, tkent wrote:
> Split this into two DCHECKs.

Done.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/ContiguousContainer.cpp
(right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/ContiguousContainer.cpp:52:
CHECK(m_begin <= object && object < m_end);
On 2017/04/09 23:17:27, tkent wrote:
> Split this into two CHECKs.

Done.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.cpp
(right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.cpp:123:
DCHECK(m_canYUVDecode && m_allDataReceived);
On 2017/04/09 23:17:27, tkent wrote:
> Split this into two DCHECKs.

Done.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp
(right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp:134:
DCHECK(m_fullSize == scaledSize);
On 2017/04/09 23:17:27, tkent wrote:
> Use DCHECK_EQ if it doesn't cause a compile failure.

it was cause a compile fail.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/ImageFrameGenerator.cpp:355:
DCHECK(fullSizeBitmap.width() == m_fullSize.width() &&
On 2017/04/09 23:17:27, tkent wrote:
> Split this into two DCHECK_EQs.

Done.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/Path.cpp (right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/Path.cpp:370: DCHECK(startAngle >= 0
&& startAngle < twoPiFloat);
On 2017/04/09 23:17:27, tkent wrote:
> Split this into two DCHECKs.

Done.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/Path.cpp:433: DCHECK(startAngle >= 0
&& startAngle < twoPiFloat);
On 2017/04/09 23:17:27, tkent wrote:
> Split this into two DCHECKs.

Done.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/gpu/WebGLImageConversion.cpp
(right):

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/gpu/WebGLImageConversion.cpp:2624:
DCHECK(params.rowLength >= 0 && params.imageHeight >= 0);
On 2017/04/09 23:17:27, tkent wrote:
> Split this into two DCHECKs.

Done.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/gpu/WebGLImageConversion.cpp:2625:
DCHECK(params.skipPixels >= 0 && params.skipRows >= 0 &&
On 2017/04/09 23:17:27, tkent wrote:
> Split this into three DCHECKs.

Done.

https://codereview.chromium.org/2807923002/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/gpu/WebGLImageConversion.cpp:2793:
DCHECK(skiaImage->width() && skiaImage->height());
On 2017/04/09 23:17:27, tkent wrote:
> Split this into two DCHECKs.

Done.

[tkent, 2017-04-11 22:43:25]

https://codereview.chromium.org/2807923002/diff/120001/third_party/WebKit/Sou...
File third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp
(right):

https://codereview.chromium.org/2807923002/diff/120001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp:317:
DCHECK_GE(decoded_size.height(), 0);
Should be DCHECK_GT, not DCHCK_GE

[Hwanseung Lee, 2017-04-11 22:51:33]

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

[Hwanseung Lee, 2017-04-11 22:51:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
chrishtr@chromium.org
Link to the patchset: https://codereview.chromium.org/2807923002/#ps140001
(title: "fix typo")

[Hwanseung Lee, 2017-04-11 22:51:34]

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

[Hwanseung Lee, 2017-04-11 22:51:36]

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 22:52:18]

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

@tkent

i fixed it. thank you.

https://codereview.chromium.org/2807923002/diff/120001/third_party/WebKit/Sou...
File third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp
(right):

https://codereview.chromium.org/2807923002/diff/120001/third_party/WebKit/Sou...
third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp:317:
DCHECK_GE(decoded_size.height(), 0);
On 2017/04/11 22:43:25, tkent wrote:
> Should be DCHECK_GT, not DCHCK_GE

Done.

[tkent, 2017-04-11 23:48:29]

lgtm

[commit-bot: I haz the power, 2017-04-12 00:54:42]

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

[commit-bot: I haz the power, 2017-04-12 00:54:43]

Dry run: Try jobs failed on following builders:
  linux_layout_tests_slimming_paint_v2 on master.tryserver.chromium.linux
(JOB_TIMED_OUT, build hasn't started yet, builder probably lacks capacity)

[Hwanseung Lee, 2017-04-12 01:03: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-12 01:04: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-12 02:04:31]

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

[commit-bot: I haz the power, 2017-04-12 02:04:32]

Dry run: This issue passed the CQ dry run.

[Hwanseung Lee, 2017-04-12 02:13:18]

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

[commit-bot: I haz the power, 2017-04-12 02:14:13]

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-12 02:26:26]

CQ is committing da patch.

Bot data: {"patchset_id": 140001, "attempt_start_ts": 1491963198620400,
"parent_rev": "54f499c7bdafdc6bd9cb1cf7833c437502d49a3b", "commit_rev":
"94e9d9cf0302323949e90805222c8aa342b53641"}

[commit-bot: I haz the power, 2017-04-12 02:27:19]

Description was changed from

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

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

BUG=707658
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

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

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

BUG=707658
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2807923002
Cr-Commit-Position: refs/heads/master@{#463903}
Committed:
https://chromium.googlesource.com/chromium/src/+/94e9d9cf0302323949e90805222c...
==========

[commit-bot: I haz the power, 2017-04-12 02:27:21]

Committed patchset #6 (id:140001) as
https://chromium.googlesource.com/chromium/src/+/94e9d9cf0302323949e90805222c...