Skip to content

Correct instantiation of HttpVideoSource with no content protection.#292

Merged
joeyparrish merged 1 commit intoshaka-project:masterfrom
sanbornhilland:master
Feb 27, 2016
Merged

Correct instantiation of HttpVideoSource with no content protection.#292
joeyparrish merged 1 commit intoshaka-project:masterfrom
sanbornhilland:master

Conversation

@sanbornhilland
Copy link
Contributor

Passing an empty config object is no longer acceptable because the
keySystem property is mandatory on drmConfig objects.

Also modified HttpVideoSource constructor comment so that the drmConfig
comments are the single source of truth regarding well-formed drmConfig
objects.

Passing an empty config object is no longer acceptable because the
keySystem property is mandatory on drmConfig objects.

Also modified HttpVideoSource constructor comment so that the drmConfig
comments are the single source of truth regarding well-formed drmConfig
objects.
@joeyparrish joeyparrish self-assigned this Feb 26, 2016
@joeyparrish
Copy link
Member

Looks good, but be warned: HttpVideoSource was originally used for testing purposes. We've deprecated it and no equivalent is planned for v2.

joeyparrish added a commit that referenced this pull request Feb 27, 2016
Correct instantiation of HttpVideoSource with no content protection.
@joeyparrish joeyparrish merged commit 96bf5b2 into shaka-project:master Feb 27, 2016
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Jul 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

status: archived Archived and locked; will not be updated

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants