Title: [223095] branches/safari-604-branch
- Revision
- 223095
- Author
- [email protected]
- Date
- 2017-10-09 17:08:00 -0700 (Mon, 09 Oct 2017)
Log Message
Cherry-pick r220484. rdar://problem/34770810
Modified Paths
Removed Paths
Diff
Modified: branches/safari-604-branch/LayoutTests/ChangeLog (223094 => 223095)
--- branches/safari-604-branch/LayoutTests/ChangeLog 2017-10-10 00:07:57 UTC (rev 223094)
+++ branches/safari-604-branch/LayoutTests/ChangeLog 2017-10-10 00:08:00 UTC (rev 223095)
@@ -1,5 +1,22 @@
2017-10-02 Jason Marcell <[email protected]>
+ Cherry-pick r220484. rdar://problem/34770810
+
+ 2017-08-09 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r219334.
+ https://bugs.webkit.org/show_bug.cgi?id=175398
+
+ Caused bug 175023 (Requested by ap on #webkit).
+
+ Reverted changeset:
+
+ "[SVG] Leak in SVGAnimatedListPropertyTearOff"
+ https://bugs.webkit.org/show_bug.cgi?id=172545
+ http://trac.webkit.org/changeset/219334
+
+2017-10-02 Jason Marcell <[email protected]>
+
Cherry-pick r220302. rdar://problem/34770803
2017-08-04 John Wilander <[email protected]>
Deleted: branches/safari-604-branch/LayoutTests/svg/animations/animation-leak-list-property-instances-expected.txt (223094 => 223095)
--- branches/safari-604-branch/LayoutTests/svg/animations/animation-leak-list-property-instances-expected.txt 2017-10-10 00:07:57 UTC (rev 223094)
+++ branches/safari-604-branch/LayoutTests/svg/animations/animation-leak-list-property-instances-expected.txt 2017-10-10 00:08:00 UTC (rev 223095)
@@ -1,7 +0,0 @@
-This test checks that adding an animation to a SVG element does not leak the whole SVGDocument.
-
-On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
-
-
-PASS 0 is 0
-
Deleted: branches/safari-604-branch/LayoutTests/svg/animations/animation-leak-list-property-instances.html (223094 => 223095)
--- branches/safari-604-branch/LayoutTests/svg/animations/animation-leak-list-property-instances.html 2017-10-10 00:07:57 UTC (rev 223094)
+++ branches/safari-604-branch/LayoutTests/svg/animations/animation-leak-list-property-instances.html 2017-10-10 00:08:00 UTC (rev 223095)
@@ -1,63 +0,0 @@
-<!DOCTYPE html>
-<script src=""
-
-<body _onload_="test()">
- <svg id="rootSVG" width="300" height="300" xmlns="http://www.w3.org/2000/svg" version="1.1"></svg>
-</body>
-
-<script>
- description("This test checks that adding an animation to a SVG element does not leak the whole SVGDocument.")
-
- function addRect()
- {
- var elem = document.createElementNS("http://www.w3.org/2000/svg", "rect");
- elem.setAttribute("id", "rect");
- elem.setAttribute("x", 50);
- elem.setAttribute("y", 50);
- elem.setAttribute("width", 50);
- elem.setAttribute("height", 50);
- elem.setAttribute("fill", "blue");
-
- document.getElementById("rootSVG").appendChild(elem);
- }
-
- function applyTransform()
- {
- var svgroot = document.getElementById("rootSVG");
- var transformList = document.getElementById("rect").transform.baseVal;
- var rotate = svgroot.createSVGTransform();
- rotate.setRotate(15,0,0);
- transformList.appendItem(rotate);
- }
-
- function removeRect()
- {
- document.getElementById("rootSVG").removeChild(document.getElementById("rect"));
- }
-
- function test()
- {
- if (!window.internals || !window.GCController) {
- testFailed("This test requires internals and GCController");
- return;
- }
-
- testRunner.dumpAsText();
-
- // One gc() call is not enough and cause flakiness in some platforms.
- gc();
- gc();
- var originalLiveElements = internals.numberOfLiveNodes();
-
- addRect();
- applyTransform();
- removeRect();
-
- // One gc() call is not enough and cause flakiness in some platforms.
- gc();
- gc();
- var delta = internals.numberOfLiveNodes() - originalLiveElements;
- shouldBeZero(delta.toString());
- var successfullyParsed = true;
- }
-</script>
Modified: branches/safari-604-branch/Source/WebCore/ChangeLog (223094 => 223095)
--- branches/safari-604-branch/Source/WebCore/ChangeLog 2017-10-10 00:07:57 UTC (rev 223094)
+++ branches/safari-604-branch/Source/WebCore/ChangeLog 2017-10-10 00:08:00 UTC (rev 223095)
@@ -1,5 +1,22 @@
2017-10-02 Jason Marcell <[email protected]>
+ Cherry-pick r220484. rdar://problem/34770810
+
+ 2017-08-09 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r219334.
+ https://bugs.webkit.org/show_bug.cgi?id=175398
+
+ Caused bug 175023 (Requested by ap on #webkit).
+
+ Reverted changeset:
+
+ "[SVG] Leak in SVGAnimatedListPropertyTearOff"
+ https://bugs.webkit.org/show_bug.cgi?id=172545
+ http://trac.webkit.org/changeset/219334
+
+2017-10-02 Jason Marcell <[email protected]>
+
Cherry-pick r220302. rdar://problem/34770803
2017-08-04 John Wilander <[email protected]>
Modified: branches/safari-604-branch/Source/WebCore/svg/properties/SVGAnimatedListPropertyTearOff.h (223094 => 223095)
--- branches/safari-604-branch/Source/WebCore/svg/properties/SVGAnimatedListPropertyTearOff.h 2017-10-10 00:07:57 UTC (rev 223094)
+++ branches/safari-604-branch/Source/WebCore/svg/properties/SVGAnimatedListPropertyTearOff.h 2017-10-10 00:08:00 UTC (rev 223095)
@@ -73,8 +73,6 @@
m_baseVal = nullptr;
else if (&property == m_animVal)
m_animVal = nullptr;
- if (!m_baseVal && !m_animVal)
- detachListWrappers(m_values.size());
}
int findItem(SVGProperty* property)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
