Add a function to convert key system into UUID

media/base/android/media_drm_bridge.cc

Index: media/base/android/media_drm_bridge.cc
diff --git a/media/base/android/media_drm_bridge.cc b/media/base/android/media_drm_bridge.cc
index 8760900a8879af28953035571011e4d7ce01dec1..b746d392717bc96a6d39ac8e38525990b711c46a 100644
--- a/media/base/android/media_drm_bridge.cc
+++ b/media/base/android/media_drm_bridge.cc
@@ -8,6 +8,27 @@
 
 using base::android::ScopedJavaLocalRef;
 
+namespace {
+
+// TODO(qinmin): figure out what is the UUID for com.widevine.alpha, and also
+// other key systems.
+const uint8 kWideVineUUID[] = {
+  0xED, 0xEF, 0x8B, 0xA9, 0x79, 0xD6, 0x4A, 0xCE, // Widevine UUID
+  0xA3, 0xC8, 0x27, 0xDC, 0xD5, 0x1D, 0x21, 0xED,
+};
+
+static std::vector<uint8> ConvertKeySystemToUUID(
+    const std::string& key_system) {
+  std::vector<uint8> uuid;
+  if (0 == key_system.compare("com.widevine"))

[xhwang, 2013/06/19 21:30:41]

Why not just if (key_system == "com.widevine") ? This could end up creating a
string for "com.widevine" but I feel it's more intuitive. WDYT?

[qinmin, 2013/06/19 21:56:05]

Done.

+    uuid.insert(uuid.end(), kWideVineUUID, kWideVineUUID + 16);

[xhwang, 2013/06/19 21:30:41]

uuid.assign(kWideVineUUID, kWideVineUUID + 16) ? We never need to append to an
existing UUID.

[qinmin, 2013/06/19 21:56:05]

Done.

+  else
+    NOTIMPLEMENTED();
+  return uuid;
+}
+
+}  // namespace
+
 namespace media {
 
 // static
@@ -16,8 +37,11 @@ bool MediaDrmBridge::IsAvailable() {
 }
 
 MediaDrmBridge::MediaDrmBridge(
-    int media_keys_id, const std::vector<uint8>& uuid)
-    : media_keys_id_(media_keys_id) {}
+    int media_keys_id, const std::string& key_system)
+    : media_keys_id_(media_keys_id) {
+  // TODO(qinmin): pass the uuid to DRM engine.
+  std::vector<uint8> uuid = ConvertKeySystemToUUID(key_system);

[xhwang, 2013/06/19 21:30:41]

MediaDrmBridge should probably keep a reference to the uuid so that when we
parse the pssh box, we can compare the SystemId with the current UUID and only
call GRK with the Data associated with the current/selected |uuid_|.

[qinmin, 2013/06/19 21:56:05]

added a uuid_ member variable.

On 2013/06/19 21:30:41, xhwang wrote:

+}
 
 MediaDrmBridge::~MediaDrmBridge() {}