-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Add pixelAspectRatio #2294
Conversation
@theodab can you review it? |
@theodab I have fixed the problem, it is ready to be reviewed again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@joeyparrish Can you review it again? |
There was a problem hiding this 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!
All tests passed! |
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
Backported to v2.5.x Change-Id: I413905c15ba5927cdd01e103c5f65070f06f4afa
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
This PR has been cherry-picked for v2.5.8. |
Parse pixelAspectRatio in DASH and expose it
Related to #2290