Closed
Bug 862276
Opened 12 years ago
Closed 2 years ago
[meta] reduce costs from descendants of simple dynamic style changes (e.g., changing 'transform' in style attribute)
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
feature-b2g | - |
People
(Reporter: dbaron, Unassigned)
References
Details
(Keywords: meta, perf, Whiteboard: [c=uniformity p= s= u=])
A number of the performance issue in Firefox OS related to style code are related to the cost of relatively simple dynamic style changes (such as changing 'transform' in a style attribute) that's the result of assigning new style contexts to the element's descendants.
Bug 779968 did some previous work to improve this case.
Two ideas on further improvements here are in bug 804975 and bug 862274. I need to figure out which, if any, of these ideas are practical and how to move forward.
Updated•11 years ago
|
Whiteboard: [c= p= s= u=]
Comment 1•11 years ago
|
||
I listed a few problems in bug 939460 where style flushes are preventing us from having smooth animations. Ideally this bug will be sufficient to close all of these. Given this I think this should be a harder blocker for 1.3.
blocking-b2g: --- → 1.3?
Comment 3•11 years ago
|
||
David, this came up in the rendering meeting, we talked about it a bit. We think it's a blocker for 1.3 (Dec 9) - can you help us figure out how to handle this?
Flags: needinfo?(dbaron)
Reporter | ||
Comment 4•11 years ago
|
||
I think bug 931668 (already on dependency list) should make a big dent here, and heycam is working on that.
Flags: needinfo?(dbaron)
Reporter | ||
Comment 5•11 years ago
|
||
(And by "make a big dent here", I mean I think it will be sufficient to address the problems we're seeing.)
Comment 6•11 years ago
|
||
Thanks for the quick response, and I agree, that's a very big dent :). I will roll this back to 1.3?, as we would want to get the results from bug 931668, and hopefully we won't need the rest for 1.3.
blocking-b2g: 1.3+ → 1.3?
Comment 7•11 years ago
|
||
Minusing from 1.3 per Milan's comment 6. Renom if necessary after bug 931668 lands.
(In reply to Milan Sreckovic [:milan] from comment #6)
> Thanks for the quick response, and I agree, that's a very big dent :). I
> will roll this back to 1.3?, as we would want to get the results from bug
> 931668, and hopefully we won't need the rest for 1.3.
blocking-b2g: 1.3? → -
Updated•11 years ago
|
Priority: -- → P1
Whiteboard: [c= p= s= u=] → [c=progress p= s= u=]
Comment 8•11 years ago
|
||
FxOS Perf P1 as this has a direct negative impact on Frame Uniformity.
Updated•11 years ago
|
Whiteboard: [c=progress p= s= u=] → [c=uniformity p= s= u=]
Comment 9•11 years ago
|
||
We need this in order to make some of the new FxOS homescreen animations perform well.
Blocks: 988401
blocking-b2g: - → 1.5?
Updated•11 years ago
|
We shouldn't block on the meta--please nom the dependent bugs as necessary.
blocking-b2g: 1.5? → ---
Flags: needinfo?(kgrandon)
Summary: reduce costs from descendants of simple dynamic style changes (e.g., changing 'transform' in style attribute) → [meta] reduce costs from descendants of simple dynamic style changes (e.g., changing 'transform' in style attribute)
Comment 12•11 years ago
|
||
This is super nice to have, but I'm guessing this won't make 2.0.
feature-b2g: --- → -
Updated•11 years ago
|
Priority: P1 → P2
Updated•10 years ago
|
Blocks: vertical-home-next
Updated•10 years ago
|
No longer blocks: vertical-homescreen
Comment 13•10 years ago
|
||
If bug 862274 is a dupe of bug 931668 then there should only be bug 804975 left. We should try to wrap it up ASAP since this bug is causing significant delays all over FFOS.
Reporter | ||
Comment 14•9 years ago
|
||
Bug 1180120 could also help here. Bug 1180118 is less directly related to comment 0 but could also help.
Comment 15•6 years ago
|
||
Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
Comment 16•6 years ago
|
||
Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Updated•2 years ago
|
Severity: normal → S3
Comment 17•2 years ago
|
||
Inactive metabug
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•