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

Unified Diff: third_party/WebKit/Source/core/html/parser/HTMLEntitySearch.cpp

Issue 2751483005: Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in core/html/parser/ (Closed)
Patch Set: rebase 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/html/parser/HTMLEntitySearch.cpp
diff --git a/third_party/WebKit/Source/core/html/parser/HTMLEntitySearch.cpp b/third_party/WebKit/Source/core/html/parser/HTMLEntitySearch.cpp
index 6c31192a30cd4f57223cce0e29cfbb9f03962ef0..bfbbbd7a167df2f58cb97d462e06b726e942c71f 100644
--- a/third_party/WebKit/Source/core/html/parser/HTMLEntitySearch.cpp
+++ b/third_party/WebKit/Source/core/html/parser/HTMLEntitySearch.cpp
@@ -69,11 +69,11 @@ const HTMLEntityTableEntry* HTMLEntitySearch::findFirst(
if (result == Before)
left = probe;
else {
- ASSERT(result == After || result == Prefix);
+ DCHECK(result == After || result == Prefix);
right = probe;
}
}
- ASSERT(left + 1 == right);
+ DCHECK_EQ(left + 1, right);
return right;
}
@@ -94,16 +94,16 @@ const HTMLEntityTableEntry* HTMLEntitySearch::findLast(
if (result == After)
right = probe;
else {
- ASSERT(result == Before || result == Prefix);
+ DCHECK(result == Before || result == Prefix);
left = probe;
}
}
- ASSERT(left + 1 == right);
+ DCHECK_EQ(left + 1, right);
return left;
}
void HTMLEntitySearch::advance(UChar nextCharacter) {
- ASSERT(isEntityPrefix());
+ DCHECK(isEntityPrefix());
if (!m_currentLength) {
m_first = HTMLEntityTable::firstEntryStartingWith(nextCharacter);
m_last = HTMLEntityTable::lastEntryStartingWith(nextCharacter);

Powered by Google App Engine
This is Rietveld 408576698