709712 - WebGL demos causing repeatable crashes

Status: RESOLVED FIXED

(Core :: Graphics: CanvasWebGL, defect)

Description

[Matt Evans [:mevans]]

Web page or screen you were on when you saw the issue: 

Steps to reproduce:
1. go to this webgl test: https://cvs.khronos.org/svn/repos/registry/trunk/public/webgl/sdk/demos/google/shiny-teapot/index.html
2. 
3. 

What you expected: 

A teapot displayed

Actual:

Crash

Crash report ID (if applicable): 

https://crash-stats.mozilla.com/report/index/bp-deffd20c-dca2-4d2c-9877-5a3172111211


I believe other tests in the suite will cause similar crashes. Need to check more of these tests in this suite.

Comment 1

[Benoit Jacob [:bjacob] (mostly away)]

Shouldn't we group together all those malloc-related crashes triggered by WebGL demos on Android? For all we can see, they're likely all the same bug.

Comment 2

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Benoit, I was thinking the same thing.  

How would you like me to group them?
Should I go ahead and dup or do you want me to create a meta bug or do you want me to create a whiteboard tag?

Comment 3

[Aaron Train [:aaronmt]]

Attachment: Nightly (12/12) Crash Log

On every single demo at http://www.chromeexperiments.com/webgl, Fennec Native crashes. 

Attached is an example crash log.

--
Samsung Galaxy SII (Android 2.3.6)
Mozilla/5.0 (Android; Linux armv7l; rv:11.0a1) Gecko/20111212 Firefox/11.0a1 Fennec/11.0a1

Comment 5

[Benoit Jacob [:bjacob] (mostly away)]

I'm testing a patch that makes us revert to using the GLSL backend instead of the ESSL backend in the ANGLE compiler. Shouldn't make a behavior difference anyway, as on ES we're just passing the original shader source to the driver, but could avoid this crash if it's related to the ESSL backend.

https://tbpl.mozilla.org/?tree=Try&rev=72ded8c7f176

Builds and logs will be available at http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-72ded8c7f176

Comment 6

[Benoit Jacob [:bjacob] (mostly away)]

We were able to confirm on AaronMT's phone that this patch fixes the crash.

Comment 7

[Benoit Jacob [:bjacob] (mostly away)]

Fixed by 709947

Comment 8

[Scoobidiver (away)]

The fix has been backported to Gecko 9+.