Add UMA reporting on failure to load ppapi plugins.

content/public/plugin/content_plugin_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 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"
 
 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
   // the sandbox is initialized for the plugin process.
   virtual void PreSandboxInitialization() {}
 
   // 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
 
 #endif  // CONTENT_PUBLIC_PLUGIN_CONTENT_PLUGIN_CLIENT_H_