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

Unified Diff: LayoutTests/fast/dom/TreeWalker/TreeWalker-leak-document.html

Issue 21274004: Fix Document leak from NodeFilter. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: don't print numberOfLiveDocuments when test passed Created 7 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: LayoutTests/fast/dom/TreeWalker/TreeWalker-leak-document.html
diff --git a/LayoutTests/fast/dom/TreeWalker/TreeWalker-leak-document.html b/LayoutTests/fast/dom/TreeWalker/TreeWalker-leak-document.html
new file mode 100644
index 0000000000000000000000000000000000000000..f61f4e7aba69a2b2e0e0d76466161c9fdbc0f333
--- /dev/null
+++ b/LayoutTests/fast/dom/TreeWalker/TreeWalker-leak-document.html
@@ -0,0 +1,18 @@
+<html>
+<body>
+ <script id='targetJS' type='text/html'>
+ // This test asserts that document doesn't leak when a NodeFilter callback referencing the document is created.
+ keepTreeWalker = document.createTreeWalker(document, NodeFilter.SHOW_ELEMENT, function(node) { return NodeFilter.SHOW_ELEMENT; }, false);
+ </script>
+
+ <script src="../../js/resources/js-test-pre.js"></script>
+ <script src="../../dom/resources/leak-check.js"></script>
+ <script>
+ var target = '<script>'+grabScriptText('targetJS')+'<'+'/script>';
+ var tolerance = {'numberOfLiveDocuments': 0};
+
+ doLeakTest(htmlToUrl(target), tolerance);
+ </script>
+ <script src="../../js/resources/js-test-post.js"></script>
+</body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698