Encrypted Media: No NeedKey if --enable-encrypted-media is not set.

webkit/media/filter_helpers.cc

Index: webkit/media/filter_helpers.cc
diff --git a/webkit/media/filter_helpers.cc b/webkit/media/filter_helpers.cc
index b7080930c4d7235a67220faf922f70ee23b13a63..379e2caad4a2cf7c31ff5b2cd825a747c2e8fc61 100644
--- a/webkit/media/filter_helpers.cc
+++ b/webkit/media/filter_helpers.cc
@@ -32,32 +32,32 @@ static void AddDefaultDecodersToCollection(
     ProxyDecryptor* proxy_decryptor) {
   scoped_refptr<media::FFmpegAudioDecoder> ffmpeg_audio_decoder =
       new media::FFmpegAudioDecoder(message_loop);
-
-  scoped_refptr<media::DecryptingAudioDecoder> decrypting_audio_decoder =
-      new media::DecryptingAudioDecoder(
-          message_loop,
-          base::Bind(&ProxyDecryptor::RequestDecryptorNotification,
-                     base::Unretained(proxy_decryptor)));
-
   filter_collection->GetAudioDecoders()->push_back(ffmpeg_audio_decoder);
-  filter_collection->GetAudioDecoders()->push_back(decrypting_audio_decoder);
 
-  scoped_refptr<media::DecryptingVideoDecoder> decrypting_video_decoder =
-      new media::DecryptingVideoDecoder(
-          message_loop,
-          base::Bind(&ProxyDecryptor::RequestDecryptorNotification,
-                     base::Unretained(proxy_decryptor)));
+  if (proxy_decryptor) {
+    scoped_refptr<media::DecryptingAudioDecoder> decrypting_audio_decoder =
+        new media::DecryptingAudioDecoder(
+            message_loop,
+            base::Bind(&ProxyDecryptor::RequestDecryptorNotification,
+                       base::Unretained(proxy_decryptor)));
+    filter_collection->GetAudioDecoders()->push_back(decrypting_audio_decoder);
+
+    scoped_refptr<media::DecryptingVideoDecoder> decrypting_video_decoder =
+        new media::DecryptingVideoDecoder(
+            message_loop,
+            base::Bind(&ProxyDecryptor::RequestDecryptorNotification,
+                       base::Unretained(proxy_decryptor)));
+    // TODO(xhwang): Ideally we should have decrypting video decoder after
+    // regular video decoder since in the real world most videos are not
+    // encrypted. For now FFmpegVideoDecoder can also do decryption
+    // (decrypt-only), and we perfer DecryptingVideoDecoder (decrypt-and-decode)
+    // to FFmpegVideoDecoder. Fix this order when we move decryption out of
+    // FFmpegVideoDecoder.
+    filter_collection->GetVideoDecoders()->push_back(decrypting_video_decoder);
+  }
 
   scoped_refptr<media::FFmpegVideoDecoder> ffmpeg_video_decoder =
       new media::FFmpegVideoDecoder(message_loop, proxy_decryptor);
-
-  // TODO(xhwang): Ideally we should have decrypting video decoder after
-  // regular video decoder since in the real world most videos are not
-  // encrypted. For now FFmpegVideoDecoder can also do decryption
-  // (decrypt-only), and we perfer DecryptingVideoDecoder (decrypt-and-decode)
-  // to FFmpegVideoDecoder. Fix this order when we move decryption out of
-  // FFmpegVideoDecoder.
-  filter_collection->GetVideoDecoders()->push_back(decrypting_video_decoder);
   filter_collection->GetVideoDecoders()->push_back(ffmpeg_video_decoder);
 }