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

Unified Diff: LayoutTests/fast/dom/resources/leak-check.js

Issue 23005006: Fix XMLHttpRequest leak document when send() is called multiple times. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: check readyState == 4 to comply with change made in r156212 Created 7 years, 4 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/resources/leak-check.js
diff --git a/LayoutTests/fast/dom/resources/leak-check.js b/LayoutTests/fast/dom/resources/leak-check.js
deleted file mode 100644
index 0a74f2b7f5e8c101f2e22329099befd1607abae7..0000000000000000000000000000000000000000
--- a/LayoutTests/fast/dom/resources/leak-check.js
+++ /dev/null
@@ -1,69 +0,0 @@
-// include fast/js/resources/js-test-pre.js before this file.
-
-function doLeakTest(src, tolerance) {
- function getCounterValues() {
- testRunner.resetTestHelperControllers();
- gc();
- return {'numberOfLiveDocuments': window.internals.numberOfLiveDocuments()};
- }
-
- var frame = document.createElement('iframe');
- document.body.appendChild(frame);
- function loadSourceIntoIframe(src, callback) {
- var originalSrc = frame.src;
-
- frame.onload = function() {
- if (frame.src === originalSrc)
- return true;
-
- callback();
- return true;
- };
- frame.src = src;
- }
-
- function compareValues(countersBefore, countersAfter, tolerance) {
- for (type in tolerance) {
- var before = countersBefore[type];
- var after = countersAfter[type];
-
- if(after - before <= tolerance[type])
- testPassed('The difference of counter "'+type+'" before and after the cycle is under the threshold of '+tolerance[type]+'.');
- else
- testFailed('counter "'+type+'" was '+before+' before and now '+after+' after the cycle. This exceeds the threshold of '+tolerance[type]+'.');
- }
- }
-
- jsTestIsAsync = true;
- if (!window.internals) {
- debug("This test only runs on DumpRenderTree, as it requires existence of window.internals and cross-domain resource access check disabled.");
- finishJSTest();
- }
-
- loadSourceIntoIframe('about:blank', function() {
- // blank document loaded...
- var countersBefore = getCounterValues();
-
- loadSourceIntoIframe(src, function() {
- // target document loaded...
-
- loadSourceIntoIframe('about:blank', function() {
- // target document unloaded...
-
- var countersAfter = getCounterValues();
- compareValues(countersBefore, countersAfter, tolerance);
- finishJSTest();
- });
- });
- });
-}
-
-function htmlToUrl(html) {
- return 'data:text/html;charset=utf-8,' + html;
-}
-
-function grabScriptText(id) {
- return document.getElementById(id).innerText;
-}
-
-// include fast/js/resources/js-test-post.js after this file.

Powered by Google App Engine
This is Rietveld 408576698