Issue metadata
Sign in to add a comment
|
Support "once" event listener option
Reported by
christop...@gmx.com,
May 27 2016
|
||||||||||||||||||||||||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.54 Safari/537.36 Vivaldi/1.2.493.2 Steps to reproduce the problem: Add an event listener with option "once" set to true What is the expected behavior? https://dom.spec.whatwg.org/#dom-addeventlisteneroptions-once What went wrong? Multiple execution Did this work before? No Chrome version: 51.0.2704.54 Channel: stable OS Version: 10.0 Flash Version:
,
Jun 23 2016
,
Jul 13 2016
Hi guys, I could have a look if it is not taken already.
,
Jul 13 2016
we don't have an implementation yet. But it should be fairly straight forward. Webkit has implemented it and they had a couple of crashes around the lifecyle of the RegisteredEventListener that you'd need to watch out for.
,
Jul 19 2016
Thanks, will start working on it.
,
Aug 12 2016
Hi guys, I've done the first version of the patch, was delayed by my holidays. Plz check https://codereview.chromium.org/2245723002
,
Aug 22 2016
Assigning to reviewer since owner isn't yet a project member.
,
Aug 30 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/16d55b8e978e4ad50c858bc89f231dc568718da8 commit 16d55b8e978e4ad50c858bc89f231dc568718da8 Author: a.obzhirov <a.obzhirov@samsung.com> Date: Tue Aug 30 21:53:55 2016 Support "once" event listener option Based partially on WebKit patch 18bb5a9195ffb934a2678a8b8805411443b552a3 submitted by cdumez@apple.com. BUG= 615384 Review-Url: https://codereview.chromium.org/2245723002 Cr-Commit-Position: refs/heads/master@{#415446} [delete] https://crrev.com/c9d62a58d87453ae177a12dcf9fa9791243fd058/third_party/WebKit/LayoutTests/imported/wpt/dom/events/AddEventListenerOptions-once-expected.txt [modify] https://crrev.com/16d55b8e978e4ad50c858bc89f231dc568718da8/third_party/WebKit/Source/core/events/AddEventListenerOptions.idl [modify] https://crrev.com/16d55b8e978e4ad50c858bc89f231dc568718da8/third_party/WebKit/Source/core/events/EventTarget.cpp [modify] https://crrev.com/16d55b8e978e4ad50c858bc89f231dc568718da8/third_party/WebKit/Source/core/events/RegisteredEventListener.h
,
Aug 30 2016
,
Aug 31 2016
,
Sep 27 2016
|
|||||||||||||||||||||||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||||||||||||||||||||||
Comment 1 by alancutter@chromium.org
, May 30 2016Labels: -Type-Bug -Pri-2 Pri-3 Type-Feature
Status: Available (was: Unconfirmed)