Closed Bug 762259 Opened 8 years ago Closed 8 years ago

Drop vendor suffixes from EGLImage for internal uses.

Categories

(Core :: Canvas: WebGL, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla16
Tracking Status
firefox15 --- fixed
firefox16 --- fixed

People

(Reporter: jgilbert, Assigned: jgilbert)

References

Details

Attachments

(1 file)

Generally we drop vendor suffixes when including extensions, but the original EGLImage patch left them in. We tend towards dropping them since they only add noise, since the suffixes don't tell us anything other than that they are from some sort of extension. ('was this one OES or KHR?'...)

We're going to be using this extension a bunch so it'd be nice to clean it up first.
Attached patch patchSplinter Review
Attachment #630733 - Flags: review?(bjacob)
Blocks: 762265
Attachment #630733 - Flags: review?(bjacob) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/50356ba76bbf
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/50356ba76bbf
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Blocks: 765388
Blocks: 774059
Depends on: 774841
Comment on attachment 630733 [details] [diff] [review]
patch

[Approval Request Comment]
Required by bug 728524
Attachment #630733 - Flags: approval-mozilla-beta?
Comment on attachment 630733 [details] [diff] [review]
patch

Approving supporting patches for bug 687267
Attachment #630733 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.