Skip to content

Move the scripting is disabled check for event handlers to compile time - #12944

Open
annevk wants to merge 1 commit into
mainfrom
event-handler-scripting-disabled
Open

annevk wants to merge 1 commit into
mainfrom
event-handler-scripting-disabled

Conversation

@annevk

@annevk annevk commented Sep 15, 2026

Copy link
Copy Markdown
Member

Move the scripting is disabled check for event handlers to compile time

Implementations block compiling an internal raw uncompiled handler, not
invoking an event handler whose value is already a function.

Also skip an event handler whose callback's global's associated Document is not
fully active, matching DOM.

DOM PR: whatwg/dom#1085

(See WHATWG Working Mode: Changes for more details.)


/webappapis.html ( diff )

Implementations block compiling an internal raw uncompiled handler, not
invoking an event handler whose value is already a function.

Also skip an event handler whose callback's global's associated Document is not
fully active, matching DOM.

DOM PR: whatwg/dom#1085
annevk added a commit to shaseley/dom that referenced this pull request Sep 15, 2026
Adopting a node does not change the realm it was created in, so for nodes the
node's relevant global object and its node document's relevant global object
can differ. Implementations go by the node document, and recompute it per
listener as a listener can adopt event's currentTarget.

Implementations also skip an event listener whose callback comes from a Window
whose associated Document is not fully active, even when the event target
itself is fully active. This does not cover event handlers, whose event
listener callback is supplied by HTML rather than by the author.

Tests: web-platform-tests/wpt#62719
HTML PR: whatwg/html#12944
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant