Add a function to convert key system into UUID

webkit/renderer/media/crypto/key_systems_unittest.cc

Index: webkit/renderer/media/crypto/key_systems_unittest.cc
diff --git a/webkit/renderer/media/crypto/key_systems_unittest.cc b/webkit/renderer/media/crypto/key_systems_unittest.cc
index efd318d79463a85d2d97af53aae431c742414924..4c1d197f24791472f1d8aeb6d30c7cd307c31986 100644
--- a/webkit/renderer/media/crypto/key_systems_unittest.cc
+++ b/webkit/renderer/media/crypto/key_systems_unittest.cc
@@ -753,4 +753,18 @@ TEST_F(KeySystemsTest, IsSupportedKeySystemWithMediaMimeType_Widevine_MP4) {
       "audio/mp4", vorbis_codec(), kWidevineAlpha));
 }
 
+#if defined(OS_ANDROID)
+TEST_F(KeySystemsTest, ConvertKeySystemToUUID) {
+  std::vector<uint8> uuid = GetUUID(kWidevineAlpha);
+  EXPECT_EQ(16u, uuid.size());

[ddorwin, 2013/06/20 22:20:15]

Do you care about the actual uuid? It seems we should maybe have at least one
value checked.

[qinmin, 2013/06/20 23:30:12]

Done.

+

[ddorwin, 2013/06/20 22:20:15]

You could break out unsupported systmes into a separate _Unrecognized test. That
has the same effect as the comment that was removed but in a better way.

[qinmin, 2013/06/20 23:30:12]

Done.

+  uuid = GetUUID(kWidevine);

[ddorwin, 2013/06/20 22:20:15]

These could all be collapsed into a single line. For example:
EXPECT_EQ(0u, GetUUID(kWidevine));

[qinmin, 2013/06/20 23:30:12]

Changed to EXPECT_TRUE(GetUUID(kWidevine).empty())

+  EXPECT_EQ(0u, uuid.size());
+  uuid = GetUUID(kClearKey);
+  EXPECT_EQ(0u, uuid.size());
+  uuid = GetUUID("");
+  EXPECT_EQ(0u, uuid.size());
+}
+#endif  // defined(OS_ANDROID)
+
 }  // namespace webkit_media