[Tick processor] Add an option to the tick-processor to print the summary.

[Tick processor] Add an option to the tick-processor to print the summary.
 - Print the summary excluding other tick information
 - Add test to verify that summary is printed correctly.

BUG=None
LOG=N

R=machenbach@chromium.org

Committed: https://crrev.com/e1f38de76a758f8640685f5b3afc0d153f0abd86
Cr-Commit-Position: refs/heads/master@{#30573}

[Michael Achenbach, 2015-09-03 08:19:11]

machenbach@chromium.org changed reviewers:
+ jkummerow@chromium.org

[Michael Achenbach, 2015-09-03 08:19:11]

machenbach@chromium.org changed required reviewers:
+ jkummerow@chromium.org

[Michael Achenbach, 2015-09-03 08:19:11]

Tentative lgtm. Adding more accurate owner.

https://codereview.chromium.org/1318933004/diff/1/tools/tickprocessor.js
File tools/tickprocessor.js (right):

https://codereview.chromium.org/1318933004/diff/1/tools/tickprocessor.js#newc...
tools/tickprocessor.js:517: }
nit: indentation of } and };

[Jakob Kummerow, 2015-09-03 08:37:08]

LGTM with comments.

https://codereview.chromium.org/1318933004/diff/1/tools/tickprocessor.js
File tools/tickprocessor.js (right):

https://codereview.chromium.org/1318933004/diff/1/tools/tickprocessor.js#newc...
tools/tickprocessor.js:461: var printAllTicks = this.printSummary_ == undefined
||
How about simply:

var printAllTicks = !this.onlySummary;

https://codereview.chromium.org/1318933004/diff/1/tools/tickprocessor.js#newc...
tools/tickprocessor.js:896: '--print-summary': ['printSummary', true,
I'd call the flag --only-summary (and the corresponding variables "onlySummary")
to make its effect more obvious.

[gdeepti1, 2015-09-03 17:50:21]

https://codereview.chromium.org/1318933004/diff/1/tools/tickprocessor.js
File tools/tickprocessor.js (right):

https://codereview.chromium.org/1318933004/diff/1/tools/tickprocessor.js#newc...
tools/tickprocessor.js:461: var printAllTicks = this.printSummary_ == undefined
||
On 2015/09/03 08:37:08, Jakob wrote:
> How about simply:
> 
> var printAllTicks = !this.onlySummary;

Done.

https://codereview.chromium.org/1318933004/diff/1/tools/tickprocessor.js#newc...
tools/tickprocessor.js:517: }
On 2015/09/03 08:19:11, Michael Achenbach wrote:
> nit: indentation of } and };

Done.

https://codereview.chromium.org/1318933004/diff/1/tools/tickprocessor.js#newc...
tools/tickprocessor.js:896: '--print-summary': ['printSummary', true,
On 2015/09/03 08:37:08, Jakob wrote:
> I'd call the flag --only-summary (and the corresponding variables
"onlySummary")
> to make its effect more obvious.

Done.

[gdeepti1, 2015-09-03 18:00:14]

The CQ bit was checked by gdeepti@google.com

[gdeepti1, 2015-09-03 18:00:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
machenbach@chromium.org, jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/1318933004/#ps20001
(title: "Review comments from Jakob and Michael.")

[commit-bot: I haz the power, 2015-09-03 18:00:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1318933004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1318933004/20001

[commit-bot: I haz the power, 2015-09-03 18:01:47]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-03 18:01:54]

Patchset 2 (id:??) landed as
https://crrev.com/e1f38de76a758f8640685f5b3afc0d153f0abd86
Cr-Commit-Position: refs/heads/master@{#30573}