Re: [mediacapture-screen-share] Address nullability of DisplayMediaStreamOptions.controller (#236)

You generally shouldn't need nullable dictionary members to indicate missing values. Optional, i.e., undefined, is the right way to go about that. For new APIs we don't want undefined and null  to mean the same thing. That typically results in null throwing, just like any other unsupported type would.

Null can be reasonable if there's a getter somewhere that returns null and it's logical to pass the value of that getter into the API taking the dictionary member.

-- 
GitHub Notification of comment by annevk
Please view or discuss this issue at https://github.com/w3c/mediacapture-screen-share/issues/236#issuecomment-1263707793 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Friday, 30 September 2022 15:18:16 UTC