Add UMA private PPAPI interface for UMA logging from NaCl plugin

webkit/plugins/ppapi/ppb_uma_private_impl.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 "webkit/plugins/ppapi/ppb_uma_private_impl.h"
+
+#include "base/metrics/histogram.h"
+#include "ppapi/c/pp_var.h"
+#include "ppapi/c/private/ppb_uma_private.h"
+#include "webkit/glue/webkit_glue.h"
+#include "webkit/plugins/ppapi/var.h"
+
+namespace webkit {
+namespace ppapi {
+
+namespace {
+
+void HistogramCustomTimes(PP_Var name, int64_t sample,
+                          int64_t min, int64_t max, uint32_t bucket_count) {

[bbudge-google, 2011/04/27 23:03:54]

I think the parameter formatting rule is all on one line, or if they don't fit,
on separate lines, with the exception that logically connected params can share
a line. So possibly...
   name,
   sample,
   min, max,
   bucket_count) {

[elijahtaylor (use chromium), 2011/04/28 19:14:47]

Done.

+  scoped_refptr<StringVar> name_string(StringVar::FromPPVar(name));
+  base::Histogram* counter =
+    base::Histogram::FactoryTimeGet(name_string->value(),
+                                    base::TimeDelta::FromMilliseconds(min),
+                                    base::TimeDelta::FromMilliseconds(max),
+                                    bucket_count,
+                                    base::Histogram::kUmaTargetedHistogramFlag);
+  counter->AddTime(base::TimeDelta::FromMilliseconds(sample));
+}
+
+void HistogramCustomCounts(PP_Var name, int32_t sample,
+                           int32_t min, int32_t max, uint32_t bucket_count) {
+  scoped_refptr<StringVar> name_string(StringVar::FromPPVar(name));
+  base::Histogram* counter =
+    base::Histogram::FactoryGet(name_string->value(),min, max, bucket_count,
+                                base::Histogram::kUmaTargetedHistogramFlag);
+  counter->Add(sample);
+}
+
+void HistogramEnumeration(PP_Var name, int32_t sample, int32_t boundary_value) {
+  scoped_refptr<StringVar> name_string(StringVar::FromPPVar(name));
+  base::Histogram* counter =
+    base::LinearHistogram::FactoryGet(name_string->value(), 1, boundary_value,
+      boundary_value + 1, base::Histogram::kUmaTargetedHistogramFlag);
+  counter->Add(sample);
+}
+
+}  // namespace
+
+const PPB_UMA_Private ppb_uma = {
+  &HistogramCustomTimes,
+  &HistogramCustomCounts,
+  &HistogramEnumeration,
+};
+
+// static
+const PPB_UMA_Private* PPB_UMA_Private_Impl::GetInterface() {
+  return &ppb_uma;
+}
+
+}  // namespace ppapi
+}  // namespace webkit