Index: third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp |
diff --git a/third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp b/third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp |
index 34ebfa02ff086decd5eb31f2e106473bd8d4c85d..ab184256008bd45070cf699147831ddd04cb550c 100644 |
--- a/third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp |
+++ b/third_party/WebKit/Source/core/svg/animation/SVGSMILElement.cpp |
@@ -1222,8 +1222,7 @@ void SVGSMILElement::notifyDependentsIntervalChanged() |
ASSERT(m_interval.begin.isFinite()); |
// |loopBreaker| is used to avoid infinite recursions which may be caused from: |
// |notifyDependentsIntervalChanged| -> |createInstanceTimesFromSyncbase| -> |add{Begin,End}Time| -> |{begin,end}TimeChanged| -> |notifyDependentsIntervalChanged| |
- // |loopBreaker| is defined as a Persistent<HeapHashSet<Member<SVGSMILElement>>>. This won't cause leaks because it is guaranteed to be empty after the root |notifyDependentsIntervalChanged| has exited. |
- DEFINE_STATIC_LOCAL(HeapHashSet<Member<SVGSMILElement>>, loopBreaker, (new HeapHashSet<Member<SVGSMILElement>>)); |
+ DEFINE_STATIC_LOCAL(HashSet<UntracedMember<SVGSMILElement>>, loopBreaker, ()); |
haraken
2016/05/11 11:53:23
Maybe worth adding a comment why the untraced memb
sof
2016/05/11 12:11:20
yes, done.
|
if (!loopBreaker.add(this).isNewEntry) |
return; |