Implement robustness strings in requestMediaKeySystemAccess().

media/base/eme_constants.h

Index: media/base/eme_constants.h
diff --git a/media/base/eme_constants.h b/media/base/eme_constants.h
index 4462c5cc0b251686c30bb1ef99b5a716248d7710..f2124a04503212119a629d04c8a65b8dea2619ed 100644
--- a/media/base/eme_constants.h
+++ b/media/base/eme_constants.h
@@ -60,6 +60,7 @@ enum EmeSessionTypeSupport {
   EME_SESSION_TYPE_SUPPORTED,
 };
 
+// Used to declare support for distinctive identifier and persistent state.
 enum EmeFeatureSupport {
   // Invalid default value.
   EME_FEATURE_INVALID,
@@ -74,12 +75,46 @@ enum EmeFeatureSupport {
   EME_FEATURE_ALWAYS_ENABLED,
 };
 
+// Used to query support for distinctive identifier and persistent state.
 enum EmeFeatureRequirement {
   EME_FEATURE_NOT_ALLOWED,
   EME_FEATURE_OPTIONAL,
   EME_FEATURE_REQUIRED,
 };
 
+enum class EmeMediaType {
+  AUDIO,
+  VIDEO,
+};
+
+// Robustness values understood by KeySystems. Each value imples all those
+// before it. (This may need to be redesigned to support non-Widevine CDMs.)
+enum class EmeRobustness {
+  INVALID,
+  EMPTY,
+  SW_SECURE_CRYPTO,
+  SW_SECURE_DECODE,
+  HW_SECURE_CRYPTO,
+  HW_SECURE_DECODE,
+  HW_SECURE_ALL,
+};
+
+// Configuration rules are recipes for determining if a feature is supported,

[ddorwin, 2015/03/16 23:25:08]

Not really "recipes"?

[sandersd (OOO until July 31), 2015/03/17 22:10:38]

Done.

+// and for updating configuration state for evaluating later rules. They are
+// used to answer queries about distinctive identifier, persistent state, and
+// robustness support.
+enum class EmeConfigRule {

[ddorwin, 2015/03/16 23:25:08]

"Rule" doesn't seem appropriate. SupportStatus? SupportCondition?

[sandersd (OOO until July 31), 2015/03/17 22:10:38]

Condition is accurate. Let me know if it still needs to be changed now that the
comments are improved.

+  // The feature is not supported.
+  NOT_SUPPORTED,
+  // The feature is supported if the encrypted media permission is granted.
+  PERMISSION_REQUIRED,
+  // The feature is supported, but the user experience may be improved if the
+  // encrypted media permission is granted.
+  PERMISSION_RECOMMENDED,
+  // The feature is supported without conditions.
+  SUPPORTED,

[ddorwin, 2015/03/16 23:25:08]

This does not indicate whether it can be disabled. Perhaps the questions are
"does support not using" to which SUPPORTED would be returned. If so, then the
comment should probably be updated - "feature" can also be not doing something.

[sandersd (OOO until July 31), 2015/03/17 22:10:38]

Done.

+};
+
 }  // namespace media
 
 #endif  // MEDIA_BASE_EME_CONSTANTS_H_