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

Initial draft of Open Screen requirements. #14

Merged
merged 4 commits into from Dec 23, 2016
Merged

Conversation

mfoltzgoogle
Copy link
Contributor

@mfoltzgoogle mfoltzgoogle commented Dec 21, 2016

This document addresses part of Issue #1 ([Meta] Write document describing protocol requirements). It has:

This is an initial draft based on some of the content used to develop the charter [1], but streamlined. Any feedback appreciated.

[1] https://github.com/webscreens/cg-charter/blob/gh-pages/requirements.md

Copy link
Contributor

@schien schien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to add "Presentation Resumption" as a functional requirement?

The Presentation API allows one browser (the *controlling user agent*, or
*controller*) to discover and initiate presentation of Web content on a second
user agent (the *receiving user agent*, or *receiver*). We use the term
*display* to refer the entire device responsible for implementing the
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/refer the/refer to the/

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

*controller*) to discover and initiate presentation of Web content on a second
user agent (the *receiving user agent*, or *receiver*). We use the term
*display* to refer the entire device responsible for implementing the
*receiver*, including browser, OS, networking and display.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/display/screen (looks a bit circular otherwise)

It's a nice definition. Actually, I wonder if we could update the definition of presentation display in the Presentation API to clarify that it refers to "the entire device responsible for implementing the receiver, including browser, OS, networking and screen". Not a big deal, the Presentation API introduction is clear about what constitutes a display in any case.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

I can check the definition of presentation display in the spec when I take on w3c/presentation-api#345.

1. Messages sent by the controller must be delivered to the receiver (or vice
versa) in a reliable and in-order fashion.

2. If a message cannot be delivered, then the controller must be able signal the
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/able signal/able to signal

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

network transport to the display.

3. A controller must be able to determine if the receiver is capable of
displaying a specific presentation request URL.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I note the Presentation API says "reasonably guarantee that the presentation of the URL on that display will succeed". Here, "capable" seems stronger. There may be no way to tell whether a specific presentation request URL can be displayed on top of "yes, I support HTTP" in some cases. Add "reasonably" as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done - alignment with the spec here should be done as much as possible. Thanks for checking.

@mfoltzgoogle
Copy link
Contributor Author

@schien I broke out the Connections section into separate requirements for initiation, resumption, and other connection requirements. Let me know if that still needs improvement.

@mfoltzgoogle
Copy link
Contributor Author

Merging, can address additional comments in follow up commits.

@mfoltzgoogle mfoltzgoogle merged commit 7f47d5c into gh-pages Dec 23, 2016
@mfoltzgoogle mfoltzgoogle deleted the issue-2-reqs branch December 23, 2016 19:48
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

Successfully merging this pull request may close these issues.

None yet

3 participants