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

Unified Diff: third_party/WebKit/Source/core/dom/StyleElement.cpp

Issue 1854423002: ASSERT -> {DCHECK|DCHECK_XX}, ENABLE(ASSERT) -> DCHECK_IS_ON() in dom (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: mark svg/as-image/svg-nested.html crash on win Created 4 years, 8 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/dom/StyleElement.cpp
diff --git a/third_party/WebKit/Source/core/dom/StyleElement.cpp b/third_party/WebKit/Source/core/dom/StyleElement.cpp
index b96cea5665cd39ae36e4f6143ce9c8ebdd476a16..4cc9498e0e9fd3c83ad41ca53e995c3c64443010 100644
--- a/third_party/WebKit/Source/core/dom/StyleElement.cpp
+++ b/third_party/WebKit/Source/core/dom/StyleElement.cpp
@@ -64,8 +64,8 @@ StyleElement::~StyleElement()
StyleElement::ProcessingResult StyleElement::processStyleSheet(Document& document, Element* element)
{
TRACE_EVENT0("blink", "StyleElement::processStyleSheet");
- ASSERT(element);
- ASSERT(element->inShadowIncludingDocument());
+ DCHECK(element);
+ DCHECK(element->inShadowIncludingDocument());
m_registeredAsCandidate = true;
document.styleEngine().addStyleSheetCandidateNode(element);
@@ -115,7 +115,7 @@ void StyleElement::clearDocumentData(Document& document, Element* element)
m_sheet->clearOwnerNode();
if (m_registeredAsCandidate) {
- ASSERT(element->inShadowIncludingDocument());
+ DCHECK(element->inShadowIncludingDocument());
document.styleEngine().removeStyleSheetCandidateNode(element, element->treeScope());
m_registeredAsCandidate = false;
}
@@ -123,7 +123,7 @@ void StyleElement::clearDocumentData(Document& document, Element* element)
StyleElement::ProcessingResult StyleElement::childrenChanged(Element* element)
{
- ASSERT(element);
+ DCHECK(element);
if (m_createdByParser)
return ProcessingSuccessful;
@@ -132,7 +132,7 @@ StyleElement::ProcessingResult StyleElement::childrenChanged(Element* element)
StyleElement::ProcessingResult StyleElement::finishParsingChildren(Element* element)
{
- ASSERT(element);
+ DCHECK(element);
ProcessingResult result = process(element);
m_createdByParser = false;
return result;
@@ -147,7 +147,7 @@ StyleElement::ProcessingResult StyleElement::process(Element* element)
void StyleElement::clearSheet(Element* ownerElement)
{
- ASSERT(m_sheet);
+ DCHECK(m_sheet);
if (ownerElement && m_sheet->isLoading())
ownerElement->document().styleEngine().removePendingSheet(ownerElement);
@@ -172,8 +172,8 @@ static bool shouldBypassMainWorldCSP(Element* element)
StyleElement::ProcessingResult StyleElement::createSheet(Element* e, const String& text)
{
- ASSERT(e);
- ASSERT(e->inShadowIncludingDocument());
+ DCHECK(e);
+ DCHECK(e->inShadowIncludingDocument());
Document& document = e->document();
const ContentSecurityPolicy* csp = document.contentSecurityPolicy();
« no previous file with comments | « third_party/WebKit/Source/core/dom/StyleChangeReason.cpp ('k') | third_party/WebKit/Source/core/dom/StyleEngine.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698