Summary: | [WebGL] Mac/ATI/AMD systems need to translate built-in GLSL functions | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dean Jackson <dino> | ||||
Component: | WebGL | Assignee: | Dean Jackson <dino> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | gman, kbr, noam, webkit-bug-importer, webkit.review.bot | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Dean Jackson
2012-08-14 16:02:33 PDT
It seems like Apple ports, unlike Chrome, are not picking up ANGLE's function emulation. In particular: TFunctionDistance1_1 which is marked as problematic on ATI and NVidia cards. Here is ANGLE adding a distance emulation: http://code.google.com/p/angleproject/source/detail?spec=svn1255&r=754 Original title was [WebGL] conformance/glsl/functions/glsl-function-distance.html failing on Apple Mountain Lion Created attachment 159835 [details]
Patch
Attachment 159835 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1
Source/WebCore/platform/graphics/ANGLEWebKitBridge.h:56: The parameter name "shaderType" adds no information, so it should be removed. [readability/parameter_name] [5]
Total errors found: 1 in 4 files
If any of these errors are false positives, please file a bug against check-webkit-style.
I deliberately ignored the style warning: didn't want to change an existing parameter def in this patch. *** Bug 94032 has been marked as a duplicate of this bug. *** *** Bug 94031 has been marked as a duplicate of this bug. *** Comment on attachment 159835 [details]
Patch
Should probably use a typedef for the flags.
(In reply to comment #10) > (From update of attachment 159835 [details]) > Should probably use a typedef for the flags. ShCompileOptions is the enum of the various options. Not sure what we could call the typedef - maybe ANGLECompileOptions? Anyway, this code will be updated very soon and I'll address it then. Committed r126342: <http://trac.webkit.org/changeset/126342> |