27283 2014-11-08 19:25:19 +0000 InvalidAccessError usage is questionable; use TypeError instead? 2015-10-19 23:48:09 +0000 1 1 1 Unclassified HTML WG Encrypted Media Extensions unspecified PC Windows NT RESOLVED MOVED external_dependency P2 normal --- 27284 1 d adrianba ddorwin jdsmith mike public-html-media public-html-bugzilla oldest_to_newest 114719 0 d 2014-11-08 19:25:19 +0000 "If keySystem is an empty string, return a promise rejected with a new DOMException whose name is InvalidAccessError. If supportedConfigurations was provided and is empty, return a promise rejected with a new DOMException whose name is InvalidAccessError." In general TypeErrors are preferred for invalid arguments. I think in general the existence of InvalidAccessError is a mistake from the days before it was realized that IDL was not a language-agnostic layer. (I will file a WebIDL bug to discuss.) No big deal though. 115085 1 jdsmith 2014-11-19 02:36:49 +0000 I've not found any definitive guidance on using TypeError for invalid argument values; but have found examples in both Media Source Extensions and Web Crypto where invalid values return TypeError. I recommend we adopt this practice for consistency with those specifications. 115793 2 d 2014-12-02 16:35:38 +0000 For the record, WebIDL bug filed at https://www.w3.org/Bugs/Public/show_bug.cgi?id=27284; no response as of yet. 115801 3 jdsmith 2014-12-02 19:12:49 +0000 I've added a comment to https://www.w3.org/Bugs/Public/show_bug.cgi?id=27284 to indicate EME has a related bug. 116019 4 ddorwin 2014-12-08 17:53:03 +0000 We also use InvalidAccessError for invalid formats and invalid operations for the session type. Would this change only apply to basic initial parameter checking (i.e. empty parameters)? 116020 5 d 2014-12-08 18:04:03 +0000 > We also use InvalidAccessError for invalid formats and invalid operations for the session type. I did a review of all the places InvalidAccessError appears. All of them seem related to invalid or unsanitizable objects appearing, and TypeError would fit, except > If this object's session type is not "persistent", return a promise rejected with a new DOMException whose name is InvalidAccessError. and > Otherwise > Reject promise with a new DOMException whose name is InvalidAccessError. (below 'If sessionType is "persistent"') which seems like they should be RangeError. 123792 6 ddorwin 2015-10-19 23:48:09 +0000 Migrated to https://github.com/w3c/encrypted-media/issues/103.