Extension's event handlers not firing when Javascript is disabled.
Reported by
smbl...@gmail.com,
Apr 27 2016
|
||
Issue descriptionUserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.108 Safari/537.36 Steps to reproduce the problem: 1. Go to any old page and disable Javascript. 2. Use any extension which installs keyboard event handlers. 3. I can provide a sample, if needed. What is the expected behavior? The extension's event handlers should fire in response to keyboard events. What went wrong? The event handlers do not fire. WebStore page: https://chrome.google.com/webstore/detail/vimium/dbepggeogbaibhgnhhndojpepiihcmeb?hl=en Did this work before? No Chrome version: 49.0.2623.108 Channel: stable OS Version: Debian Flash Version: Shockwave Flash 21.0 r0 This is related to chrome bug #90843 (https://bugs.chromium.org/p/chromium/issues/detail?id=90843). There, there is discussion of relationship between "disabling Jacascript" on a page and its effect on extensions, and the conclusion is that extension's JS should nevertheless run. This appears not to be the case for keyboard (and probably other) event handlers. This breaks extensions like Vimium. The Vimium issue is here: https://github.com/philc/vimium/issues/753.
,
May 3 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/866d1237c72059624def2242e218a7dfe78b125e commit 866d1237c72059624def2242e218a7dfe78b125e Author: jochen <jochen@chromium.org> Date: Tue May 03 11:08:14 2016 Dispatch events in isolated worlds, even if script execution is forbidden BUG=608286, 607072 R=mkwst@chromium.org Review-Url: https://codereview.chromium.org/1939713003 Cr-Commit-Position: refs/heads/master@{#391197} [modify] https://crrev.com/866d1237c72059624def2242e218a7dfe78b125e/components/test_runner/mock_content_settings_client.cc [modify] https://crrev.com/866d1237c72059624def2242e218a7dfe78b125e/components/test_runner/mock_content_settings_client.h [add] https://crrev.com/866d1237c72059624def2242e218a7dfe78b125e/third_party/WebKit/LayoutTests/fast/events/events-in-isolated-world-expected.txt [add] https://crrev.com/866d1237c72059624def2242e218a7dfe78b125e/third_party/WebKit/LayoutTests/fast/events/events-in-isolated-world.html [modify] https://crrev.com/866d1237c72059624def2242e218a7dfe78b125e/third_party/WebKit/LayoutTests/permissionclient/script-permissions-expected.txt [modify] https://crrev.com/866d1237c72059624def2242e218a7dfe78b125e/third_party/WebKit/Source/bindings/core/v8/V8EventListener.cpp
,
May 3 2016
|
||
►
Sign in to add a comment |
||
Comment 1 by rdevlin....@chromium.org
, Apr 29 2016Owner: jochen@chromium.org
Status: Assigned (was: Unconfirmed)