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

Side by Side Diff: third_party/WebKit/Source/core/css/PropertySetCSSStyleDeclaration.cpp

Issue 2755493004: Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in core/css/ (Closed)
Patch Set: All windows error are Resolved now. Created 3 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * (C) 1999-2003 Lars Knoll (knoll@kde.org) 2 * (C) 1999-2003 Lars Knoll (knoll@kde.org)
3 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All 3 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All
4 * rights reserved. 4 * rights reserved.
5 * Copyright (C) 2011 Research In Motion Limited. All rights reserved. 5 * Copyright (C) 2011 Research In Motion Limited. All rights reserved.
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 class StyleAttributeMutationScope { 55 class StyleAttributeMutationScope {
56 WTF_MAKE_NONCOPYABLE(StyleAttributeMutationScope); 56 WTF_MAKE_NONCOPYABLE(StyleAttributeMutationScope);
57 STACK_ALLOCATED(); 57 STACK_ALLOCATED();
58 58
59 public: 59 public:
60 DISABLE_CFI_PERF 60 DISABLE_CFI_PERF
61 StyleAttributeMutationScope(AbstractPropertySetCSSStyleDeclaration* decl) { 61 StyleAttributeMutationScope(AbstractPropertySetCSSStyleDeclaration* decl) {
62 ++s_scopeCount; 62 ++s_scopeCount;
63 63
64 if (s_scopeCount != 1) { 64 if (s_scopeCount != 1) {
65 ASSERT(s_currentDecl == decl); 65 DCHECK_EQ(s_currentDecl, decl);
66 return; 66 return;
67 } 67 }
68 68
69 ASSERT(!s_currentDecl); 69 DCHECK(!s_currentDecl);
70 s_currentDecl = decl; 70 s_currentDecl = decl;
71 71
72 if (!s_currentDecl->parentElement()) 72 if (!s_currentDecl->parentElement())
73 return; 73 return;
74 74
75 m_mutationRecipients = 75 m_mutationRecipients =
76 MutationObserverInterestGroup::createForAttributesMutation( 76 MutationObserverInterestGroup::createForAttributesMutation(
77 *s_currentDecl->parentElement(), HTMLNames::styleAttr); 77 *s_currentDecl->parentElement(), HTMLNames::styleAttr);
78 bool shouldReadOldValue = 78 bool shouldReadOldValue =
79 (m_mutationRecipients && m_mutationRecipients->isOldValueRequested()) || 79 (m_mutationRecipients && m_mutationRecipients->isOldValueRequested()) ||
(...skipping 339 matching lines...) Expand 10 before | Expand all | Expand 10 after
419 return m_parentElement ? m_parentElement->document().propertyRegistry() 419 return m_parentElement ? m_parentElement->document().propertyRegistry()
420 : nullptr; 420 : nullptr;
421 } 421 }
422 422
423 DEFINE_TRACE(InlineCSSStyleDeclaration) { 423 DEFINE_TRACE(InlineCSSStyleDeclaration) {
424 visitor->trace(m_parentElement); 424 visitor->trace(m_parentElement);
425 AbstractPropertySetCSSStyleDeclaration::trace(visitor); 425 AbstractPropertySetCSSStyleDeclaration::trace(visitor);
426 } 426 }
427 427
428 } // namespace blink 428 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698