Add UMA reporting on failure to load ppapi plugins.

content/public/plugin/content_plugin_client.h

Index: content/public/plugin/content_plugin_client.h
diff --git a/content/public/plugin/content_plugin_client.h b/content/public/plugin/content_plugin_client.h
index b89e46a42826b32b4140559bb64654129df3be12..12fc8a6d7009bfd3a0b12a158ad183e4d0d7c43a 100644
--- a/content/public/plugin/content_plugin_client.h
+++ b/content/public/plugin/content_plugin_client.h
@@ -5,6 +5,7 @@
 #ifndef CONTENT_PUBLIC_PLUGIN_CONTENT_PLUGIN_CLIENT_H_
 #define CONTENT_PUBLIC_PLUGIN_CONTENT_PLUGIN_CLIENT_H_
 
+#include "base/files/file_path.h"
 #include "base/string16.h"
 #include "content/public/common/content_client.h"
 
@@ -13,6 +14,15 @@ namespace content {
 // Embedder API for participating in plugin logic.
 class CONTENT_EXPORT ContentPluginClient {
  public:
+  enum PluginLoadStatus {

[ddorwin, 2013/04/08 18:04:34]

Status or Result?

[xhwang, 2013/04/09 00:34:11]

Done.

+    LOAD_SUCCESS,
+    UNKNOWN_ERROR,

[piman, 2013/04/08 19:38:43]

You don't use this. Remove?

[xhwang, 2013/04/09 00:34:11]

Done.

+    LOAD_FAILED,
+    ENTRY_POINT_MISSING,
+    INIT_FAILED,
+    LOAD_STATUS_MAX

[piman, 2013/04/08 19:38:43]

nit: this last value I was very confused about until I looked at the use of it.
Could you add a comment?

[xhwang, 2013/04/09 00:34:11]

Done.

+  };
+
   virtual ~ContentPluginClient() {}
 
   // Called to perform any per-process initialization necessary before
@@ -21,6 +31,10 @@ class CONTENT_EXPORT ContentPluginClient {
 
   // Notifies that a plugin process has started.
   virtual void PluginProcessStarted(const string16& plugin_name) {}
+
+  // Notifies that a plugin has been loaded.

[ddorwin, 2013/04/08 18:04:34]

Could also be that a plugin load failed. I'm not sure how to word that, though.

[xhwang, 2013/04/09 00:34:11]

Done.

+  virtual void PluginLoaded(const base::FilePath& plugin_path,
+                            PluginLoadStatus status) {}
 };
 
 }  // namespace content