Bug 198864 - Optimize `resolve` method lookup in Promise static methods
Summary: Optimize `resolve` method lookup in Promise static methods
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Minor
Assignee: Nobody
URL: https://github.com/tc39/ecma262/pull/...
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-06-14 13:18 PDT by Alexey Shvayka
Modified: 2019-06-19 17:39 PDT (History)
10 users (show)

See Also:


Attachments
Patch (7.69 KB, patch)
2019-06-14 13:23 PDT, Alexey Shvayka
no flags Details | Formatted Diff | Diff
Patch (7.57 KB, patch)
2019-06-19 15:45 PDT, Alexey Shvayka
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Alexey Shvayka 2019-06-14 13:23:37 PDT
Created attachment 372138 [details]
Patch
Comment 2 Yusuke Suzuki 2019-06-19 13:01:51 PDT
Comment on attachment 372138 [details]
Patch

r=me
Comment 3 WebKit Commit Bot 2019-06-19 13:03:57 PDT
Comment on attachment 372138 [details]
Patch

Rejecting attachment 372138 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'apply-attachment', '--no-update', '--non-interactive', 372138, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Logging in as commit-queue@webkit.org...
Fetching: https://bugs.webkit.org/attachment.cgi?id=372138&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=198864&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Processing patch 372138 from bug 198864.
Fetching: https://bugs.webkit.org/attachment.cgi?id=372138
Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Yusuke Suzuki']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Parsed 4 diffs from patch file(s).
patching file JSTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file JSTests/test262/expectations.yaml
Hunk #1 FAILED at 1080.
Hunk #2 succeeded at 1110 (offset 18 lines).
1 out of 2 hunks FAILED -- saving rejects to file JSTests/test262/expectations.yaml.rej
patching file Source/JavaScriptCore/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Source/JavaScriptCore/builtins/PromiseConstructor.js

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'Yusuke Suzuki']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: https://webkit-queues.webkit.org/results/12523463
Comment 4 Alexey Shvayka 2019-06-19 15:45:13 PDT
Created attachment 372496 [details]
Patch

Fix merge conflicts.
Comment 5 Joseph Pecoraro 2019-06-19 16:17:51 PDT
Nice!
Comment 6 Yusuke Suzuki 2019-06-19 16:55:09 PDT
Comment on attachment 372496 [details]
Patch

r=me
Comment 7 WebKit Commit Bot 2019-06-19 17:38:25 PDT
Comment on attachment 372496 [details]
Patch

Clearing flags on attachment: 372496

Committed r246620: <https://trac.webkit.org/changeset/246620>
Comment 8 WebKit Commit Bot 2019-06-19 17:38:27 PDT
All reviewed patches have been landed.  Closing bug.
Comment 9 Radar WebKit Bug Importer 2019-06-19 17:39:20 PDT
<rdar://problem/51922037>