Bug 152670 – Update data reporting and analysis

• Status: RESOLVED
• Resolution: FIXED
• Component: Animations

[Jon Lee]

Avoid doing a lot of processing during the benchmark itself. Also show raw data on top of the filtered data.

[Radar WebKit Bug Importer]

<rdar://problem/24033996>

[Jon Lee]

Created attachment 268165 [details]
Patch

[Jon Lee]

Created attachment 268166 [details]
1. Avoid processing the data through the Experiment while the test is running.

[Jon Lee]

Created attachment 268167 [details]
2. Refactor how Animator does its recording.

[Jon Lee]

Created attachment 268168 [details]
3. Producing the JSON can take a while with all of the data.

[Jon Lee]

Created attachment 268169 [details]
4. Show new graph data.

[Simon Fraser (smfr)]

Comment on attachment 268165 [details]
Patch

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

> PerformanceTests/Animometer/resources/debug-runner/animometer.js:362
> +        document.querySelector("#results-json button").remove();

Does .remove() work in all browsers we care about?

[Jon Lee]

(In reply to comment #7)
> Comment on attachment 268165 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=268165&action=review
> 
> > PerformanceTests/Animometer/resources/debug-runner/animometer.js:362
> > +        document.querySelector("#results-json button").remove();
> 
> Does .remove() work in all browsers we care about?

Yes.

[Jon Lee]

Committed r194523: <http://trac.webkit.org/changeset/194523>