Re: [deviceorientation] Invalid IDL in DeviceMotionEventInit

> We can update the spec but implementations will likely need to keep accepting null values for these members because both [Blink](https://chromium.googlesource.com/chromium/src/+/1e6e30b0e19a1d2d2db6e6b7184d26c741959d78/third_party/blink/renderer/modules/device_orientation/device_motion_event_init.idl) and [Gecko](https://hg.mozilla.org/mozilla-central/raw-file/ebeba937ca2a01df86d98089f2368bb975182dcc/dom/webidl/DeviceMotionEvent.webidl) mark these as optional.

Every member of a dictionary that is not marked as required is already optional. The `?` really doesn't do anything since both "null" and "undefined" get converted to a dictionary the same way anyway. So getting rid of the `?` shouldn't have any behavioral effect (although it's hard to discuss what the behavior is of what is written now, since what is written isn't valid WebIDL to begin with).

-- 
GitHub Notification of comment by mkruisselbrink
Please view or discuss this issue at https://github.com/w3c/deviceorientation/issues/54#issuecomment-423672723 using your GitHub account

Received on Friday, 21 September 2018 21:15:20 UTC