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

Unified Diff: third_party/WebKit/LayoutTests/fast/dom/Window/timer-null-script-execution-context.html

Issue 2847943002: Cleanup LayoutTests that define a function gc(). (Closed)
Patch Set: Fixing Layout Tests Failures Created 3 years, 8 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: third_party/WebKit/LayoutTests/fast/dom/Window/timer-null-script-execution-context.html
diff --git a/third_party/WebKit/LayoutTests/fast/dom/Window/timer-null-script-execution-context.html b/third_party/WebKit/LayoutTests/fast/dom/Window/timer-null-script-execution-context.html
index aa4f66cffa12e7e6730374e8b49f7e235f22e88b..adee002163b490073a50e716bc509a632be31900 100644
--- a/third_party/WebKit/LayoutTests/fast/dom/Window/timer-null-script-execution-context.html
+++ b/third_party/WebKit/LayoutTests/fast/dom/Window/timer-null-script-execution-context.html
@@ -2,24 +2,13 @@
<p>Test for <a href="https://bugs.webkit.org/show_bug.cgi?id=33815">bug 33815</a>: Crash when using DOMTimer from a detached frame.</p>
<p>Pass if no crash.</p>
<iframe src="about:blank"></iframe>
-
+<script src="../../../resources/gc.js"></script>
<script>
if (window.testRunner) {
testRunner.dumpAsText();
testRunner.waitUntilDone();
}
-function gc()
-{
- if (window.GCController)
- return GCController.collect();
-
- for (var i = 0; i < 10000; i++) { // > force garbage collection (FF requires about 9K allocations before a collect)
- var s = new String("");
- }
-}
-
-
function test()
{
var w = frames[0];

Powered by Google App Engine
This is Rietveld 408576698