Report in devtools when preventDefault is called inside a forced passive event listener |
|||||
Issue descriptionWe should change the warning we report in devtools instead of the generic you've called preventDefault inside a passive event listener to something more descriptive indicating the UA is taking an action and a link to the blog post.
,
Aug 19 2016
,
Aug 25 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0e41c539d0f1d7cf0d4bcf425472ec6e7057d25a commit 0e41c539d0f1d7cf0d4bcf425472ec6e7057d25a Author: dtapuska <dtapuska@chromium.org> Date: Thu Aug 25 17:16:28 2016 Clarify devtools console log when a passive event listener is preventDefaulted. Add a reference to the chromestatus entry regarding document level passive event listeners. BUG= 627104 Review-Url: https://codereview.chromium.org/2272993004 Cr-Commit-Position: refs/heads/master@{#414462} [modify] https://crrev.com/0e41c539d0f1d7cf0d4bcf425472ec6e7057d25a/third_party/WebKit/Source/core/events/Event.cpp [modify] https://crrev.com/0e41c539d0f1d7cf0d4bcf425472ec6e7057d25a/third_party/WebKit/Source/core/events/Event.h [modify] https://crrev.com/0e41c539d0f1d7cf0d4bcf425472ec6e7057d25a/third_party/WebKit/Source/core/events/EventTarget.cpp [modify] https://crrev.com/0e41c539d0f1d7cf0d4bcf425472ec6e7057d25a/third_party/WebKit/Source/core/events/TouchEvent.cpp
,
Aug 29 2016
,
Aug 29 2016
Your change meets the bar and is auto-approved for M54 (branch: 2840)
,
Aug 29 2016
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by dtapu...@chromium.org
, Jul 11 2016