cdm: Fix comment on DecryptAndDecodeFrame()

kEmptyVideoFrame doesn't actually exist and the comment is wrong. I
checked the history and this has been the case since 2012. This CL fixes
the comment.

Bug: 854943
Change-Id: Id825a101fb4a9e8340fbc91366c5682595fa7c84
Reviewed-on: https://chromium-review.googlesource.com/1246687
Reviewed-by: Rintaro Kuroiwa <rkuroiwa@chromium.org>
1 file changed
tree: 8cd7b4977d7766d47fce5fcaf41535798474dd0d
  1. codereview.settings
  2. content_decryption_module.h
  3. content_decryption_module_export.h
  4. content_decryption_module_ext.h
  5. content_decryption_module_proxy.h
  6. OWNERS
  7. README.md
README.md

Library CDM Interface

This directory contains files that define the shared library interface between an Encrypted Media Extensions (EME) Content Decryption Module (CDM) and Chromium or other user agents. It is used to build both user agents and CDMs.

This is also referred to as the “CDM interface” in the context of library CDM and in this doc.

TODO(xhwang): Add more sections describing the CDM interface.

Experimental and Stable CDM interface

A new CDM interface that‘s still under development is subject to change. This is called an “experimental CDM interface”. To avoid compatibility issues, a user agent should not support an experimental CDM interface by default (it’s okay to support it behind a flag). Similarly, a CDM vendor should not ship a CDM using an experimental CDM interface to end users.

The experimental status of a CDM interface ends when the development is complete and the CDM interface is marked as stable. A stable CDM interface defines an application binary interface (ABI) between the CDM and the host, which should never be changed. Some minor ABI-compatible changes can still be made on a stable CDM interface to improve readability or to avoid unnecessary complexity. For example:

  • Adding clarifying comments.
  • Extending an existing enum type with new values, without changing the enum-base.

Note that even though these changes are ABI-compatible, the implementation may still need to be updated, e.g. to handle the new enum type in a switch case block.

On newer CDM interfaces, a static boolean member kIsStable is present to indicate whether the CDM interface is stable or experimental.