Bug 230203

Summary: Add the ability to show a flat list of the slowest 200 tests in the tests treemap viewer
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Tools / TestsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, jbedard, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Simon Fraser (smfr)
Reported 2021-09-12 11:20:31 PDT
Add the ability to show a flat list of the slowest 200 tests in the tests treemap viewer
Attachments
Patch (8.18 KB, patch)
2021-09-12 11:21 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2021-09-12 11:21:59 PDT
Created attachment 437991 [details] Patch
Jonathan Bedard
Comment 2 2021-09-13 08:01:27 PDT
Where do we actually expose the tree-map viewer right now?
Simon Fraser (smfr)
Comment 3 2021-09-13 08:34:07 PDT
We don't expose it anywhere (I wish we did though). A user would have to manually open the html file and drag in stats.json.
Jonathan Bedard
Comment 4 2021-09-13 08:40:57 PDT
Comment on attachment 437991 [details] Patch I guess we're not making the bad decision of making test-duration-treemap.html part of layout tests any worse, but it does seem like the entire concept of the test tree should be owned by results.webkit.org or our layout test results
EWS
Comment 5 2021-09-13 13:34:09 PDT
Committed r282352 (241618@main): <https://commits.webkit.org/241618@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 437991 [details].
Radar WebKit Bug Importer
Comment 6 2021-09-13 13:35:38 PDT
<rdar://problem/83070043>
Note You need to log in before you can comment on or make changes to this bug.