Don't blacklist OGL on Linux if we don't have TextureFromPixmap

RESOLVED FIXED in Firefox 49

Status

()

RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: BenWa, Assigned: lsalzman)

Tracking

(Blocks: 1 bug)

unspecified
mozilla49
Points:
---

Firefox Tracking Flags

(firefox49 fixed, relnote-firefox -)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
This blacklist doesn't make any sense since we don't use the extension.
(Reporter)

Updated

3 years ago
Blocks: 594876
(Assignee)

Comment 1

2 years ago
Created attachment 8746153 [details] [diff] [review]
don't block OpenGL layers when lacking EXT_texture_from_pixmap

We don't actually use TFP unless it is pref'd on at current, and when it is pref'd on, it is broken anyway until we have time to reevaluate it and GL context initialization for it.

Since OpenGL layers still offers a performance advantage over basic layers even without TFP, it makes no sense to blacklist because of a lack of TFP at present.
Assignee: nobody → lsalzman
Status: NEW → ASSIGNED
Attachment #8746153 - Flags: review?(karlt)
(Assignee)

Updated

2 years ago
See Also: → bug 1167557
Comment on attachment 8746153 [details] [diff] [review]
don't block OpenGL layers when lacking EXT_texture_from_pixmap

Any reason to keep the code?
Are you planning to resurrect a variant at some stage?
Attachment #8746153 - Flags: review?(karlt) → review+

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/963cf67776be
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Release Note Request (optional, but appreciated)
[Why is this notable]:
[Suggested wording]: Linux installations not supporting EXT_texture_from_pixmap OpenGL extension are not blocked from OpenGL acceleration anymore.
[Links (documentation, blog post, etc)]:
relnote-firefox: --- → ?
Relnote -, pretty technical description that we are not sure would be helpful to end users.
relnote-firefox: ? → 49+
relnote-firefox: 49+ → -
You need to log in before you can comment on or make changes to this bug.