[tools] Add callstats.html to analize --runtime-call-stats output

[tools] Add callstats.html to analize --runtime-call-stats output

callstats.html in combination with callstats.py can be used to get an in-depth
view on where we spend time in v8. Furthermore, with callstats.html we can
compare different versions of v8/chrome and see where and how performnce
evolved.

Drive-by-fix: add missing copyrights header to ic-explorer.html and format
ic-explorer.html

BUG=

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

[Camillo Bruni, 2016-05-06 11:18:59]

Description was changed from

==========
[tools] Add callstats.html to analize --runtime-call-stats output

callstats.html in combination with callstats.py can be used to get an in-depth
view on where we spend time in v8. Furthermore, with callstats.html we can
compare different versions of v8/chrome and see where and how performnce
evolved.

BUG=
==========

to

==========
[tools] Add callstats.html to analize --runtime-call-stats output

callstats.html in combination with callstats.py can be used to get an in-depth
view on where we spend time in v8. Furthermore, with callstats.html we can
compare different versions of v8/chrome and see where and how performnce
evolved.

Drive-by-fix: add missing copyrights header to ic-explorer.html

BUG=
==========

[Camillo Bruni, 2016-05-06 11:19:16]

cbruni@chromium.org changed reviewers:
+ nikolaos@chromium.org

[Camillo Bruni, 2016-05-06 11:19:16]

PTAL

[nickie, 2016-05-06 12:40:49]

LGTM, other than my non-intelligent comments (mostly about formatting).  Also,
the formatter complains about trailing whitespace and a missing newline at the
end of ic-explorer.html.

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html
File tools/callstats.html (right):

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:316: // Filter out entries that do not exist in the first
column for the default
Comment line too long.  Move "default" to next line?

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:331: // Don't show entries that do not exist on the current
page or if we compare
Same.  Move "compare" to next line?

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:355: } else if (baselineVersion !== undefined &&
referenceEntry && page.version !==
Bad formatting.  Break after last "&&"?

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:511: impactView.querySelector('h3 span').innerHTML += " " +
selectedEntry.name;
Bad formatting.  Break after "+="?

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:513: return entry.name == selectedEntry.name ||
"(entry) => { return ... }" can be written as "entry => ..."

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:528: return each.name + ' (' +
percent(each.getEntry(entry).timePercent) +
Bad formatting.  You can skip the braces and the "return" in the arrow function.
 You can also skip the parentheses around "each", but that's elsewhere too.

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:570: popover.querySelector(prefix + '.percent').innerHTML =
percent(entry.timePercent);
Bad formatting.  Break after "="?

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:576: popover.querySelector(prefix +
'.timeImpact').innerHTML = ms(entry.getTimeImpact());
Bad formatting.  Break after "="?

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:581: //
===========================================================================
Bad formatting.  Fewer "="?

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:656: //
===========================================================================
Bad formatting.  Fewer "="?  Also, insert one blank line before this.

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:737: //
===========================================================================
Bad formatting.  Fewer "="?

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:903: // Iterate over all the entries in the order they
appear on the reference page.
Bad formatting.  Move two words to next line?

https://codereview.chromium.org/1947413004/diff/1/tools/callstats.html#newcod...
tools/callstats.html:1031: // If we have a baslineVersion to compare against
show also all entries from the
Bad formatting.  Move two words to next line?

[Camillo Bruni, 2016-05-06 12:42:23]

Description was changed from

==========
[tools] Add callstats.html to analize --runtime-call-stats output

callstats.html in combination with callstats.py can be used to get an in-depth
view on where we spend time in v8. Furthermore, with callstats.html we can
compare different versions of v8/chrome and see where and how performnce
evolved.

Drive-by-fix: add missing copyrights header to ic-explorer.html

BUG=
==========

to

==========
[tools] Add callstats.html to analize --runtime-call-stats output

callstats.html in combination with callstats.py can be used to get an in-depth
view on where we spend time in v8. Furthermore, with callstats.html we can
compare different versions of v8/chrome and see where and how performnce
evolved.

Drive-by-fix: add missing copyrights header to ic-explorer.html and format
ic-explorer.html

BUG=
==========

[Camillo Bruni, 2016-05-06 12:49:48]

The CQ bit was checked by cbruni@chromium.org

[Camillo Bruni, 2016-05-06 12:49:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
nikolaos@chromium.org
Link to the patchset: https://codereview.chromium.org/1947413004/#ps40001
(title: "addressing nits")

[commit-bot: I haz the power, 2016-05-06 12:49:55]

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

[commit-bot: I haz the power, 2016-05-06 13:23:56]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-06 13:24:45]

Description was changed from

==========
[tools] Add callstats.html to analize --runtime-call-stats output

callstats.html in combination with callstats.py can be used to get an in-depth
view on where we spend time in v8. Furthermore, with callstats.html we can
compare different versions of v8/chrome and see where and how performnce
evolved.

Drive-by-fix: add missing copyrights header to ic-explorer.html and format
ic-explorer.html

BUG=
==========

to

==========
[tools] Add callstats.html to analize --runtime-call-stats output

callstats.html in combination with callstats.py can be used to get an in-depth
view on where we spend time in v8. Furthermore, with callstats.html we can
compare different versions of v8/chrome and see where and how performnce
evolved.

Drive-by-fix: add missing copyrights header to ic-explorer.html and format
ic-explorer.html

BUG=

Committed: https://crrev.com/aa1b7f6b816d95aa381dcdab0ee15ddf956b5a8f
Cr-Commit-Position: refs/heads/master@{#36080}
==========

[commit-bot: I haz the power, 2016-05-06 13:24:46]

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