Add video decoding methods in Decryptor and add DecryptingVideoDecoder.

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 ec19cc1d56110f652ce2be33835d368ea3f4cbb1..a07087fb682adc73ad0e7a813407fa15e78eef8e 100644
--- a/webkit/media/crypto/ppapi_decryptor.cc
+++ b/webkit/media/crypto/ppapi_decryptor.cc
@@ -13,6 +13,8 @@
 #include "base/message_loop_proxy.h"
 #include "media/base/decoder_buffer.h"
 #include "media/base/decryptor_client.h"
+#include "media/base/video_decoder_config.h"

[ddorwin, 2012/09/21 00:36:08]

These don't seem necessary.

[xhwang, 2012/09/25 23:52:32]

Surprise! After I removed these two everything still build. 

However, VideoDecoderConfig and VideoFrame are all forward declared in header
files. Everything still build because VideoDecoderConfig and VideoFrame are
declared indirectly through "webkit/plugins/ppapi/ppapi_plugin_instance.h".

By IWYU, I think we should include them explicitly.

[ddorwin, 2012/09/26 02:34:27]

We use VideoDecodeCB and DecoderInitCB, but not VideoDecoderConfig (just a const
ref). So, we don't need definitions for everything. I'd have to look at where
those two CB's are defined, though, to see what we need to include.

+#include "media/base/video_frame.h"
 #include "webkit/media/crypto/key_systems.h"
 #include "webkit/plugins/ppapi/ppapi_plugin_instance.h"
 
@@ -100,6 +102,28 @@ void PpapiDecryptor::Decrypt(
 void PpapiDecryptor::Stop() {
 }
 
+void PpapiDecryptor::InitializeVideoDecoder(
+    const media::VideoDecoderConfig& config,
+    const InitializeCB& init_cb) {
+  NOTIMPLEMENTED();

[ddorwin, 2012/09/21 00:36:08]

TODO(xhwang): Impelement.?
It seems these are intended to be whereas the AesDecryptor ones are not.

[xhwang, 2012/09/25 23:52:32]

Added comments.

+  init_cb.Run(false);
+}
+
+void PpapiDecryptor::DecryptAndDecodeVideo(
+    const scoped_refptr<media::DecoderBuffer>& encrypted,
+    const VideoDecodeCB& video_decode_cb) {
+  NOTIMPLEMENTED();
+  video_decode_cb.Run(kError, NULL);
+}
+
+void PpapiDecryptor::ResetVideoDecoder() {
+  NOTIMPLEMENTED();
+}
+
+void PpapiDecryptor::StopVideoDecoder() {
+  NOTIMPLEMENTED();
+}
+
 void PpapiDecryptor::ReportFailureToCallPlugin(const std::string& key_system,
                                                const std::string& session_id) {
   DVLOG(1) << "Failed to call plugin.";