Re: [presentation-api] One prompt for both PresentationRequest and RemotePlayback (#470)

This looks good to me, but we should be aware that there are two limitations:

1.  You can't pass in multiple PresentationRequests or RemotePlaybacks (unless we added an argument to startPresentation/startRemotePlayback and generalized prompt to a list of objects).  But this is probably fine for PresentationRequest and RemotePlayback since they have a list of URLs.

2.  We can't easily add new types of things to prompt for in the future, unless we generalize prompt () and maybe make a general supports(x) and start(x).  This will only matter if we think this will one day support more types of devices.

3.  This doesn't support knowing whether or not the device supports both audio and video.  Maybe it would be easy to extend that with a .supportsAudio and .supportsVideo on the SecondScreenDevice?  



-- 
GitHub Notification of comment by pthatcherg
Please view or discuss this issue at https://github.com/w3c/presentation-api/issues/470#issuecomment-528688233 using your GitHub account

Received on Friday, 6 September 2019 03:02:37 UTC