628403 - introduce webgl.force-enabled and webgl.disabled prefs

Status: RESOLVED FIXED

(Core :: Graphics: CanvasWebGL, defect)

Description

[Benoit Jacob [:bjacob] (mostly away)]

Attachment: webgl force-enabled and disabled prefs

This replaces the old webgl.enabled_for_all_sites by a tri-state like we already do for layers and for d2d.

force-enabled bypasses the blacklist, allowing us to land bug 623338.

This patch also removed the old mochitest_native_gl pref that we no longer use.

Comment 1

[Benoit Jacob [:bjacob] (mostly away)]

note: this patch makes the WebGL mochitest ignore the blacklist.

Comment 2

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Comment on attachment 506524 [details] [diff] [review]
webgl force-enabled and disabled prefs

(fwiw, enabled_for_all_sites was intended to be part of a per-site permissions thing that we never implemented :)

While you're in this code, can you change prefer_gl to prefer-native-gl and add it to all.js?

Comment 3

[Benoit Jacob [:bjacob] (mostly away)]

Attachment: prefer-native-gl pref

Comment 4

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Comment on attachment 506854 [details] [diff] [review]
prefer-native-gl pref

great, thanks :)

Comment 5

[Benoit Jacob [:bjacob] (mostly away)]

http://hg.mozilla.org/mozilla-central/rev/828decbcc564
http://hg.mozilla.org/mozilla-central/rev/2a7ac4407669