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

Unified Diff: third_party/WebKit/Source/core/html/parser/HTMLTreeBuilder.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/core/html/parser/HTMLTreeBuilder.cpp
diff --git a/third_party/WebKit/Source/core/html/parser/HTMLTreeBuilder.cpp b/third_party/WebKit/Source/core/html/parser/HTMLTreeBuilder.cpp
index b7cec90696711cadc138a282e20bb86f55d9ec27..ce80c79feda1f1ed9968d322301f756b4aca6471 100644
--- a/third_party/WebKit/Source/core/html/parser/HTMLTreeBuilder.cpp
+++ b/third_party/WebKit/Source/core/html/parser/HTMLTreeBuilder.cpp
@@ -220,9 +220,6 @@ HTMLTreeBuilder::HTMLTreeBuilder(HTMLDocumentParser* parser,
ParserContentPolicy parserContentPolicy,
const HTMLParserOptions& options)
: m_framesetOk(true),
-#if ENABLE(ASSERT)
- m_isAttached(true),
-#endif
m_tree(parser->reentryPermit(), document, parserContentPolicy),
m_insertionMode(InitialMode),
m_originalInsertionMode(InitialMode),
@@ -283,7 +280,7 @@ DEFINE_TRACE(HTMLTreeBuilder) {
}
void HTMLTreeBuilder::detach() {
-#if ENABLE(ASSERT)
+#if DCHECK_IS_ON()
// This call makes little sense in fragment mode, but for consistency
// DocumentParser expects detach() to always be called before it's destroyed.
m_isAttached = false;

Powered by Google App Engine
This is Rietveld 408576698