Bug 231201

Summary: [JSC] Update test262
Product: WebKit Reporter: Philip Chimento <philip.chimento>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: clopez, darin, keith_miller, mark.lam, ross.kirsling, saam, webkit-bug-importer, xan.lopez, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Second patch none

Description Philip Chimento 2021-10-04 17:55:30 PDT 
[JSC] Update test262
Comment 1 Philip Chimento 2021-10-04 17:59:31 PDT 
Created attachment 440130 [details]
Patch
Comment 2 Carlos Alberto Lopez Perez 2021-10-05 20:39:55 PDT 
TIP: pass the flag '--suggest-reviewers' to webkit-patch so the tool automatically recommends the reviewers to CC. Otherwise the patch won't get noticed.
Comment 3 Philip Chimento 2021-10-06 10:54:42 PDT 
Created attachment 440388 [details]
Patch
Comment 4 Philip Chimento 2021-10-06 11:24:32 PDT 
Created attachment 440394 [details]
Patch
Comment 5 EWS 2021-10-06 13:20:55 PDT 
Committed r283651 (242599@main): <https://commits.webkit.org/242599@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 440394 [details].
Comment 6 Radar WebKit Bug Importer 2021-10-06 13:22:14 PDT 
<rdar://problem/83948851>
Comment 7 Philip Chimento 2021-10-06 13:35:15 PDT 
Reopening to attach new patch.
Comment 8 Philip Chimento 2021-10-06 13:35:17 PDT 
Created attachment 440433 [details]
Second patch
Comment 9 Philip Chimento 2021-10-06 13:36:28 PDT 
When I went to rebase my local branch I found that there were several tests that didn't make it in to the previous patch. Not sure what went wrong, I wonder if the patch was too long and they got cut off somehow! Here is a second patch with the remaining test262 tests.
Comment 10 EWS 2021-10-06 14:38:19 PDT 
Committed r283666 (242606@main): <https://commits.webkit.org/242606@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 440433 [details].