Reenable ANGLE shader translation on Android

RESOLVED FIXED in mozilla15

Status

()

Core
Canvas: WebGL
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: bjacob, Assigned: bjacob)

Tracking

unspecified
mozilla15
ARM
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: webgl-conformance webgl-next)

Attachments

(1 attachment)

In bug 709947, we turned off shader translation on Android, with this comment:

// see bug 709947. On Android, we can't use the ESSL backend because of strange crashes (might be
// an allocator mismatch). So we use the GLSL backend, and discard the output, instead just passing
// the original WebGL shader source to the GL (since that's ESSL already). The problem is that means
// we can't use shader translations on Android, in particular we can't use long identifier shortening,
// which means we can't reach 100% conformance. We need to fix that by debugging the ESSL backend
// memory crashes.

We should figure out what really causes these crashes (allocator mismatch?) and fix them, and re-enabled shader translation. It's a requirement for full conformance on Android, and it's important for stability/security.
(Assignee)

Updated

5 years ago
Blocks: 719232
(Assignee)

Comment 1

5 years ago
Created attachment 618775 [details] [diff] [review]
reenable shader translation on android
Assignee: nobody → bjacob
Attachment #618775 - Flags: review?(jgilbert)
(Assignee)

Updated

5 years ago
Blocks: 749295
Depends on: 748654
Attachment #618775 - Flags: review?(jgilbert)
Attachment #618775 - Flags: review+
(Assignee)

Comment 2

5 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/4bb0aa38c2d4
Target Milestone: --- → mozilla14
(Assignee)

Comment 3

5 years ago
Comment on attachment 618775 [details] [diff] [review]
reenable shader translation on android

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: we strongly suspect that the lack of shader translation on android is the cause of webgl crashes. in particular, without shader translation, we can't do long identifier mapping. this also prevents us from being webgl conformant on android.
Testing completed (on m-c, etc.): inbound
Risk to taking this patch (and alternatives if risky): I am rather confident that bug 748654 removed the reason why we had to disable shader translation on android. So this should be safe. If we see a surge of crashes, we'll re-disable.
String changes made by this patch: none
Attachment #618775 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/4bb0aa38c2d4
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: mozilla14 → mozilla15

Comment 5

5 years ago
Comment on attachment 618775 [details] [diff] [review]
reenable shader translation on android

[Triage Comment]
Android only, and we'll disable if we see more crashiness with WebGL. Approved for Aurora 14.
Attachment #618775 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 6

5 years ago
Oops, apparently this was never backported to 14, sorry... anyway 14's been out for a while on Android,  and this doesn't matter for non-Android platforms.
You need to log in before you can comment on or make changes to this bug.