Add a html-based visualizer for TurboFan graphs

Add a html-based visualizer for TurboFan graphs

Committed: https://crrev.com/9c15c055966b670058d1bfd73bdbadf4fb1ae3ae
Cr-Commit-Position: refs/heads/master@{#36351}

[danno, 2014-11-18 14:42:50]

Patchset #2 (id:20001) has been deleted

[danno, 2014-11-18 14:42:57]

Patchset #1 (id:1) has been deleted

[danno, 2014-11-18 14:44:48]

Patchset #1 (id:40001) has been deleted

[danno, 2016-05-12 10:33:03]

danno@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[danno, 2016-05-12 10:33:04]

PTAL. I almost accidentally deleted my local copy, so although I'm still pretty
ashamed of the .js code, I want to get it somewhere permanent.

[dingding, 2016-05-12 17:19:06]

On 2016/05/12 10:33:04, danno wrote:
> PTAL. I almost accidentally deleted my local copy, so although I'm still
pretty
> ashamed of the .js code, I want to get it somewhere permanent.

Danno, the file referred in index.html
(https://google-code-prettify.googlecode.com/svn/loader/run_prettify.js) no
longer exists. You will need to find another source or users have to get their
copy.

[danno, 2016-05-12 20:47:34]

On 2016/05/12 17:19:06, dingding wrote:
> On 2016/05/12 10:33:04, danno wrote:
> > PTAL. I almost accidentally deleted my local copy, so although I'm still
> pretty
> > ashamed of the .js code, I want to get it somewhere permanent.
> 
> Danno, the file referred in index.html
> (https://google-code-prettify.googlecode.com/svn/loader/run_prettify.js) no
> longer exists. You will need to find another source or users have to get their
> copy.

Thanks for the reminder. I uploaded a the newest version that cleans up a bunch
of the code as well as fixing the run_prettify.js URL.

[danno, 2016-05-12 20:47:59]

danno@chromium.org changed reviewers:
+ bmeurer@chromium.org

[Michael Starzinger, 2016-05-13 08:14:18]

LGTM (rubber-stamped).

https://codereview.chromium.org/729913004/diff/580001/tools/turbolizer/README
File tools/turbolizer/README (right):

https://codereview.chromium.org/729913004/diff/580001/tools/turbolizer/README...
tools/turbolizer/README:1: Graph visualization and manipulation based on Mike
Bostock's sample code for an
nit: I think we should add one paragraph at the top of this file which briefly
describes the purpose of the "turbolizer" and mentions that graph traces can be
created by running V8 with the --trace-turbo flag.

https://codereview.chromium.org/729913004/diff/580001/tools/turbolizer/README...
tools/turbolizer/README:9: 
nit: Let's drop the whitespace at the end of this file.

[danno, 2016-05-19 07:46:50]

Addressed feedback, landing.

[danno, 2016-05-19 07:47:02]

The CQ bit was checked by danno@chromium.org

[danno, 2016-05-19 07:47:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/729913004/#ps600001
(title: "Review feedback")

[commit-bot: I haz the power, 2016-05-19 07:47:07]

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

[commit-bot: I haz the power, 2016-05-19 08:17:35]

Committed patchset #28 (id:600001)

[commit-bot: I haz the power, 2016-05-19 08:18:10]

Description was changed from

==========
Add a html-based visualizer for TurboFan graphs
==========

to

==========
Add a html-based visualizer for TurboFan graphs

Committed: https://crrev.com/9c15c055966b670058d1bfd73bdbadf4fb1ae3ae
Cr-Commit-Position: refs/heads/master@{#36351}
==========

[commit-bot: I haz the power, 2016-05-19 08:18:12]

Patchset 28 (id:??) landed as
https://crrev.com/9c15c055966b670058d1bfd73bdbadf4fb1ae3ae
Cr-Commit-Position: refs/heads/master@{#36351}