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

Unified Diff: test/mjsunit/tools/tickprocessor.js

Issue 1318933004: [Tick processor] Add an option to the tick-processor to print the summary. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Review comments from Jakob and Michael. Created 5 years, 3 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 | « no previous file | test/mjsunit/tools/tickprocessor-test.only-summary » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/tools/tickprocessor.js
diff --git a/test/mjsunit/tools/tickprocessor.js b/test/mjsunit/tools/tickprocessor.js
index b04b9a1765b38fffa8572ee38510a9e18a56e661..4ea25f94458ccf8c5d3391aa3b7daac2a47172a2 100644
--- a/test/mjsunit/tools/tickprocessor.js
+++ b/test/mjsunit/tools/tickprocessor.js
@@ -370,7 +370,7 @@ PrintMonitor.prototype.finish = function() {
function driveTickProcessorTest(
- separateIc, ignoreUnknown, stateFilter, logInput, refOutput) {
+ separateIc, ignoreUnknown, stateFilter, logInput, refOutput, onlySummary) {
// TEST_FILE_NAME must be provided by test runner.
assertEquals('string', typeof TEST_FILE_NAME);
var pathLen = TEST_FILE_NAME.lastIndexOf('/');
@@ -387,7 +387,10 @@ function driveTickProcessorTest(
undefined,
"0",
"auto,auto",
- false);
+ false,
+ false,
+ false,
+ onlySummary);
var pm = new PrintMonitor(testsPath + refOutput);
tp.processLogFileInTest(testsPath + logInput);
tp.printStatistics();
@@ -399,19 +402,23 @@ function driveTickProcessorTest(
var testData = {
'Default': [
false, false, null,
- 'tickprocessor-test.log', 'tickprocessor-test.default'],
+ 'tickprocessor-test.log', 'tickprocessor-test.default', false],
'SeparateIc': [
true, false, null,
- 'tickprocessor-test.log', 'tickprocessor-test.separate-ic'],
+ 'tickprocessor-test.log', 'tickprocessor-test.separate-ic', false],
'IgnoreUnknown': [
false, true, null,
- 'tickprocessor-test.log', 'tickprocessor-test.ignore-unknown'],
+ 'tickprocessor-test.log', 'tickprocessor-test.ignore-unknown', false],
'GcState': [
false, false, TickProcessor.VmStates.GC,
- 'tickprocessor-test.log', 'tickprocessor-test.gc-state'],
+ 'tickprocessor-test.log', 'tickprocessor-test.gc-state', false],
'FunctionInfo': [
false, false, null,
- 'tickprocessor-test-func-info.log', 'tickprocessor-test.func-info']
+ 'tickprocessor-test-func-info.log', 'tickprocessor-test.func-info',
+ false],
+ 'OnlySummary': [
+ false, false, null,
+ 'tickprocessor-test.log', 'tickprocessor-test.only-summary', true]
};
for (var testName in testData) {
print('=== testProcessing-' + testName + ' ===');
« no previous file with comments | « no previous file | test/mjsunit/tools/tickprocessor-test.only-summary » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698