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

chrome/renderer/chrome_content_renderer_client.h

 // 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.
 
 #ifndef CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_
 #define CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 121 matching lines @@
   virtual bool IsExternalPepperPlugin(const std::string& module_name) OVERRIDE;
   // TODO(victorhsieh): move to ChromeContentBrowserClient once we migrate
   // PPAPI FileIO host to browser.
   virtual bool IsPluginAllowedToCallRequestOSFileHandle(
       WebKit::WebPluginContainer* container) OVERRIDE;
   virtual WebKit::WebSpeechSynthesizer* OverrideSpeechSynthesizer(
       WebKit::WebSpeechSynthesizerClient* client) OVERRIDE;
   virtual bool ShouldReportDetailedMessageForSource(
       const base::string16& source) const OVERRIDE;
   virtual bool AllowPepperMediaStreamAPI(const GURL& url) OVERRIDE;
 

[xhwang, 2013/09/04 21:48:40]

remove this empty line? I don't see why AddKeySystems() is special.

[ddorwin, 2013/09/06 18:30:00]

Done.

+  virtual void AddKeySystems(
+      std::vector<content::KeySystemInfo>* key_systems) OVERRIDE;
+
   // For testing.
   void SetExtensionDispatcher(extensions::Dispatcher* extension_dispatcher);
 
 #if defined(ENABLE_SPELLCHECK)
   // Sets a new |spellcheck|. Used for low-mem restart and testing only.
   // Takes ownership of |spellcheck|.
   void SetSpellcheck(SpellCheck* spellcheck);
 #endif
 
   // Called in low-memory conditions to dump the memory used by the spellchecker
@@ skipping 53 matching lines @@
 #endif
 
 #if defined(ENABLE_PLUGINS)
   std::set<std::string> allowed_file_handle_origins_;
 #endif
 };
 
 }  // namespace chrome
 
 #endif  // CHROME_RENDERER_CHROME_CONTENT_RENDERER_CLIENT_H_