Commit c93f357
committed
deps: V8: cherry-pick fa4cb172cde2
Original commit message:
[runtime] Fix Promise.all context promise hooks
We have to take the slow path in Promise.all if context promise hooks
are set. The fast-path doesn't create intermediate promises by default.
Bug: chromium:1204132, v8:11025
Change-Id: Ide92de00a4f6df05e0ddbc8814f6673bd667f426
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2866771
Reviewed-by: Victor Gomes <victorgomes@chromium.org>
Commit-Queue: Camillo Bruni <cbruni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#74326}
Refs: v8/v8@fa4cb17
PR-URL: #38273
Backport-PR-URL: #38991
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Mary Marchini <oss@mmarchini.me>1 parent 3c6c28b commit c93f357Copy full SHA for c93f357
File tree
Expand file treeCollapse file tree
3 files changed
+14
-4
lines changedOpen diff view settings
Filter options
- deps/v8
- src/builtins
- test/mjsunit
Expand file treeCollapse file tree
3 files changed
+14
-4
lines changedOpen diff view settings
Collapse file
+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
36 | 36 | |
37 | 37 | |
38 | 38 | |
39 | | - |
| 39 | + |
40 | 40 | |
41 | 41 | |
42 | 42 | |
|
Collapse file
deps/v8/src/builtins/promise-all.tq
Copy file name to clipboardExpand all lines: deps/v8/src/builtins/promise-all.tq+1-2Lines changed: 1 addition & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
231 | 231 | |
232 | 232 | |
233 | 233 | |
234 | | - |
235 | | - |
| 234 | + |
236 | 235 | |
237 | 236 | |
238 | 237 | |
|
Collapse file
deps/v8/test/mjsunit/promise-hooks.js
Copy file name to clipboardExpand all lines: deps/v8/test/mjsunit/promise-hooks.js+12-1Lines changed: 12 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
246 | 246 | |
247 | 247 | |
248 | 248 | |
249 | | - |
| 249 | + |
250 | 250 | |
251 | 251 | |
252 | 252 | |
| ||
262 | 262 | |
263 | 263 | |
264 | 264 | |
| 265 | + |
| 266 | + |
| 267 | + |
| 268 | + |
| 269 | + |
| 270 | + |
| 271 | + |
| 272 | + |
| 273 | + |
| 274 | + |
| 275 | + |
0 commit comments