Bug 210726 - [JSC] Enable BigInt
Summary: [JSC] Enable BigInt
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Yusuke Suzuki
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-04-19 17:06 PDT by Yusuke Suzuki
Modified: 2020-04-20 03:02 PDT (History)
11 users (show)

See Also:


Attachments
Patch (2.85 KB, patch)
2020-04-19 17:10 PDT, Yusuke Suzuki
mark.lam: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yusuke Suzuki 2020-04-19 17:06:18 PDT
[JSC] Enable BigInt
Comment 1 Yusuke Suzuki 2020-04-19 17:10:38 PDT
Created attachment 396930 [details]
Patch
Comment 2 Mark Lam 2020-04-19 17:12:24 PDT
Comment on attachment 396930 [details]
Patch

r=me
Comment 3 Yusuke Suzuki 2020-04-19 17:59:05 PDT
Committed r260345: <https://trac.webkit.org/changeset/260345>
Comment 4 Radar WebKit Bug Importer 2020-04-19 18:00:17 PDT
<rdar://problem/62035906>
Comment 5 Yusuke Suzuki 2020-04-19 19:17:20 PDT
https://bugs.webkit.org/show_bug.cgi?id=210728
Comment 6 Simon Fraser (smfr) 2020-04-19 21:54:04 PDT
This is causing test crashes: bug 210731.
Comment 7 Simon Fraser (smfr) 2020-04-19 21:54:30 PDT
Why was this landed with crashing tests in EWS?
Comment 8 Yusuke Suzuki 2020-04-20 02:59:23 PDT
(In reply to Simon Fraser (smfr) from comment #6)
> This is causing test crashes: bug 210731.

This is existing IDB bug. bug 210731 describes the details.
In bug 210731, we skipped this test. Here is analysis https://bugs.webkit.org/show_bug.cgi?id=210731#c3 and this is rationale. https://trac.webkit.org/changeset/260352/webkit
Comment 9 Yusuke Suzuki 2020-04-20 03:00:27 PDT
(In reply to Yusuke Suzuki from comment #8)
> (In reply to Simon Fraser (smfr) from comment #6)
> > This is causing test crashes: bug 210731.
> 
> This is existing IDB bug. bug 210731 describes the details.
> In bug 210731, we skipped this test. Here is analysis
> https://bugs.webkit.org/show_bug.cgi?id=210731#c3 and this is rationale.
> https://trac.webkit.org/changeset/260352/webkit

And tracked in https://bugs.webkit.org/show_bug.cgi?id=210735