Markus Stange [:mstange] Reporter
	Description • 7 years ago

Profile: https://perfht.ml/2zJJO0C

Steps to reproduce:
 1. Go to http://browserbench.org/MotionMark/developer.html
 2. In the right column, check "Keep at a fixed complexity"
 3. In the left column, only check the Animometer: Multiply test, and set its complexity to 400
 4. Click "Run benchmark" and profile.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com) Assignee
	Comment 1 • 7 years ago

https://searchfox.org/mozilla-central/rev/7a8c667bdd2a4a32746c9862356e199627c0896d/gfx/layers/wr/ScrollingLayersHelper.cpp#360-362

When I wrote this code the thing I had in mind to fix the potential cliff was what I described in bug 1415932, but that turned out to be harder than expected (see comment 1 on that bug). I can try again to see if I can come up with a better way.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com) Assignee
	Comment 2 • 7 years ago

I have an idea on how to fix this without going the route of bug 1415932. Picking this up as if it works it will be a big win.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com) Assignee
	Comment 3 • 7 years ago

I wasn't able to reproduce the problem as described in comment 0, however I did see RecurseAndDefineClip taking up 50% of time on MazeSolver as well, so I used that instead to test. The patch I have drastically improves MazeSolver time (~57 seconds down to ~14 seconds).

Try push to make sure it doesn't break anything: https://treeherder.mozilla.org/#/jobs?repo=try&revision=48ec1ceb1bc3aefaafc775ce080ab95efba32d49

	Kartikaya Gupta (email:kats@mozilla.staktrace.com) Assignee
	Comment 4 • 7 years ago

For comparison the same build with WR disabled gives me results of ~17 seconds.

	Markus Stange [:mstange] Reporter
	Comment 6 • 7 years ago
mozreview-review

Comment on attachment 8932595 [details]
Bug 1421054 - Remove performance cliff resulting from linear search through clips.

https://reviewboard.mozilla.org/r/203652/#review209514

	Pulsebot
	Comment 7 • 7 years ago

Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/92847636d7b1
Remove performance cliff resulting from linear search through clips. r=mstange

	Stefan Hindli [:stefan_hindli]
	Comment 8 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/92847636d7b1