Stop using an atom to store plugin name/version on Windows.

content/browser/plugin_service_impl.cc

Index: content/browser/plugin_service_impl.cc
diff --git a/content/browser/plugin_service_impl.cc b/content/browser/plugin_service_impl.cc
index e4be1a024b32ffc27afe5fbdfdf55ff69b625235..1b7ad7641c43cbb8d517b76869f17def73732935 100644
--- a/content/browser/plugin_service_impl.cc
+++ b/content/browser/plugin_service_impl.cc
@@ -841,10 +841,15 @@ bool PluginServiceImpl::GetPluginInfoFromWindow(
   if (!IsPluginWindow(window))
     return false;
 
-  GetPluginPropertyFromWindow(
-          window, kPluginNameAtomProperty, plugin_name);
-  GetPluginPropertyFromWindow(
-          window, kPluginVersionAtomProperty, plugin_version);
+
+  DWORD process_id = 0;
+  GetWindowThreadProcessId(window, &process_id);
+  WebPluginInfo info;
+  if (!PluginProcessHost::GetWebPluginInfoFromPluginPid(process_id, &info))

[brettw, 2014/09/10 00:17:27]

I'm surprised this doesn't give you an error about downconverting from DWORD ->
int.

I would have expected base::ProcessId to be used in the places where you use
int, and base::kNullProcessId to be used where you use -1.

[jam, 2014/09/10 01:15:20]

Done.

+    return false;
+
+  *plugin_name = info.name;
+  *plugin_version = info.version;
   return true;
 }