passive event listener console warning should link to chromestatus entry |
|||
Issue descriptionhttps://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/events/EventTarget.cpp?q=%22Added+non-passive+event+listener+to+a+scroll-blocking%22+package:%5Echromium$&l=245 There's no link to understand more when you see the violation.
,
Jun 1 2017
We have other warnings for this intervention that do include the chromestatus entry (and Dave and I have even taking a pass over GitHub and stackoverflow responding to comments about the warning).
,
Jun 7 2017
,
Jun 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e212ac9d8386240fc03f183f9e1fe4a08ea7414e commit e212ac9d8386240fc03f183f9e1fe4a08ea7414e Author: eirage <eirage@chromium.org> Date: Thu Jun 08 16:17:16 2017 Add chromestatus link to a warning passive event listener console warning link to chromestatus entry BUG= 728786 Review-Url: https://codereview.chromium.org/2926073003 Cr-Commit-Position: refs/heads/master@{#477986} [modify] https://crrev.com/e212ac9d8386240fc03f183f9e1fe4a08ea7414e/third_party/WebKit/Source/core/events/EventTarget.cpp
,
Jun 8 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by rbyers@chromium.org
, Jun 1 2017Owner: dtapu...@chromium.org