Closed Bug 933354 Opened 7 years ago Closed 7 years ago

Talos SVG-ASAP (tsvg) regression from bug 923193 (transform-origin in SVG)


(Core :: SVG, defect)

27 Branch
Not set



Tracking Status
firefox26 --- unaffected
firefox27 + fixed
firefox28 --- fixed


(Reporter: mbrubeck, Assigned: roc)



(Keywords: perf, regression, Whiteboard: [qa-])


(1 file)

Bug 923193 caused a regression in the Talos tsvg benchmark on all platforms:

Regression: Mozilla-Inbound-Non-PGO - SVG-ASAP - WINNT 6.1 (ix) - 25% increase
    Previous: avg 312.057 stddev 7.814 of 12 runs up to revision 9c8ab7e9ae41
    New     : avg 389.992 stddev 3.334 of 12 runs since revision 45d9e6cd3473
    Change  : +77.936 (25% / z=9.974)
    Graph   : 

This was partially fixed by bug 929021:

Improvement: Mozilla-Inbound-Non-PGO - SVG-ASAP - WINNT 6.1 (ix) - 7.7% decrease
    Previous: avg 393.485 stddev 2.711 of 12 runs up to revision c79088ab4e0c
    New     : avg 363.197 stddev 3.229 of 12 runs since revision c1578a4fc86d
    Change  : -30.288 (7.7% / z=11.171)
    Graph   : 

Filing this bug to track the remaining regression.  See bug 929021 comment 11 and 12 for some relevant discussion.
Roc, should we backout bug 923193 or track and try to fix in FF27? Any reason to believe this isn't an important perf regression?
Flags: needinfo?(roc)
Attached patch fixSplinter Review
This should fix the rest of the regression.
Assignee: nobody → roc
Attachment #827130 - Flags: review?(cam)
Flags: needinfo?(roc)
Attachment #827130 - Flags: review?(cam) → review+
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Comment on attachment 827130 [details] [diff] [review]

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 923193
User impact if declined: slight performance regression for SVG elements with transforms
Testing completed (on m-c, etc.): just landed
Risk to taking this patch (and alternatives if risky): seems low risk
String or IDL/UUID changes made by this patch: none
Attachment #827130 - Flags: approval-mozilla-aurora?
Attachment #827130 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
It's hard to tell whether this regression is fixed or not since bug 922942 regressed Tsvg more in the meantime.

I think I will back out bug 923193, this fix, and the fix in bug 929021. Then I will fold those patches together and reland them to see if there is still a regression.
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.