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

Unified Diff: third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp

Issue 2640163004: Replace ENABLE(ASSERT) with DCHECK_IS_ON(). (Closed)
Patch Set: Created 3 years, 11 months 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/modules/accessibility/AXNodeObject.cpp
diff --git a/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp b/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp
index 58b6cd37e5ce8175a7e970004de4385f78d8dd0f..8f25c79a5582cb1bf7edc33d44cdfc7c631a80bf 100644
--- a/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp
+++ b/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp
@@ -74,9 +74,6 @@ AXNodeObject::AXNodeObject(Node* node, AXObjectCacheImpl& axObjectCache)
: AXObject(axObjectCache),
m_ariaRole(UnknownRole),
m_childrenDirty(false),
-#if ENABLE(ASSERT)
- m_initialized(false),
-#endif
m_node(node) {
}
@@ -124,7 +121,7 @@ AXObject* AXNodeObject::activeDescendant() {
bool AXNodeObject::computeAccessibilityIsIgnored(
IgnoredReasons* ignoredReasons) const {
-#if ENABLE(ASSERT)
+#if DCHECK_IS_ON()
// Double-check that an AXObject is never accessed before
// it's been initialized.
ASSERT(m_initialized);
@@ -741,7 +738,7 @@ AccessibilityRole AXNodeObject::remapAriaRoleDueToParent(
}
void AXNodeObject::init() {
-#if ENABLE(ASSERT)
+#if DCHECK_IS_ON()
ASSERT(!m_initialized);
m_initialized = true;
#endif

Powered by Google App Engine
This is Rietveld 408576698