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

Unified Diff: third_party/WebKit/Source/platform/text/SegmentedString.h

Issue 2394683005: Remove ASSERT_UNUSED (Closed)
Patch Set: Created 4 years, 2 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/platform/text/SegmentedString.h
diff --git a/third_party/WebKit/Source/platform/text/SegmentedString.h b/third_party/WebKit/Source/platform/text/SegmentedString.h
index 9ba74e1207f1c5ecba29d3ba32dc6b54dc920702..49f00fab74dedfefba7429f5165e6ea9da23c8b1 100644
--- a/third_party/WebKit/Source/platform/text/SegmentedString.h
+++ b/third_party/WebKit/Source/platform/text/SegmentedString.h
@@ -259,14 +259,13 @@ class PLATFORM_EXPORT SegmentedString {
}
void advanceAndASSERT(UChar expectedCharacter) {
- ASSERT_UNUSED(expectedCharacter, currentChar() == expectedCharacter);
+ DCHECK_EQ(expectedCharacter, currentChar());
advance();
}
void advanceAndASSERTIgnoringCase(UChar expectedCharacter) {
- ASSERT_UNUSED(expectedCharacter,
- WTF::Unicode::foldCase(currentChar()) ==
- WTF::Unicode::foldCase(expectedCharacter));
+ DCHECK_EQ(WTF::Unicode::foldCase(currentChar()),
+ WTF::Unicode::foldCase(expectedCharacter));
advance();
}

Powered by Google App Engine
This is Rietveld 408576698