Make ifdefs consistent in WebKit/Source/core/

third_party/WebKit/Source/core/layout/LayoutObject.h

Index: third_party/WebKit/Source/core/layout/LayoutObject.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutObject.h b/third_party/WebKit/Source/core/layout/LayoutObject.h
index c4bc1dcd7bdf96640eb76f6ad202c34f5e0d966b..a7ce63c3f0e31d9a4e470c35e76395a34c02e678 100644
--- a/third_party/WebKit/Source/core/layout/LayoutObject.h
+++ b/third_party/WebKit/Source/core/layout/LayoutObject.h
@@ -1935,7 +1935,7 @@ class CORE_EXPORT LayoutObject : public ImageResourceObserver,
     m_previousVisualRect = rect;
   }
 
-#if DCHECK_IS_ON()
+#ifndef NDEBUG

[Yuta Kitamura, 2016/11/21 05:32:06]

This is wrong (and the original code is wrong, too).

This should be "#if ENABLE(ASSERT)". This is how we say "if ASSERT() is
enabled".

[Yuta Kitamura, 2016/11/21 09:26:11]

This comment was wrong.

This should be DCHECK_IS_ON() or ENABLE(ASSERT), and all the use
sites must consistently use either DCHECK or ASSERT.

   virtual bool paintInvalidationStateIsDirty() const {
     return backgroundChangedSinceLastPaintInvalidation() ||
            shouldCheckForPaintInvalidationRegardlessOfPaintInvalidationState();