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

Side by Side Diff: third_party/WebKit/PerformanceTests/resources/runner.js

Issue 2860673004: Remove unused code in runner.js (Closed)
Patch Set: Created 3 years, 7 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // There are tests for computeStatistics() located in LayoutTests/fast/harness/p erftests 1 // There are tests for computeStatistics() located in LayoutTests/fast/harness/p erftests
2 2
3 if (window.testRunner) { 3 if (window.testRunner) {
4 testRunner.waitUntilDone(); 4 testRunner.waitUntilDone();
5 testRunner.dumpAsText(); 5 testRunner.dumpAsText();
6 } 6 }
7 7
8 (function () { 8 (function () {
9 var logLines = null; 9 var logLines = null;
10 var completedIterations = -1; 10 var completedIterations = -1;
(...skipping 154 matching lines...) Expand 10 before | Expand all | Expand 10 after
165 iterationCount = test.iterationCount || (window.testRunner ? 5 : 20); 165 iterationCount = test.iterationCount || (window.testRunner ? 5 : 20);
166 if (test.warmUpCount && test.warmUpCount > 0) 166 if (test.warmUpCount && test.warmUpCount > 0)
167 completedIterations = -test.warmUpCount; 167 completedIterations = -test.warmUpCount;
168 logLines = PerfTestRunner.bufferedLog || window.testRunner ? [] : null; 168 logLines = PerfTestRunner.bufferedLog || window.testRunner ? [] : null;
169 PerfTestRunner.log("Running " + iterationCount + " times"); 169 PerfTestRunner.log("Running " + iterationCount + " times");
170 if (test.doNotIgnoreInitialRun) 170 if (test.doNotIgnoreInitialRun)
171 completedIterations++; 171 completedIterations++;
172 172
173 if (runner && test.tracingCategories && window.testRunner && 173 if (runner && test.tracingCategories && window.testRunner &&
174 window.testRunner.supportTracing) { 174 window.testRunner.supportTracing) {
175 window.testRunner.traceEventsToMeasure = test.traceEventsToMeasure;
nednguyen 2017/05/03 16:29:18 This was mistakenly left out when I switch to usin
176 window.testRunner.startTracing(test.tracingCategories, function() { 175 window.testRunner.startTracing(test.tracingCategories, function() {
177 scheduleNextRun(scheduler, runner); 176 scheduleNextRun(scheduler, runner);
178 }); 177 });
179 } else if (runner) { 178 } else if (runner) {
180 if (test.tracingCategories && !(window.testRuner && 179 if (test.tracingCategories && !(window.testRuner &&
181 window.testRunner.supportTracing)) { 180 window.testRunner.supportTracing)) {
182 PerfTestRunner.log("Tracing based metrics are specified but " + 181 PerfTestRunner.log("Tracing based metrics are specified but " +
183 "tracing is not supported on this platform. To get those " + 182 "tracing is not supported on this platform. To get those " +
184 "metrics from this test, you can run the test using " + 183 "metrics from this test, you can run the test using " +
185 "tools/perf/run_benchmarks script."); 184 "tools/perf/run_benchmarks script.");
(...skipping 241 matching lines...) Expand 10 before | Expand all | Expand 10 after
427 426
428 iframe.contentDocument.close(); 427 iframe.contentDocument.close();
429 document.body.removeChild(iframe); 428 document.body.removeChild(iframe);
430 }; 429 };
431 430
432 PerfTestRunner.measureTime(test); 431 PerfTestRunner.measureTime(test);
433 } 432 }
434 433
435 window.PerfTestRunner = PerfTestRunner; 434 window.PerfTestRunner = PerfTestRunner;
436 })(); 435 })();
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698