The default bug view has changed. See this FAQ.

glEGLImageTargetTexture2DOES shows error 501 frequently

RESOLVED FIXED in Firefox 11

Status

()

Firefox for Android
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: snorp, Assigned: snorp)

Tracking

unspecified
Firefox 12
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

Attachments

(1 attachment, 1 obsolete attachment)

This is actually from other GL commands and not the one it is reporting. Should fix this up so at least it doesn't show bogus errors.
Created attachment 585797 [details] [diff] [review]
Clear existing GL errors before trying to check for a new one
Attachment #585797 - Flags: review?(chrislord.net)
Created attachment 585798 [details] [diff] [review]
Clear existing GL errors before trying to check for a new one
Attachment #585797 - Attachment is obsolete: true
Attachment #585797 - Flags: review?(chrislord.net)
Attachment #585798 - Flags: review?(chrislord.net)

Comment 3

5 years ago
Comment on attachment 585798 [details] [diff] [review]
Clear existing GL errors before trying to check for a new one

Review of attachment 585798 [details] [diff] [review]:
-----------------------------------------------------------------

Fine by me - I wonder if we want to have wrappers for GL commands though (I think gfx does this?) that will read and report errors automatically for debug builds, or something along those lines... But this can be addressed in a future patch.

Optionally, it'd be cool to have comments above the while loops, at least in the Java file, explaining why it's being done - something like "GL errors may have been encountered elsewhere, but we're only interested in those that occur here"
Attachment #585798 - Flags: review?(chrislord.net) → review+
I think there might be a debug EGLContext in Android that basically wraps everything and checks for errors. Maybe we should just add support for using that and ditch the specific error checking.

In the mean time I'll add the comment and check this in.
Pushed to mozilla-inbound:

http://hg.mozilla.org/integration/mozilla-inbound/rev/4d652a824f11
Landed on mozilla-central:

https://hg.mozilla.org/mozilla-central/rev/4d652a824f11
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Target Milestone: --- → Firefox 12
Comment on attachment 585798 [details] [diff] [review]
Clear existing GL errors before trying to check for a new one

[Triage Comment]
Attachment #585798 - Flags: approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/d53dfe552b54
tracking-fennec: --- → 11+
status-firefox11: --- → fixed
You need to log in before you can comment on or make changes to this bug.