public/browser
1
0
Fork 0
mirror of https://github.com/lightpanda-io/browser.git synced 2025-10-29 23:23:28 +00:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Explicit error if an AddEventListenerOption flag is set that we dont' support

Browse Source
This commit is contained in:
Karl Seguin
2025-05-15 13:09:16 +08:00
parent 903168b3a6
commit 206e34ac80
1 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/browser/dom/event_target.zig
View File

@@ -65,7 +65,18 @@ pub const EventTarget = struct {
if (opts_) |opts| { if (opts_) |opts| {
switch (opts) { switch (opts) {
.capture => |c| capture = c, .capture => |c| capture = c,
.opts => |o| capture = o.capture orelse false, .opts => |o| {
// Done this way so that, for common cases that _only_ set
// capture, i.e. {captrue: true}, it works.
// But for any case that sets any of the other flags, we
// error. If we don't error, this function call would succeed
// but the behavior might be wrong. At this point, it's
// better to be explicit and error.
if (o.once orelse false) return error.NotImplemented;
if (o.signal orelse false) return error.NotImplemented;
if (o.passive orelse false) return error.NotImplemented;
capture = o.capture orelse false;
},
} }
} }