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

Unified Diff: third_party/WebKit/Source/core/layout/line/LineBoxList.cpp

Issue 2748103015: Replace ASSERT with DCHECK in core/layout/<sub dirs> (Closed)
Patch Set: Rebase with latest Created 3 years, 9 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/core/layout/line/LineBoxList.cpp
diff --git a/third_party/WebKit/Source/core/layout/line/LineBoxList.cpp b/third_party/WebKit/Source/core/layout/line/LineBoxList.cpp
index 29888e5529daebad3b6c03170f6ae4e62360f505..7161032283ce08806a16e79b83965d38a8dd83e2 100644
--- a/third_party/WebKit/Source/core/layout/line/LineBoxList.cpp
+++ b/third_party/WebKit/Source/core/layout/line/LineBoxList.cpp
@@ -42,8 +42,8 @@ namespace blink {
#if DCHECK_IS_ON()
LineBoxList::~LineBoxList() {
- ASSERT(!m_firstLineBox);
- ASSERT(!m_lastLineBox);
+ DCHECK(!m_firstLineBox);
+ DCHECK(!m_lastLineBox);
}
#endif
@@ -372,10 +372,10 @@ void LineBoxList::checkConsistency() const {
const InlineFlowBox* prev = nullptr;
for (const InlineFlowBox* child = m_firstLineBox; child;
child = child->nextLineBox()) {
- ASSERT(child->prevLineBox() == prev);
+ DCHECK_EQ(child->prevLineBox(), prev);
prev = child;
}
- ASSERT(prev == m_lastLineBox);
+ DCHECK_EQ(prev, m_lastLineBox);
#endif
}

Powered by Google App Engine
This is Rietveld 408576698