Closed Bug 1312606 Opened 5 years ago Closed 5 years ago

Fix gfxPref syncing stale prefs to GPU process

Categories

(Core :: Graphics, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: rhunt, Assigned: rhunt)

References

Details

Attachments

(1 file)

Don't use the cached value for syncing to the GPU process when we observer a pref change, because the cached value may not have been updated yet. This can cause the prefs in the GPU process to get out of sync.
Attached patch stale-pref.patchSplinter Review
Attachment #8804105 - Flags: review?(dvander)
Attachment #8804105 - Flags: review?(dvander) → review+
Pushed by rhunt@eqrion.net:
https://hg.mozilla.org/integration/mozilla-inbound/rev/95e09163347a
Use live pref values when syncing gfxPrefs to the GPU process r=dvander
Backout by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a1f1a53aac73
Backed out changeset 95e09163347a for build bustage. r=backout on a CLOSED TREE
https://treeherder.mozilla.org/#/jobs?repo=try&revision=55fd19db6b57

Fix for bustage. There should be no behavioral differences except for in the gpu process which is not enabled yet.
Pushed by rhunt@eqrion.net:
https://hg.mozilla.org/integration/mozilla-inbound/rev/28986ba05618
Use live pref values when syncing gfxPrefs to the GPU process r=dvander
https://hg.mozilla.org/mozilla-central/rev/28986ba05618
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.