Report disabled plugin in CreatePlugin.

chrome/renderer/chrome_content_renderer_client.cc

Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index 1de8a7c71c5620b8418d5b6ddd940dadf922466e..c9ab86d38407f5771eb75fa2cde4bd526c0bf90c 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -522,8 +522,8 @@ WebPlugin* ChromeContentRendererClient::CreatePlugin(
       return PluginPlaceholder::CreateMobileYoutubePlugin(render_view, frame,
           original_params)->plugin();
 #endif
-    MissingPluginReporter::GetInstance()->ReportPluginMissing(
-        orig_mime_type, url);
+    // TODO(xhwang): Distinguish between kNotFound and kBrowserPluginMimeType.

[ddorwin, 2013/04/04 23:56:47]

I don't think that's quite right. Either way, if you're doing this in the other
CL, you don't need this. (You would just have to modify this file to remove the
TODO later.)

[xhwang, 2013/04/05 01:00:24]

Done.

+    PluginUMAReporter::GetInstance()->ReportPluginMissing(orig_mime_type, url);
     placeholder = PluginPlaceholder::CreateMissingPlugin(
         render_view, frame, original_params);
   } else {
@@ -647,6 +647,8 @@ WebPlugin* ChromeContentRendererClient::CreatePlugin(
         break;
       }
       case ChromeViewHostMsg_GetPluginInfo_Status::kDisabled: {
+        PluginUMAReporter::GetInstance()->ReportPluginDisabled(orig_mime_type,
+                                                               url);
         placeholder = PluginPlaceholder::CreateBlockedPlugin(
             render_view, frame, params, plugin, identifier, group_name,
             IDR_DISABLED_PLUGIN_HTML,