[wpt-sync] Sync PR 42017 - LoAF: Expose script URL for promise resolvers
Categories
(Testing :: web-platform-tests, task, P4)
Tracking
(firefox120 fixed)
| Tracking | Status | |
|---|---|---|
| firefox120 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 42017 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/42017
Details from upstream follow.
Noam Rosenthal <nrosenthal@chromium.org> wrote:
LoAF: Expose script URL for promise resolvers
Capture the current script URL when constructing a
promise resolver (e.g. calling fetch()), and read
it when the promise resolver is invoked (e.g.
the response is being processed).Confirmed with the V8 team that this should (in theory)
have acceptable overhead, as GetCurrentScriptUrl() is
called only in places that are "heavy" to begin with,
like invoking a new fetch or calling a platform API.Change-Id: I2c81a3694849bd73f0986f19f3de752093f6d7b0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4802170
Reviewed-by: Yoav Weiss \<yoavweiss@chromium.org>
Commit-Queue: Noam Rosenthal \<nrosenthal@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1197775}
| Assignee | ||
Comment 1•2 years ago
|
||
| Assignee | ||
Comment 2•2 years ago
|
||
CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 1 tests and 4 subtests
Status Summary
Firefox
ERROR: 1
Chrome
OK : 1
PASS : 3
FAIL : 1
Safari
ERROR: 1
Links
Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base
Details
New Tests That Don't Pass
- /long-animation-frame/tentative/loaf-source-location.html [wpt.fyi]:
ERROR(Chrome:OK, Safari:ERROR)
Comment 4•2 years ago
|
||
| bugherder | ||
Description
•