Closed Bug 1298813 Opened 6 years ago Closed 6 years ago

Back shared persistent buffer provider out from aurora


(Core :: Graphics: Layers, defect)

Not set



Tracking Status
firefox50 --- fixed


(Reporter: nical, Assigned: nical)



(Whiteboard: [gfx-noted])


(1 file, 1 obsolete file)

Enabled on all platforms except windows. There are regressions and a number of patches that fix them, a few of which have been uplifted already, but I'm not super comfortable with uplifting all of it considering the numerous changes.
I would like to secure plan B: backing the regressing patch out from aurora along with whatever got uplifted, as a potentially less risky solution.
Attached patch Disable BufferProviderShared (obsolete) — Splinter Review
Attachment #8788180 - Flags: review?(bas)
It turns out to be tricky to back every buffer provider related change, so my current plan is to disable it and only uplift the few patches that affect the code paths where it is disabled.
The previous patch was empty.
Attachment #8788180 - Attachment is obsolete: true
Attachment #8788180 - Flags: review?(bas)
Attachment #8788198 - Flags: review?(bas)
Attachment #8788198 - Flags: review?(bas) → review+
Comment on attachment 8788198 [details] [diff] [review]
Disable BufferProviderShared

Approval Request Comment
[Feature/regressing bug #]:
[User impact if declined]: crashes and correctness issues.
[Describe test coverage new/current, TreeHerder]: None.
[Risks and why]: low risk, the majority of our users have this pref disabled already.
[String/UUID change made/needed]: None.

This patch go to aurora directly, not central.
Attachment #8788198 - Flags: approval-mozilla-aurora?
Comment on attachment 8788198 [details] [diff] [review]
Disable BufferProviderShared

Makes sense, Aurora50+
Attachment #8788198 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.