Separate CDM initialization from GenerateKeyRequest & remove key_system parameters.

content/common/media/media_player_messages_android.h

Index: content/common/media/media_player_messages_android.h
diff --git a/content/common/media/media_player_messages_android.h b/content/common/media/media_player_messages_android.h
index 47146eec197c04e73a19280fd6cadc5874fb487a..efc313c47bbcc848e67f57efa2fd46d6df6a0796 100644
--- a/content/common/media/media_player_messages_android.h
+++ b/content/common/media/media_player_messages_android.h
@@ -215,39 +215,33 @@ IPC_MESSAGE_ROUTED3(MediaPlayerHostMsg_NotifyExternalSurface,
 
 // Messages for encrypted media extensions API ------------------------------
 
-IPC_MESSAGE_ROUTED4(MediaPlayerHostMsg_GenerateKeyRequest,
+IPC_MESSAGE_ROUTED3(MediaPlayerHostMsg_GenerateKeyRequest,
                     int /* player_id */,
-                    std::string /* key_system */,

[qinmin, 2013/06/18 17:01:00]

If no key_system in GenerateKeyRequest, do we need a separate message for it?
I am wondering how to support systems other than widewine. I am ok removing it
from the rest of the messages, but we still need to tell the DRM bridge sth
about the key_system

[ddorwin, 2013/06/18 17:34:06]

The key system should be specified before generating a key request. This is how
the new EME API works and can be made compatible with the existing one as I have
done for desktop. We'll need to add an Initialize message similar to
https://codereview.chromium.org/17289006/diff/53001/webkit/plugins/ppapi/cont....
Since I was not working on an Android workspace and the bridge is not yet
implemented, I left that for a future CL.

                     std::string /* type */,
                     std::vector<uint8> /* init_data */)
 
-IPC_MESSAGE_ROUTED5(MediaPlayerHostMsg_AddKey,
+IPC_MESSAGE_ROUTED4(MediaPlayerHostMsg_AddKey,
                     int /* player_id */,
-                    std::string /* key_system */,
                     std::vector<uint8> /* key */,
                     std::vector<uint8> /* init_data */,
                     std::string /* session_id */)
 
-IPC_MESSAGE_ROUTED3(MediaPlayerHostMsg_CancelKeyRequest,
+IPC_MESSAGE_ROUTED2(MediaPlayerHostMsg_CancelKeyRequest,
                     int /* player_id */,
-                    std::string /* key_system */,
                     std::string /* session_id */)
 
-IPC_MESSAGE_ROUTED3(MediaPlayerMsg_KeyAdded,
+IPC_MESSAGE_ROUTED2(MediaPlayerMsg_KeyAdded,
                     int /* player_id */,
-                    std::string /* key_system */,
                     std::string /* session_id */)
 
-IPC_MESSAGE_ROUTED5(MediaPlayerMsg_KeyError,
+IPC_MESSAGE_ROUTED4(MediaPlayerMsg_KeyError,
                     int /* player_id */,
-                    std::string /* key_system */,
                     std::string /* session_id */,
                     media::MediaKeys::KeyError /* error_code */,
                     int /* system_code */)
 
-IPC_MESSAGE_ROUTED5(MediaPlayerMsg_KeyMessage,
+IPC_MESSAGE_ROUTED4(MediaPlayerMsg_KeyMessage,
                     int /* player_id */,
-                    std::string /* key_system */,
                     std::string /* session_id */,
                     std::string /* message */,
                     std::string /* destination_url */)