Bug 202872

Summary: USE_ANGLE=1 build broken with last ANGLE roll
Product: WebKit Reporter: Kenneth Russell <kbr>
Component: ANGLEAssignee: Kenneth Russell <kbr>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, annulen, commit-queue, dino, ews-watchlist, graouts, gyuyoung.kim, jdarpinian, justin_fan, kondapallykalyan, rakuco, ryuan.choi, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 202780    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch none

Kenneth Russell
Reported 2019-10-11 17:08:44 PDT
The last ANGLE roll in https://bugs.webkit.org/show_bug.cgi?id=202780 broke WebKit's USE_ANGLE=1 build due to not including FrameCapture.cpp. Upstream ANGLE bug https://bugs.chromium.org/p/angleproject/issues/detail?id=4001 has been filed about this issue and a patch is up for review. A small patch can be applied to WebKit's copy of ANGLE to fix this build.
Attachments
Patch (4.02 KB, patch)
2019-10-11 17:13 PDT, Kenneth Russell 		no flags
DetailsFormatted DiffDiff
Patch (5.96 KB, patch)
2019-10-11 18:00 PDT, Kenneth Russell 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Kenneth Russell
Comment 1 2019-10-11 17:13:17 PDT
Created attachment 380803 [details] Patch
Kenneth Russell
Comment 2 2019-10-11 17:13:59 PDT
Tested locally with USE_ANGLE=1 build. Sorry for the breakage; I was obviously testing the USE_ANGLE=0 build locally for a while for another patch.
EWS Watchlist
Comment 3 2019-10-11 17:14:18 PDT
Note that there are important steps to take when updating ANGLE. See http://trac.webkit.org/wiki/UpdatingANGLE
Kenneth Russell
Comment 4 2019-10-11 17:31:28 PDT
Comment on attachment 380803 [details] Patch Never mind, there's probably a better build fix per feedback on the upstream ANGLE bug.
Kenneth Russell
Comment 5 2019-10-11 18:00:49 PDT
Created attachment 380806 [details] Patch
Kenneth Russell
Comment 6 2019-10-11 18:01:14 PDT
Comment on attachment 380806 [details] Patch This is the approach recommended by the ANGLE developers.
WebKit Commit Bot
Comment 7 2019-10-11 19:56:26 PDT
The commit-queue encountered the following flaky tests while processing attachment 380806 [details]: fetch/fetch-worker-crash.html bug 187257 (author: youennf@gmail.com) The commit-queue is continuing to process your patch.
WebKit Commit Bot
Comment 8 2019-10-11 19:57:18 PDT
Comment on attachment 380806 [details] Patch Clearing flags on attachment: 380806 Committed r251037: <https://trac.webkit.org/changeset/251037>
WebKit Commit Bot
Comment 9 2019-10-11 19:57:20 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 10 2019-10-15 14:37:22 PDT
<rdar://problem/56308615>
Note You need to log in before you can comment on or make changes to this bug.