Bug 92295 - [WebGL] ANGLEWebKitBridge should support ESSL platforms
Summary: [WebGL] ANGLEWebKitBridge should support ESSL platforms
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2012-07-25 14:04 PDT by Joshua Netterfield
Modified: 2012-07-26 08:26 PDT (History)
3 users (show)

See Also:

Patch (2.94 KB, patch)
2012-07-25 16:44 PDT, Joshua Netterfield
no flags Details | Formatted Diff | Diff
Patch (3.10 KB, patch)
2012-07-26 06:53 PDT, Joshua Netterfield
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joshua Netterfield 2012-07-25 14:04:11 PDT
There should be a way for ESSL platforms to request ESSL output from ANGLE.
Comment 1 Joshua Netterfield 2012-07-25 16:04:07 PDT
Didn't see https://bugs.webkit.org/show_bug.cgi?id=87718 (it hasn't landed in our downstream repo yet). This bug is now for having Blackberry take advantage of that.
Comment 2 Joshua Netterfield 2012-07-25 16:44:54 PDT
Created attachment 154480 [details]
Comment 3 Rob Buis 2012-07-25 19:38:16 PDT
Comment on attachment 154480 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=154480&action=review

> Source/WebCore/ChangeLog:9
> +        glsl-mat4-to-mat3.html)

To me as an GLES and shader n00b, it is not clear how the two code sections relate? What does SH_ESSL_OUTPUT do, and why does it mean the hack can be removed? You may have to improve the commit message so more people understand how this change fixes the problem.
Comment 4 Joshua Netterfield 2012-07-26 06:53:14 PDT
Created attachment 154645 [details]
Comment 5 Rob Buis 2012-07-26 06:54:57 PDT
Comment on attachment 154645 [details]

Comment 6 WebKit Review Bot 2012-07-26 08:26:09 PDT
Comment on attachment 154645 [details]

Clearing flags on attachment: 154645

Committed r123749: <http://trac.webkit.org/changeset/123749>
Comment 7 WebKit Review Bot 2012-07-26 08:26:12 PDT
All reviewed patches have been landed.  Closing bug.