Bug 56396 – Update ANGLE to at least r575

• Status: RESOLVED
• Resolution: FIXED
• Component: WebGL

[David Kilzer (:ddkilzer)]

The ANGLE project needs to be updated to at least r575 before Bug 56337 can be fixed.

[David Kilzer (:ddkilzer)]

Alok: Are you working on updating WebKit's copy of ANGLE?  This is still blocking Bug 56337.

[Zhenyao Mo]

I am working on another ANGLE shader translator feature at the moment.  Once it's done, I'll do the rolling.

[David Kilzer (:ddkilzer)]

(In reply to comment #2)
> I am working on another ANGLE shader translator feature at the moment.  Once it's done, I'll do the rolling.

I merged the following commits from upstream to unblock Bug 56337:

r572
r573
r574
r575

Committed r85761.  <http://trac.webkit.org/changeset/85761>

[Zhenyao Mo]

Finally I got the Angle shader translator features in.  I'll roll Angle to r696 as I promised.

[Zhenyao Mo]

Created attachment 97873 [details]
Patch

[Zhenyao Mo]

This patch has been tested with run-webkit-tests LayoutTests/fast/canvas/webgl and also run-safari with some webgl demos.  Please review.

[WebKit Review Bot]

Attachment 97873 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/fast..." exit_code: 1

Last 3072 characters of output:
ce/ThirdParty/ANGLE/src/libGLESv2/vertexconversion.h:199:  This { should be at the end of the previous line  [whitespace/braces] [4]
Source/ThirdParty/ANGLE/src/libGLESv2/vertexconversion.h:208:  One space before end of line comments  [whitespace/comments] [5]
Source/ThirdParty/ANGLE/src/compiler/glslang_tab.cpp:3666:  One line control clauses should not use braces.  [whitespace/braces] [4]
Source/ThirdParty/ANGLE/src/compiler/glslang_tab.cpp:4096:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:39:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:40:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:41:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:71:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:72:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:77:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:78:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:79:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:80:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:81:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:82:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:83:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:84:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:147:  The parameter name "output" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:242:  Use 0 or null instead of NULL (even in *comments*).  [readability/null] [4]
Source/ThirdParty/ANGLE/include/GLSLANG/ShaderLang.h:269:  Use 0 or null instead of NULL (even in *comments*).  [readability/null] [4]
Total errors found: 1202 in 97 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Kenneth Russell]

Comment on attachment 97873 [details]
Patch

Looks good to me.

[Zhenyao Mo]

Committed r89330: <http://trac.webkit.org/changeset/89330>