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

content/public/common/key_system_info.cc

Index: content/public/common/key_system_info.cc
diff --git a/content/public/browser/javascript_dialog_manager.cc b/content/public/common/key_system_info.cc
similarity index 50%
copy from content/public/browser/javascript_dialog_manager.cc
copy to content/public/common/key_system_info.cc
index 0a58d9ffb3817aaa9045086bd13d92a9a4af4148..2a6b784abee6405410aaad6df65e34dd1bf45462 100644
--- a/content/public/browser/javascript_dialog_manager.cc
+++ b/content/public/common/key_system_info.cc
@@ -2,15 +2,16 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "content/public/browser/javascript_dialog_manager.h"
+#include "content/public/common/key_system_info.h"
 
 namespace content {
 
-bool JavaScriptDialogManager::HandleJavaScriptDialog(
-    WebContents* web_contents,
-    bool accept,
-    const string16* prompt_override) {
-  return false;
+KeySystemInfo::KeySystemInfo(const std::string& key_system)

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

nit: if you can inline this constructor in the header and get rid of the cc
file, that's preferred. the reason you wouldn't be able to is clang, but it has
some heuristics that i don't know off the top of my head. so please try that and
see if you get green try runs.

also no need for destructor?

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

Clang is unhappy:

../../content/public/renderer/key_system_info.h:37:3: error: [chromium-style]
Complex constructor has an inlined body.
  explicit KeySystemInfo(const std::string& key_system)
  ^
../../content/public/renderer/key_system_info.h:34:1: error: [chromium-style]
Complex class/struct needs an explicit out-of-line destructor.
struct CONTENT_EXPORT KeySystemInfo {
^

+    : key_system(key_system),
+      use_aes_decryptor(false) {
+}
+
+KeySystemInfo::~KeySystemInfo() {
 }
 
 }  // namespace content