Closed
Bug 1493421
Opened 6 years ago
Closed 6 years ago
[wpt-sync] Sync PR 13166 - Reland "bindings: Implement timers with V8Function"
Categories
(Core :: DOM: Core & HTML, enhancement, P4)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox65 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 13166 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/13166
Details from upstream follow.
Timothy Gu <timothygu@chromium.org> wrote:
> Reland "bindings: Implement timers with V8Function"
>
> This is a reland of 254369a5f6df06c2c6be067d14c2cb2a036ba173
>
> Original change's description:
> > bindings: Implement timers with V8Function
> >
> > This fixes bug 866610 by using the IDL infrastructure to properly enter
> > the v8::Context before calling the registered callback.
> >
> > Also ensure eager finalization of ScheduledAction in DOMTimer to
> > prevent a memory leak. Added two more effective DCHECKs to confirm.
> >
> > Bug: 866610
> > Change-Id: I37d7bd05f035fe31856cfe68bae51aa0632cd3b1
> > Reviewed-on: https://chromium-review.googlesource.com/1220486
> > Reviewed-by: Nate Chapin \<japhet@chromium.org>
> > Reviewed-by: Yuki Shiino \<yukishiino@chromium.org>
> > Reviewed-by: Hitoshi Yoshida \<peria@chromium.org>
> > Commit-Queue: Timothy Gu \<timothygu@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#593108}
>
> Bug: 866610
> Change-Id: Iee5c1d6917ad7770383e06a425f96000835a663a
>
> Reviewed-on: https://chromium-review.googlesource.com/1239624
> WPT-Export-Revision: 68e4eca995d2880d3dca9e0dd5d88e7de11665f9
Assignee | ||
Updated•6 years ago
|
Component: web-platform-tests → DOM
Product: Testing → Core
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=1f93ccd88f2060e263f7bb0aa2618ad106837095
Assignee | ||
Updated•6 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Assignee | ||
Updated•6 years ago
|
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Assignee | ||
Comment 3•6 years ago
|
||
Assignee | ||
Comment 4•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=525da9f2bdddedf9f2ba458b2626a07e6d3c04b9
Assignee | ||
Comment 5•6 years ago
|
||
Assignee | ||
Comment 6•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=e78e26c858ca127783be9d1bf741bc642e7196e3
Assignee | ||
Comment 7•6 years ago
|
||
Assignee | ||
Comment 8•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=2bd16a07cb924e541102ba66d7a8b32248f8efa2
Assignee | ||
Comment 9•6 years ago
|
||
Ran 1 tests and 1 subtests
OK : 1
PASS : 1
Comment 10•6 years ago
|
||
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d8f6cb92b3a9
[wpt PR 13166] - Reland "bindings: Implement timers with V8Function", a=testonly
Comment 11•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•