Bug 235312

Summary: [JSC] Support import assertion syntax
Product: WebKit Reporter: Yusuke Suzuki <ysuzuki>
Component: New BugsAssignee: Yusuke Suzuki <ysuzuki>
Status: RESOLVED FIXED    
Severity: Normal CC: ews-watchlist, keith_miller, mark.lam, msaboff, ross.kirsling, saam, tzagallo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
ross.kirsling: review+, ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
Patch none

Description Yusuke Suzuki 2022-01-18 01:04:31 PST
[JSC] Support import assertion syntax
Comment 1 Yusuke Suzuki 2022-01-18 01:05:52 PST
Created attachment 449367 [details]
Patch
Comment 2 Ross Kirsling 2022-01-18 09:45:07 PST Comment hidden (obsolete)
Comment 3 Ross Kirsling 2022-01-18 09:45:08 PST
Comment on attachment 449367 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=449367&action=review

Code looks good overall, so r=me once EWS is all green.

> Source/JavaScriptCore/runtime/ScriptFetchParameters.h:32
> +enum class ImportDataType : uint8_t {

Are the changes to this file actually being used in this patch? If not, should they be postponed 'til the next one?
Comment 4 Yusuke Suzuki 2022-01-19 21:49:07 PST
Created attachment 449552 [details]
Patch
Comment 5 Yusuke Suzuki 2022-01-19 23:46:47 PST
Created attachment 449556 [details]
Patch
Comment 6 EWS 2022-01-24 14:52:28 PST
Committed r288473 (246356@main): <https://commits.webkit.org/246356@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 449556 [details].
Comment 7 Radar WebKit Bug Importer 2022-01-24 14:53:17 PST
<rdar://problem/87991797>