Include PPAPI plugin crashes in the plugin crash data.

chrome/browser/metrics/metrics_service_unittest.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/browser/metrics/metrics_service.h"
 
 #include <string>
 
 #include "base/base64.h"
 
@@ skipping 10 matching lines @@
   for (uint32 i = 0; i < clientid.length(); i++) {
     char current = clientid.at(i);
     if (i == 8 || i == 13 || i == 18 || i == 23) {
       EXPECT_EQ('-', current);
     } else {
       EXPECT_TRUE(std::string::npos != hexchars.find(current));
     }
   }
 }
 
+TEST(MetricsServiceTest, IsPluginProcess) {
+  EXPECT_TRUE(
+      MetricsService::IsPluginProcess(ChildProcessInfo::PLUGIN_PROCESS));
+  EXPECT_FALSE(
+      MetricsService::IsPluginProcess(ChildProcessInfo::GPU_PROCESS));
+#if defined(OS_CHROMEOS)
+  bool ppapi_plugin = true;
+#else
+  bool ppapi_plugin = false;
+#endif
+  EXPECT_EQ(
+      ppapi_plugin,
+      MetricsService::IsPluginProcess(ChildProcessInfo::PPAPI_PLUGIN_PROCESS));

[jar (doing other things), 2011/11/08 22:16:31]

This ifdef should also go away when you use the logical or in the definition.

[petkov, 2011/11/09 14:21:07]

Done.

+}