Populate canPlayType to renderer

chrome/renderer/chrome_content_renderer_client.cc

 // Copyright (c) 2012 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 "chrome/renderer/chrome_content_renderer_client.h"
 
 #include "base/command_line.h"
 #include "base/debug/crash_logging.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
@@ skipping 90 matching lines @@
 
 #if defined(ENABLE_WEBRTC)
 #include "chrome/renderer/media/webrtc_logging_message_filter.h"
 #endif
 
 #if defined(ENABLE_SPELLCHECK)
 #include "chrome/renderer/spellchecker/spellcheck.h"
 #include "chrome/renderer/spellchecker/spellcheck_provider.h"
 #endif
 
+#if defined(OS_ANDROID)
+#include "chrome/common/encrypted_media_messages_android.h"
+#include "chrome/renderer/media/encrypted_media_utils_android.h"
+#endif
+
 using autofill::AutofillAgent;
 using autofill::PasswordAutofillAgent;
 using autofill::PasswordGenerationManager;
 using content::RenderThread;
 using content::WebPluginInfo;
 using extensions::Extension;
 using WebKit::WebCache;
 using WebKit::WebConsoleMessage;
 using WebKit::WebDataSource;
 using WebKit::WebDocument;
@@ skipping 1179 matching lines @@
   }
 #endif  // !defined(OS_ANDROID)
   return false;
 }
 
 void ChromeContentRendererClient::AddKeySystems(
     std::vector<content::KeySystemInfo>* key_systems) {
   // TODO(ddorwin): In the next CL, move code from
   // content/renderer/media/crypto/key_systems_info.cc to chrome_key_systems.cc
   // and call AddKeySystems.
+#if defined(OS_ANDROID)
+  std::vector<SupportedKeySystem> supported_systems;
+  RenderThread::Get()->GetChannel()->Send(

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

Note: I used content::RenderThread::Get()->Send(), which did not requires
MSG_ROUTING_NONE. Not sure which is better.

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

ok, make it consistent with ur CL.
On 2013/09/16 17:58:10, ddorwin wrote:

[jam, 2013/09/17 20:26:02]

RenderThread::Get()->Send is enough

regarding routing id, since it's a message not sent from a routed object (i.e.
RenderView) and received in a message filter (i.e. not a routed class like
RenderViewHost), this is right

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

This is moved to chrome_key_systems.cc, and using
content::RenderThread::Get()->Send()

+      new ChromeViewHostMsg_GetSupportedKeySystems(
+          MSG_ROUTING_NONE, &supported_systems));
+  for (unsigned i = 0; i < supported_systems.size(); ++i) {

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

size_t

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

Done.

+    key_systems->push_back(
+        SupportedKeySystemToKeySystemInfo(supported_systems[i]));

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

See ToT and https://codereview.chromium.org/23828007/. My plan was to
conditionally call Add<KeySystem> functions based on platform support.

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

Done.

+  }
+#endif  // defined(OS_ANDROID)
 }
 
 bool ChromeContentRendererClient::ShouldReportDetailedMessageForSource(
     const base::string16& source) const {
   return extensions::IsSourceFromAnExtension(source);
 }
 
 bool ChromeContentRendererClient::ShouldEnableSiteIsolationPolicy() const {
   // SiteIsolationPolicy is off by default. We would like to activate cross-site
   // document blocking (for UMA data collection) for normal renderer processes
   // running a normal web page from the Internet. We only turn on
   // SiteIsolationPolicy for a renderer process that does not have the extension
   // flag on.
   CommandLine* command_line = CommandLine::ForCurrentProcess();
   return !command_line->HasSwitch(switches::kExtensionProcess);
 }
 
 }  // namespace chrome