Make ifdefs consistent in WebKit/Source/core/

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

Index: third_party/WebKit/Source/core/layout/LayoutObject.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutObject.cpp b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
index c34902ecec7e5da81b2824dc66d53d77890c1574..f603d96452c7cefd0063463fce7f267283797672 100644
--- a/third_party/WebKit/Source/core/layout/LayoutObject.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
@@ -3450,8 +3450,10 @@ void LayoutObject::setMayNeedPaintInvalidationAnimatedBackgroundImage() {
 void LayoutObject::clearPaintInvalidationFlags() {
   // paintInvalidationStateIsDirty should be kept in sync with the
   // booleans that are cleared below.
+#ifndef NDEBUG

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

This is wrong. This shouldn't be necessary.

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

NDEBUG does not make sense here. Use DCHECK_IS_ON() or ENABLE(ASSERT) as
mentioned earlier.

[Alexander Alekseev, 2016/11/23 09:20:30]

Done.

   ASSERT(!shouldCheckForPaintInvalidationRegardlessOfPaintInvalidationState() ||
          paintInvalidationStateIsDirty());
+#endif
   clearShouldDoFullPaintInvalidation();
   m_bitfields.setChildShouldCheckForPaintInvalidation(false);
   m_bitfields.setMayNeedPaintInvalidation(false);