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

Side by Side Diff: Source/core/svg/SVGDocumentExtensions.cpp

Issue 327323002: Start removing the double-negative !ASSERT_DISABLED (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 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 | Annotate | Revision Log
« no previous file with comments | « Source/core/svg/SVGDocumentExtensions.h ('k') | Source/core/svg/SVGElement.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006 Apple Inc. All rights reserved. 2 * Copyright (C) 2006 Apple Inc. All rights reserved.
3 * Copyright (C) 2006 Nikolas Zimmermann <zimmermann@kde.org> 3 * Copyright (C) 2006 Nikolas Zimmermann <zimmermann@kde.org>
4 * Copyright (C) 2007 Rob Buis <buis@kde.org> 4 * Copyright (C) 2007 Rob Buis <buis@kde.org>
5 * 5 *
6 * This library is free software; you can redistribute it and/or 6 * This library is free software; you can redistribute it and/or
7 * modify it under the terms of the GNU Library General Public 7 * modify it under the terms of the GNU Library General Public
8 * License as published by the Free Software Foundation; either 8 * License as published by the Free Software Foundation; either
9 * version 2 of the License, or (at your option) any later version. 9 * version 2 of the License, or (at your option) any later version.
10 * 10 *
(...skipping 21 matching lines...) Expand all
32 #include "core/svg/SVGZoomAndPan.h" 32 #include "core/svg/SVGZoomAndPan.h"
33 #include "core/svg/animation/SMILTimeContainer.h" 33 #include "core/svg/animation/SMILTimeContainer.h"
34 #include "wtf/TemporaryChange.h" 34 #include "wtf/TemporaryChange.h"
35 #include "wtf/text/AtomicString.h" 35 #include "wtf/text/AtomicString.h"
36 36
37 namespace WebCore { 37 namespace WebCore {
38 38
39 SVGDocumentExtensions::SVGDocumentExtensions(Document* document) 39 SVGDocumentExtensions::SVGDocumentExtensions(Document* document)
40 : m_document(document) 40 : m_document(document)
41 , m_resourcesCache(adoptPtr(new SVGResourcesCache)) 41 , m_resourcesCache(adoptPtr(new SVGResourcesCache))
42 #if !ASSERT_DISABLED 42 #if ASSERT_ENABLED
43 , m_inRelativeLengthSVGRootsInvalidation(false) 43 , m_inRelativeLengthSVGRootsInvalidation(false)
44 #endif 44 #endif
45 { 45 {
46 } 46 }
47 47
48 SVGDocumentExtensions::~SVGDocumentExtensions() 48 SVGDocumentExtensions::~SVGDocumentExtensions()
49 { 49 {
50 } 50 }
51 51
52 void SVGDocumentExtensions::addTimeContainer(SVGSVGElement* element) 52 void SVGDocumentExtensions::addTimeContainer(SVGSVGElement* element)
(...skipping 345 matching lines...) Expand 10 before | Expand all | Expand 10 after
398 } 398 }
399 399
400 bool SVGDocumentExtensions::isSVGRootWithRelativeLengthDescendents(SVGSVGElement * svgRoot) const 400 bool SVGDocumentExtensions::isSVGRootWithRelativeLengthDescendents(SVGSVGElement * svgRoot) const
401 { 401 {
402 return m_relativeLengthSVGRoots.contains(svgRoot); 402 return m_relativeLengthSVGRoots.contains(svgRoot);
403 } 403 }
404 404
405 void SVGDocumentExtensions::invalidateSVGRootsWithRelativeLengthDescendents(Subt reeLayoutScope* scope) 405 void SVGDocumentExtensions::invalidateSVGRootsWithRelativeLengthDescendents(Subt reeLayoutScope* scope)
406 { 406 {
407 ASSERT(!m_inRelativeLengthSVGRootsInvalidation); 407 ASSERT(!m_inRelativeLengthSVGRootsInvalidation);
408 #if !ASSERT_DISABLED 408 #if ASSERT_ENABLED
409 TemporaryChange<bool> inRelativeLengthSVGRootsChange(m_inRelativeLengthSVGRo otsInvalidation, true); 409 TemporaryChange<bool> inRelativeLengthSVGRootsChange(m_inRelativeLengthSVGRo otsInvalidation, true);
410 #endif 410 #endif
411 411
412 WillBeHeapHashSet<RawPtrWillBeMember<SVGSVGElement> >::iterator end = m_rela tiveLengthSVGRoots.end(); 412 WillBeHeapHashSet<RawPtrWillBeMember<SVGSVGElement> >::iterator end = m_rela tiveLengthSVGRoots.end();
413 for (WillBeHeapHashSet<RawPtrWillBeMember<SVGSVGElement> >::iterator it = m_ relativeLengthSVGRoots.begin(); it != end; ++it) 413 for (WillBeHeapHashSet<RawPtrWillBeMember<SVGSVGElement> >::iterator it = m_ relativeLengthSVGRoots.begin(); it != end; ++it)
414 (*it)->invalidateRelativeLengthClients(scope); 414 (*it)->invalidateRelativeLengthClients(scope);
415 } 415 }
416 416
417 #if ENABLE(SVG_FONTS) 417 #if ENABLE(SVG_FONTS)
418 void SVGDocumentExtensions::registerSVGFontFaceElement(SVGFontFaceElement* eleme nt) 418 void SVGDocumentExtensions::registerSVGFontFaceElement(SVGFontFaceElement* eleme nt)
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
480 { 480 {
481 visitor->trace(m_document); 481 visitor->trace(m_document);
482 visitor->trace(m_timeContainers); 482 visitor->trace(m_timeContainers);
483 visitor->trace(m_svgFontFaceElements); 483 visitor->trace(m_svgFontFaceElements);
484 visitor->trace(m_pendingSVGFontFaceElementsForRemoval); 484 visitor->trace(m_pendingSVGFontFaceElementsForRemoval);
485 visitor->trace(m_elementDependencies); 485 visitor->trace(m_elementDependencies);
486 visitor->trace(m_relativeLengthSVGRoots); 486 visitor->trace(m_relativeLengthSVGRoots);
487 } 487 }
488 488
489 } 489 }
OLDNEW
« no previous file with comments | « Source/core/svg/SVGDocumentExtensions.h ('k') | Source/core/svg/SVGElement.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698