[Bug 20376] Terminology feedback

https://www.w3.org/Bugs/Public/show_bug.cgi?id=20376

--- Comment #1 from Jussi Kalliokoski <jussi.kalliokoski@gmail.com> ---
(In reply to comment #0)
> The above needs to be a SHOULD.

Fixed: https://dvcs.w3.org/hg/audio/rev/bf4f6fd3ecef

> accepts should be "If the user gives express permission"

Fixed: https://dvcs.w3.org/hg/audio/rev/bf4f6fd3ecef

> All the above should really be in the algorithm or all this should be
> labelled as non-normative (i.e., this is a note of how it works
> conceptually, but can't be implemented).

Not fixed yet, not sure which is better.

> I think it' redundant for NavigatorMIDIAccessErrorCallback to be nullable
> when it's already optional.

Fixed: https://dvcs.w3.org/hg/audio/rev/bf4f6fd3ecef

> This is wrong. Error handling is handled by WebIDL. Having this here
> conflicts with WebIDL's behavior.

Fixed: https://dvcs.w3.org/hg/audio/rev/bf4f6fd3ecef

> Better would be to say, "if errorCallback is a callable". WebIDL would have
> caught other types already.

Fixed: https://dvcs.w3.org/hg/audio/rev/bf4f6fd3ecef

> Change this to DOMError (this is either a "securityError" or
> "NotSupportedError" depending on situation). Please don't introduce new
> Error types into the platform.
> 
> I have another email about this… but basically, you don't need
> NavigatorMIDIAccessError. Please delete it.

Done: https://dvcs.w3.org/hg/audio/rev/bf4f6fd3ecef

-- 
You are receiving this mail because:
You are the QA Contact for the bug.

Received on Thursday, 13 December 2012 15:35:40 UTC