[go: nahoru, domu]

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop step 6 of getAvailability for consistency with start? #383

Closed
tidoust opened this issue Dec 1, 2016 · 1 comment
Closed

Drop step 6 of getAvailability for consistency with start? #383

tidoust opened this issue Dec 1, 2016 · 1 comment

Comments

@tidoust
Copy link
Member
tidoust commented Dec 1, 2016

That did not strike me as odd before, but why we do we need to call out the unability to "monitor the list of available presentation displays for the entire duration of the controlling browsing context" in step 6. of the getAvailability algorithm?

This concept does not exist in the start algorithm. In start, I think this is captured by the fact that the algorithm to monitor the list of available presentation displays will simply create an empty list of available presentation displays.

Following the same logic in getAvailability, given an empty list of available presentation displays, step 9. correctly returns a PresentationAvailability object whose value is set to false if the list of available presentation displays. In other words, step 6. seems useless to me.

A user agent may let users disable the ability to monitor the list of available presentation displays at any time. If that happens and there's already a list of available presentation displays, I would expect the user agent to run the monitoring algorithm once again (with an empty list of presentation displays) in the background. I'm not sure this needs to be made explicit in the spec given that we leave it up to implementers to run the monitoring algorithm again whenever they want, including "never" if they so choose.

@mfoltzgoogle
Copy link
Contributor
mfoltzgoogle commented Dec 1, 2016

On removing step 6: That seems fine, but the step is not useless. I would want to adjust the language in the "monitor" steps to set the list of displays to empty and abort if monitoring is disabled, effectively moving the step to a different list of steps. That seems like an editorial improvement.

On disabling monitoring: From a user experience point of view, agreed it would be better to run the monitoring algorithm with an empty list, so that existing PresentationAvailability objects get a change event. Otherwise, a page may believe there are available display, and its attempt to start presentation would reject with NotFoundError.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants