Closed Bug 709947 Opened 8 years ago Closed 8 years ago
Don't use the ESSL backend of ANGLE
We don't need a ESSL backend in ANGLE, since we don't currently perform shader transformations on platforms that use OpenGL ES. So we used to just happily use the GLSL backend everywhere, and actually that used to be all what there was, but a few months ago I saw that new ESSL backend and I though: oh, shiny! And I proudly wrote that line of code: gl->IsGLES2() ? SH_ESSL_OUTPUT : SH_GLSL_OUTPUT, Now it turns out that we've had terrible WebGL crashiness on Android lately (see e.g. bug 684863 and bug 709712) and revertion to just SH_GLSL_OUTPUT, fixes all of these crashes. Again, it doesn't matter that we're getting GLSL output, on ES we just discard it and use the original source (stripped from comments) to feed glShaderSource. So no functional behavior change will happen as a result from this change. This is true as long as we don't use any shader transformation on ES. (We currently only do on Mac, which uses desktop GL).
Attachment #581036 - Flags: review?(jgilbert)
Attachment #581036 - Flags: review?(jgilbert) → review+
Comment on attachment 581075 [details] [diff] [review] don't use ESSL backend, for landing on mozilla-beta [Triage Comment] Approving for beta given the WebGL strategic landscape on mobile, and the fact that this is now even lower risk (Android specific).
Attachment #581075 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
http://hg.mozilla.org/releases/mozilla-beta/rev/d2f1a400e430 keeping open for central and aurora
I verified this on Firefox9 Beta6 and didn't see any crashes. Mozilla /5.0 (Android;Linux armv7l;rv:9.0) Gecko/20111212 Firefox/9.0 Fennec/9.0 Device: HTC Desire (Android 2.2)
Comment on attachment 581075 [details] [diff] [review] don't use ESSL backend, for landing on mozilla-beta OK, there only remains to land this on Aurora. Please!
Attachment #581075 - Flags: approval-mozilla-aurora?
I accidentally left myself as the committer of record when landing this in mozilla-central. I understand that this will go on my permanent record. http://hg.mozilla.org/mozilla-central/rev/dc12262285a4
Assignee: nobody → bjacob
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
We need this on aurora as well.
That's why there is a approval? flag on this patch. Please grant!
Comment on attachment 581075 [details] [diff] [review] don't use ESSL backend, for landing on mozilla-beta Whoops, must have missed it.
Attachment #581075 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.