Add a function to convert key system into UUID

media/base/android/media_drm_bridge.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/android/media_drm_bridge.h"
 
 #include "base/logging.h"
 
 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
 bool MediaDrmBridge::IsAvailable() {
   return false;
 }
 
 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() {}
 
 bool MediaDrmBridge::GenerateKeyRequest(const std::string& type,
                                         const uint8* init_data,
                                         int init_data_length) {
   NOTIMPLEMENTED();
   return false;
 }
 
@@ skipping 23 matching lines @@
   NOTIMPLEMENTED();
 }
 
 void MediaDrmBridge::OnDrmEvent(
     JNIEnv* env, jobject j_media_drm, jstring session_id, jint event,
     jint extra, jstring data) {
   NOTIMPLEMENTED();
 }
 
 }  // namespace media