Add a standalone version of the test runtime tree map viewer, that can accept stats.json from a local test run

[Simon Fraser (smfr)]

Add a standalone version of the test runtime tree map viewer, that can accept stats.json from a local test run

[Simon Fraser (smfr)]

Created attachment 349111 [details]
Patch

[Simon Fraser (smfr)]

This should probably live in some other directory. Maybe it should be bundled in layout_test_results and linked from results.html.

[Alexey Proskuryakov]

Comment on attachment 349111 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=349111&action=review

> Tools/ChangeLog:12
> +        This should probably live in some other directory. Maybe it should be bundled in layout_test_results
> +        and linked from results.html.

How about somewhere on build.webkit.org, and having a link to it from "Tests and Performance" section of the landing page?

[Simon Fraser (smfr)]

I want someone to be able to see the tree map for their local results, so the page would have to be able to load a local stats.json (and probably the results.json in future). A single click from results.html would be ideal, rather than forcing someone to drag and drop the json file.

[Alexey Proskuryakov]

It that something that you want to figure out before the patch lands? Not sure if it’s meant for review if so. 

Given its current behavior though, I think that changes in comment 3 may be appropriate before landing.

[Simon Fraser (smfr)]

Created attachment 349227 [details]
Patch

[Simon Fraser (smfr)]

Bump.

[Simon Fraser (smfr)]

I think we should put this file into fast/harness and include it in layout-test-results, with a link from results.html.

[Simon Fraser (smfr)]

Created attachment 407909 [details]
Patch

[EWS]

Committed r266727: <https://trac.webkit.org/changeset/266727>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 407909 [details].

[Radar WebKit Bug Importer]

<rdar://problem/68513459>