Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(25)

Unified Diff: third_party/WebKit/Source/core/layout/LayoutObject.cpp

Issue 2513893002: Make ifdefs consistent in WebKit/Source/core/ (Closed)
Patch Set: Revert small part Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 22c9acdecad7fc9b01f776650e8885a8d5bcc3b9..33bbefa7bcdf52280092eabc4920f5ad1cde6445 100644
--- a/third_party/WebKit/Source/core/layout/LayoutObject.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
@@ -103,7 +103,7 @@ const LayoutUnit& caretWidth() {
return gCaretWidth;
}
-#if ENABLE(ASSERT)
+#if DCHECK_IS_ON()
LayoutObject::SetLayoutNeededForbiddenScope::SetLayoutNeededForbiddenScope(
LayoutObject& layoutObject)
@@ -121,7 +121,7 @@ struct SameSizeAsLayoutObject : DisplayItemClient {
virtual ~SameSizeAsLayoutObject() {} // Allocate vtable pointer.
void* pointers[5];
Member<void*> members[1];
-#if ENABLE(ASSERT)
+#if DCHECK_IS_ON()
unsigned m_debugBitfields : 2;
#endif
unsigned m_bitfields;
@@ -225,7 +225,7 @@ LayoutObject::LayoutObject(Node* node)
m_parent(nullptr),
m_previous(nullptr),
m_next(nullptr),
-#if ENABLE(ASSERT)
+#if DCHECK_IS_ON()
m_hasAXObject(false),
m_setNeedsLayoutForbidden(false),
#endif
@@ -796,7 +796,7 @@ void LayoutObject::markContainerChainForLayout(bool scheduleRelayout,
last->scheduleRelayout();
}
-#if ENABLE(ASSERT)
+#if DCHECK_IS_ON()
void LayoutObject::checkBlockPositionedObjectsNeedLayout() {
ASSERT(!needsLayout());
@@ -3376,8 +3376,10 @@ void LayoutObject::setMayNeedPaintInvalidationAnimatedBackgroundImage() {
void LayoutObject::clearPaintInvalidationFlags() {
// paintInvalidationStateIsDirty should be kept in sync with the
// booleans that are cleared below.
- ASSERT(!shouldCheckForPaintInvalidationRegardlessOfPaintInvalidationState() ||
+#if DCHECK_IS_ON()
+ DCHECK(!shouldCheckForPaintInvalidationRegardlessOfPaintInvalidationState() ||
paintInvalidationStateIsDirty());
+#endif
clearShouldDoFullPaintInvalidation();
m_bitfields.setChildShouldCheckForPaintInvalidation(false);
m_bitfields.setMayNeedPaintInvalidation(false);
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutObject.h ('k') | third_party/WebKit/Source/core/svg/SVGScriptElement.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698