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

Add pixelAspectRatio #2294

Merged
merged 4 commits into from
Jan 6, 2020
Merged

Conversation

avelad
Copy link
Member

@avelad avelad commented Dec 11, 2019

Parse pixelAspectRatio in DASH and expose it

Related to #2290

@avelad
Copy link
Member Author

avelad commented Dec 17, 2019

@theodab can you review it?

@avelad
Copy link
Member Author

avelad commented Dec 30, 2019

@theodab I have fixed the problem, it is ready to be reviewed again.

Copy link
Member

@joeyparrish joeyparrish left a comment

Choose a reason for hiding this comment

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

Thanks!

test/offline/manifest_convert_unit.js Outdated Show resolved Hide resolved
lib/offline/indexeddb/v1_storage_cell.js Outdated Show resolved Hide resolved
@avelad
Copy link
Member Author

avelad commented Jan 6, 2020

@joeyparrish Can you review it again?

Copy link
Member

@joeyparrish joeyparrish left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks!

@shaka-bot
Copy link
Collaborator

All tests passed!

@joeyparrish joeyparrish merged commit bdca05f into shaka-project:master Jan 6, 2020
@avelad avelad deleted the pixelAspectRatio branch January 7, 2020 10:28
shaka-bot pushed a commit that referenced this pull request Jan 15, 2020
For backward compatibility, the new pixelAspectRatio field in Stream
should be string|undefined rather than being nullable.  This ensures
that manifest parser plugins that do not output that field will still
compile against the externs.

PR #2294

Change-Id: I1aae03994a213c8ce52dc64e8a34bf179045f4fb
joeyparrish pushed a commit that referenced this pull request Jan 15, 2020
Backported to v2.5.x

Change-Id: I413905c15ba5927cdd01e103c5f65070f06f4afa
joeyparrish added a commit that referenced this pull request Jan 15, 2020
For backward compatibility, the new pixelAspectRatio field in Stream
should be string|undefined rather than being nullable.  This ensures
that manifest parser plugins that do not output that field will still
compile against the externs.

PR #2294

Backported to v2.5.x

Change-Id: I1aae03994a213c8ce52dc64e8a34bf179045f4fb
@joeyparrish
Copy link
Member

This PR has been cherry-picked for v2.5.8.

@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.

4 participants