Refactor KeySystems code to call a function to populate the info.

content/public/common/content_client.h

Index: content/public/common/content_client.h
diff --git a/content/public/common/content_client.h b/content/public/common/content_client.h
index e2e0c73a08f8afe1bef63eaf675a68f034d087aa..373a2ca01e22987aaa3bdc623d08a64d4d5c45f2 100644
--- a/content/public/common/content_client.h
+++ b/content/public/common/content_client.h
@@ -45,6 +45,7 @@ class ContentClient;
 class ContentPluginClient;
 class ContentRendererClient;
 class ContentUtilityClient;
+struct KeySystemInfo;
 struct PepperPluginInfo;
 
 // Setter and getter for the client.  The client should be set early, before any
@@ -93,8 +94,7 @@ class CONTENT_EXPORT ContentClient {
   virtual void SetGpuInfo(const gpu::GPUInfo& gpu_info) {}
 
   // Gives the embedder a chance to register its own pepper plugins.
-  virtual void AddPepperPlugins(
-      std::vector<content::PepperPluginInfo>* plugins) {}
+  virtual void AddPepperPlugins(std::vector<PepperPluginInfo>* plugins) {}
 
   // Gives the embedder a chance to register its own standard and saveable
   // url schemes early on in the startup sequence.
@@ -102,6 +102,9 @@ class CONTENT_EXPORT ContentClient {
       std::vector<std::string>* standard_schemes,
       std::vector<std::string>* savable_schemes) {}
 
+  // Gives the embedder a chance to register the key system(s) it supports.
+  virtual void AddKeySystems(std::vector<KeySystemInfo>* key_systems) {}

[jam, 2013/09/03 20:50:52]

why is this in ContentClient? it should only be here if it's needed in multiple
child process types (i.e. content/renderer and content/utility or
content/public).

if you only need this in the renderer, then it should be in
ContentRendererClient

[ddorwin, 2013/09/04 00:41:55]

Done.

+
   // Returns whether the given message should be sent in a swapped out renderer.
   virtual bool CanSendWhileSwappedOut(const IPC::Message* message);