smartdevicelink / sdl_evolution

Tracking and proposing changes to SDL's public APIs.
https://smartdevicelink.github.io/sdl_evolution/
BSD 3-Clause "New" or "Revised" License
33 stars 122 forks source link

Revise SDL-0332 Additional Video Streaming Capabilities Validation #1146

Closed Jack-Byrne closed 3 years ago

Jack-Byrne commented 3 years ago

Introduction

This is a revision to an accepted, but not yet implemented proposal.

Motivation

The review process of the original proposal is summarized here: https://github.com/smartdevicelink/sdl_evolution/issues/1139#issuecomment-843303438

Proposed Solution

Suggested Revisions:

  1. Add the following validation requirement "A unique screen size is determined by the following parameters: preferredResolution, diagonalScreenSize, pixelPerInch, and scale. Each given screen size should provide one and only one preferredFPS, maxBitrate, supportedFormats, and hapticSpatialDataSupported.

  2. Add example response and parameter details for OnAppCapability.

Author's Updates:

Potential Downsides

The author does not foresee any potential downsides.

Impact On Existing Code

No impact to code outside of original proposal. All revisions are for providing improved readability.

Jack-Byrne commented 3 years ago

@theresalech Ready for review.