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

Unified Diff: third_party/WebKit/PerformanceTests/BlinkGC/resources/measure-gc.js

Issue 2858783003: Refactor prepareToMeasureValuesAsync to startMeasureValuesAsyn which run test through callback (Closed)
Patch Set: Address Xianzhu's review comments 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/PerformanceTests/BlinkGC/resources/measure-gc.js
diff --git a/third_party/WebKit/PerformanceTests/BlinkGC/resources/measure-gc.js b/third_party/WebKit/PerformanceTests/BlinkGC/resources/measure-gc.js
index 60d2d2eb1b0a23f4525daab4db49bbed6d864a16..979e4e66e5ec728e9345038a11796167c3a1a47e 100644
--- a/third_party/WebKit/PerformanceTests/BlinkGC/resources/measure-gc.js
+++ b/third_party/WebKit/PerformanceTests/BlinkGC/resources/measure-gc.js
@@ -14,9 +14,11 @@ if (!window.GCController && !window.gc)
test.unit = 'ms';
if (!test.warmUpCount)
test.warmUpCount = warmUpCount;
+ if (!test.run)
+ test.run = function() {};
completedIterations = 0;
- PerfTestRunner.prepareToMeasureValuesAsync(test);
+ PerfTestRunner.startMeasureValuesAsync(test);
// Force a V8 GC before running Blink GC test to avoid measuring marking from stale V8 wrappers.
if (window.GCController)

Powered by Google App Engine
This is Rietveld 408576698