Populate canPlayType to renderer

media/base/android/media_source_player.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/android/media_source_player.h"
 
 #include <limits>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
@@ skipping 10 matching lines @@
 namespace {
 
 // Use 16bit PCM for audio output. Keep this value in sync with the output
 // format we passed to AudioTrack in MediaCodecBridge.
 const int kBytesPerAudioOutputSample = 2;
 }
 
 namespace media {
 
 // static
 bool MediaSourcePlayer::IsTypeSupported(

[ddorwin, 2013/09/16 17:58:10]

What is this function used for?

[qinmin, 2013/09/17 19:21:06]

This function is not used anywhere, I suppose we can remove it.

On 2013/09/16 17:58:10, ddorwin wrote:

     const std::vector<uint8>& scheme_uuid,
     const std::string& security_level,
     const std::string& container,

[ddorwin, 2013/09/16 17:58:10]

What is the plan for the container parameter?

[qinmin, 2013/09/17 19:21:06]

see above, the function is removed
On 2013/09/16 17:58:10, ddorwin wrote:

     const std::vector<std::string>& codecs) {
-  if (!MediaDrmBridge::IsCryptoSchemeSupported(scheme_uuid, container)) {
-    DVLOG(1) << "UUID and container '" << container << "' not supported.";

[xhwang, 2013/09/17 18:56:46]

ditto, probably leave that as is? 

[qinmin, 2013/09/17 21:26:23]

Done.

+  if (!MediaDrmBridge::IsCryptoSchemeSupported(scheme_uuid)) {
+    DVLOG(1) << "UUID not supported.";
     return false;
   }
 
   if (!MediaDrmBridge::IsSecurityLevelSupported(scheme_uuid, security_level)) {
     DVLOG(1) << "UUID and security level '" << security_level
              << "' not supported.";
     return false;
   }
 
   bool is_secure = MediaDrmBridge::IsSecureDecoderRequired(security_level);
@@ skipping 682 matching lines @@
 
 void MediaSourcePlayer::ClearPendingEvent(PendingEventFlags event) {
   DVLOG(1) << __FUNCTION__ << "(" << GetEventName(event) << ")";
   DCHECK_NE(event, NO_EVENT_PENDING);
   DCHECK(IsEventPending(event)) << GetEventName(event);
 
   pending_event_ &= ~event;
 }
 
 }  // namespace media