Bug 174304 – A/B testing results page show results for the top-level tests instead of the one being analyzed

• Status: RESOLVED
• Resolution: FIXED
• Component: Perf Dashboard

[Ryosuke Niwa]

After the perf try bots support was added, the results shown right above the revision table
always show the results for the top-level tests instead of the one being biesected.

e.g. If we're bisecting MotionMark's Images subtest, we'd like to see how A/B compares in Images subtest,
not the overall MotionMark's score, which may or may not show the actual regression / progression
due to the noise from other tests.

What's even more confusing is the fact the stacking graph charts DO show the progression/regression
amount based on the subtest.

[Ryosuke Niwa]

Created attachment 314959 [details]
Fixes the bug

[Antti Koivisto]

Comment on attachment 314959 [details]
Fixes the bug

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

> Websites/perf.webkit.org/public/v3/models/analysis-results.js:23
> +        for (let metric of test.metrics()) {

const?

[Ryosuke Niwa]

Comment on attachment 314959 [details]
Fixes the bug

Clearing flags on attachment: 314959

Committed r219321: <http://trac.webkit.org/changeset/219321>

[Ryosuke Niwa]

All reviewed patches have been landed.  Closing bug.