Summary: | Analysis task page shows an empty results for an irrelevant top-level test | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||||
Component: | Perf Dashboard | Assignee: | Ryosuke Niwa <rniwa> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cdumez, commit-queue, dewei_zhu, kling, koivisto, rniwa, saam, simon.fraser, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | Safari Technology Preview | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Ryosuke Niwa
2017-08-06 17:10:31 PDT
Created attachment 317388 [details]
Fixes the bug
Created attachment 320410 [details]
Rebased for trunk
This patch fixes the bug that analysis task pages for perf try bots show enumerate the union of all top level tests tested instead of the one tested in a particular test group. e.g. if you tested Speedometer & JetStream on a single perf try bots page, then both test groups enumerate both Speedometer & JetStream despite of the fact each test group tested exactly one of them. Comment on attachment 320410 [details]
Rebased for trunk
r=me
Comment on attachment 320410 [details]
Rebased for trunk
Thanks for the review.
Comment on attachment 320410 [details] Rebased for trunk Clearing flags on attachment: 320410 Committed r221871: <http://trac.webkit.org/changeset/221871> All reviewed patches have been landed. Closing bug. |