Bug 43500

Summary: Update ANGLE to r352
Product: WebKit Reporter: Kenneth Russell <kbr>
Component: WebGLAssignee: Kenneth Russell <kbr>
Status: RESOLVED FIXED    
Severity: Normal CC: adele, cmarrin, dglazkov, japhet, psawaya, webkit.review.bot, zmo
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 42405    
Attachments:
Description Flags
Patch dglazkov: review+, kbr: commit-queue-

Description Kenneth Russell 2010-08-04 12:06:24 PDT
We need to update the copy of ANGLE checked in to WebKit to r352 to track the version used in Chromium. Recent API changes in the shader validator are preventing the patch in https://bugs.webkit.org/show_bug.cgi?id=42405 from building on Chromium.
Comment 1 Chris Marrin 2010-08-04 12:11:25 PDT
I don't think this bug should block https://bugs.webkit.org/show_bug.cgi?id=42405. We can still land the integration with WebCore without making this change
Comment 2 Kenneth Russell 2010-08-04 12:19:23 PDT
(In reply to comment #1)
> I don't think this bug should block https://bugs.webkit.org/show_bug.cgi?id=42405. We can still land the integration with WebCore without making this change

That is true, but the Chromium project could not then update to a newer WebKit without updating both the ANGLE in WebKit and the use of ANGLE in WebCore. Any public API changes to the shader validator will cause this problem. I'd like to address the known outstanding issue before checking in the initial code to WebCore which uses ANGLE. I'll have a patch up in a few minutes so it should not block 42405 for long.
Comment 3 Kenneth Russell 2010-08-04 12:24:01 PDT
Created attachment 63478 [details]
Patch

Updated ANGLE to r352 (July 26, 2010). Built WebKit to test.
Comment 4 WebKit Review Bot 2010-08-04 12:25:34 PDT
Attachment 63478 [details] did not pass style-queue:

Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1
Last 3072 characters of output:
er/Initialize.cpp:330:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:331:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:332:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:333:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:335:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:336:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:337:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:338:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:340:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:341:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:342:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:343:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:361:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:392:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:393:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:394:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:395:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:397:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:398:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:399:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:400:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:424:  One space before end of line comments  [whitespace/comments] [5]
ANGLE/src/compiler/Initialize.cpp:425:  One space before end of line comments  [whitespace/comments] [5]
ANGLE/src/compiler/Initialize.cpp:426:  One space before end of line comments  [whitespace/comments] [5]
ANGLE/src/compiler/Initialize.cpp:594:  Missing space before ( in switch(  [whitespace/parens] [5]
ANGLE/src/compiler/Initialize.cpp:598:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:599:  Should have a space between // and comment  [whitespace/comments] [4]
ANGLE/src/compiler/Initialize.cpp:600:  Should have a space between // and comment  [whitespace/comments] [4]
Total errors found: 48 in 9 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 Dimitri Glazkov (Google) 2010-08-04 12:29:14 PDT
Comment on attachment 63478 [details]
Patch

rs=me.
Comment 6 Kenneth Russell 2010-08-04 12:32:03 PDT
Committed r64665: <http://trac.webkit.org/changeset/64665>