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

Unified Diff: LayoutTests/resources/magnitude-perf.js

Issue 211933008: Remove multiple GC calls and replace GCController.collect calls with GCController.collectAll (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: revert change to worker-event-listener Created 6 years, 9 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
« no previous file with comments | « LayoutTests/resources/gc.js ('k') | LayoutTests/storage/indexeddb/cursor-cast.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/resources/magnitude-perf.js
diff --git a/LayoutTests/resources/magnitude-perf.js b/LayoutTests/resources/magnitude-perf.js
index 70d69efd42d1bed5a0dcd58cfacb68b44c5aea9e..7d61b93862d3b67dbdb512feb4c2423b9036e65e 100644
--- a/LayoutTests/resources/magnitude-perf.js
+++ b/LayoutTests/resources/magnitude-perf.js
@@ -199,11 +199,7 @@ Magnitude._runTime = function(setup, test, magnitude)
if (GCController.getJSObjectCount)
debugStr += ' jsObjectCountBefore ' + GCController.getJSObjectCount();
- // Do a gc to reduce likelihood of gc during the test run.
- // Do multiple gc's for V8 to clear DOM wrappers.
- GCController.collect();
- GCController.collect();
- GCController.collect();
+ GCController.collectAll();
if (GCController.getJSObjectCount)
debugStr += ' jsObjectCountAfter ' + GCController.getJSObjectCount();
« no previous file with comments | « LayoutTests/resources/gc.js ('k') | LayoutTests/storage/indexeddb/cursor-cast.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698