New issue
Advanced search Search tips

Issue 606680 link

Starred by 3 users

Issue metadata

Status: Duplicate
Merged: issue 605792
Owner:
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocked on:
issue 605792



Sign in to add a comment

Investigate performance problems for 'HTML suite > CSS bouncing circles' benchmark

Project Member Reported by dk...@chromium.org, Apr 26 2016

Issue description

All benchmarks are under pr.gg/animometer/developer.html

- Set "Ramp" mode for benchmarking.
- Set "Keep at a fixed complexity" for profiling.  Set an appropriate [high] complexity next to each test's name.

File any performance bugs found as blocking this issue.
 

Comment 1 by dk...@chromium.org, Apr 27 2016

Owner: rbyers@chromium.org
Status: Assigned (was: Untriaged)

Comment 2 by dk...@chromium.org, May 20 2016

Ping! Any specific insights for this benchmark?

Comment 3 by rbyers@chromium.org, May 26 2016

Shoot, sorry - I missed that I was going to get an animometer benchmark to investigate (although I have been looking at some others).  I will get on this.

Comment 4 by rbyers@chromium.org, May 26 2016

Blockedon: 605792
Cc: esprehn@chromium.org
Chrome performance on this has more than doubled in recent builds on Mac, perhaps due to Ganesh now being enabled.

Still, a significant fraction of the time (~25%) is spent parsing CSS transforms of the form: "translate(877px, 205px) rotateZ(0deg)".  Issue 605792 tracks the larger issue here, but it looks like https://codereview.chromium.org/1904633002/ may be a hack to get us a bunch of benefits.

Comment 5 by rbyers@chromium.org, May 26 2016

Status: Started (was: Assigned)

Comment 6 by rbyers@chromium.org, May 26 2016

It's kinda hard to measure the impact of improvements on this benchmark as the results seem to vary wildly.  On my macbook even in Safari it reports as +-20% but typically varies overall by 50% (between 400 and 600).

Comment 7 by rbyers@chromium.org, May 26 2016

Running for 120s and "maintain target frame rate at 50fps" I can get much more stable results.  Using Elliott's patch (https://codereview.chromium.org/1904633002/) gives me about a 20% increase in score.

In particular, it cuts the time spent in JS (which is almost entirely just setting the transform on all the nodes) in half (eg. from 6ms/frame to 3ms/frame).

Comment 8 by rbyers@chromium.org, Aug 31 2016

Blockedon: -605792
Components: Blink>CSS
Mergedinto: 605792
Status: Duplicate (was: Started)
I don't think there's anything left here other than continuing to improve the CSS fast paths (issue 605792).  Let's continue tracking the work there.

Comment 9 by rbyers@chromium.org, Aug 31 2016

Blockedon: 605792

Sign in to add a comment