Chromium Code Reviews| Index: Source/core/svg/SVGSVGElement.cpp |
| diff --git a/Source/core/svg/SVGSVGElement.cpp b/Source/core/svg/SVGSVGElement.cpp |
| index 0de4dbd7f4c1f51af322eeb568ff88b43cefb261..f8ec63489d0e00963964c9294ffb69cbc55cd33b 100644 |
| --- a/Source/core/svg/SVGSVGElement.cpp |
| +++ b/Source/core/svg/SVGSVGElement.cpp |
| @@ -96,11 +96,15 @@ SVGSVGElement::~SVGSVGElement() |
| if (m_viewSpec) |
| m_viewSpec->detachContextElement(); |
| +#if !ENABLE(OILPAN) |
| // There are cases where removedFromDocument() is not called. |
|
Erik Corry
2014/04/25 09:23:48
Insert "Pre-oilpan " at the start of this comment.
Mads Ager (chromium)
2014/04/25 10:58:25
I don't think we need to when it is right after a
|
| // see ContainerNode::removeAllChildren, called by its destructor. |
| + // With Oilpan, either removedFrom is called or the document |
| + // is dead as well and there is no reason to clear the extensions. |
| document().accessSVGExtensions().removeTimeContainer(this); |
| ASSERT(inDocument() || !accessDocumentSVGExtensions().isSVGRootWithRelativeLengthDescendents(this)); |
| +#endif |
| } |
| PassRefPtr<SVGRectTearOff> SVGSVGElement::viewport() const |