Separate MediaKeys interface from Decryptor interface.

webkit/media/crypto/ppapi_decryptor.cc

Index: webkit/media/crypto/ppapi_decryptor.cc
diff --git a/webkit/media/crypto/ppapi_decryptor.cc b/webkit/media/crypto/ppapi_decryptor.cc
index 7f45178ca6a88dc73a6f3d314debed7bfafd625e..8cad8a339bdd18f1d2d833ed0b13f672d5f4443d 100644
--- a/webkit/media/crypto/ppapi_decryptor.cc
+++ b/webkit/media/crypto/ppapi_decryptor.cc
@@ -264,7 +264,7 @@ void PpapiDecryptor::KeyAdded(const std::string& key_system,
 
 void PpapiDecryptor::KeyError(const std::string& key_system,

[ddorwin, 2013/05/24 20:39:12]

Why do we need a proxy for key_error_cb_? Is it just how plugin_cdm_delegate_
works? (I noticed that AesDecryptor does not have such a proxy.)

[xhwang, 2013/05/24 23:15:56]

We don't like to have callbacks jump layers, i.e. an object should not pass a
callback that belongs to another object to a third object. Passing key_error_cb_
directly in line 64 would violate this rule.

                               const std::string& session_id,
-                              media::Decryptor::KeyError error_code,
+                              media::MediaKeys::KeyError error_code,
                               int system_code) {
   DCHECK(render_loop_proxy_->BelongsToCurrentThread());
   key_error_cb_.Run(key_system, session_id, error_code, system_code);