Closed Bug 965844 Opened 7 years ago Closed 7 years ago

Remove the pref "layers.prefer-memory-over-shmem"

Categories

(Core :: Graphics: Layers, defect)

Product:
Core
Component:
Graphics: Layers
Platform:
x86_64
All
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla29

People

(Reporter: nical, Assigned: nical)

Details

Attachments

(1 file)

Remove the pref
1.85 KB, patch
gal
: review+
Details | Diff | Splinter Review 
I was shy enough to add MemoryTextureClient behind a pref and turn the pref on later on but nobody ever uses that pref anymore, and there is no reason to disable MemoryTextureClient. So now the pref is just polluting about:config and adding a potential divergence between the official config and what users run.

So let's remove the pref. The choice between MemoryTextureClient and ShmemTextureClient should be only dictated by whether we do cross-thread or cross-process sharing.
Attached patch Remove the pref — — Splinter Review
Assignee: nobody → nical.bugzilla
Attachment #8367993 - Flags: review?(jmuizelaar)
Comment on attachment 8367993 [details] [diff] [review]
Remove the pref

Righteous.
Attachment #8367993 - Flags: review?(jmuizelaar) → review+
https://hg.mozilla.org/mozilla-central/rev/176fa19e610c
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
Even the DOM fuzzer wasn't using this pref, because of bug 937603.
You need to log in before you can comment on or make changes to this bug.