Finalize new UseCounter histogram names

third_party/WebKit/Source/core/frame/UseCounter.cpp

 /*
  * Copyright (C) 2012 Google, Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 1247 matching lines @@
   return 0;
 }
 
 EnumerationHistogram& UseCounter::featuresHistogram() const {
   // Every SVGImage has it's own Page instance, and multiple web pages can
   // share the usage of a single SVGImage.  Ideally perhaps we'd delegate
   // metrics from an SVGImage to one of the Page's it's displayed in, but
   // that's tricky (SVGImage is intentionally isolated, and the Page that
   // created it may not even exist anymore).
   // So instead we just use a dedicated histogram for the SVG case.
-  DEFINE_STATIC_LOCAL(blink::EnumerationHistogram, histogram,
-                      ("WebCore.UseCounter_TEST.Features",
-                       blink::UseCounter::NumberOfFeatures));
+  DEFINE_STATIC_LOCAL(
+      blink::EnumerationHistogram, histogram,
+      ("Blink.UseCounter.Features", blink::UseCounter::NumberOfFeatures));
   DEFINE_STATIC_LOCAL(blink::EnumerationHistogram, svgHistogram,
-                      ("WebCore.UseCounter_TEST.SVGImage.Features",
+                      ("Blink.UseCounter.SVGImage.Features",
                        blink::UseCounter::NumberOfFeatures));
 
   return m_context == SVGImageContext ? svgHistogram : histogram;
 }
 
 EnumerationHistogram& UseCounter::cssHistogram() const {
-  DEFINE_STATIC_LOCAL(
-      blink::EnumerationHistogram, histogram,
-      ("WebCore.UseCounter_TEST.CSSProperties", kMaximumCSSSampleId));
+  DEFINE_STATIC_LOCAL(blink::EnumerationHistogram, histogram,
+                      ("Blink.UseCounter.CSSProperties", kMaximumCSSSampleId));
   DEFINE_STATIC_LOCAL(
       blink::EnumerationHistogram, svgHistogram,
-      ("WebCore.UseCounter_TEST.SVGImage.CSSProperties", kMaximumCSSSampleId));
+      ("Blink.UseCounter.SVGImage.CSSProperties", kMaximumCSSSampleId));
 
   return m_context == SVGImageContext ? svgHistogram : histogram;
 }
 
 /*
  *
  * LEGACY metrics support - WebCore.FeatureObserver is to be superceded by
  * WebCore.UseCounter

[tyoshino (SeeGerritForStatus), 2017/03/27 05:54:07]

This should be also updated?

[Rick Byers, 2017/03/27 16:24:47]

Yep, sorry I missed that.  I'm pretty close to being able to delete that code
entirely though (should happen in the next month or so -
https://crbug.com/676837).  Worth me landing a separate fix for anyway do you
think?

  *
  */
 
 static EnumerationHistogram& featureObserverHistogram() {
   DEFINE_STATIC_LOCAL(
       EnumerationHistogram, histogram,
       ("WebCore.FeatureObserver", UseCounter::NumberOfFeatures));
   return histogram;
 }
 
@@ skipping 42 matching lines @@
     }
   }
 
   if (needsPagesMeasuredUpdate)
     cssPropertiesHistogram.count(totalPagesMeasuredCSSSampleId());
 
   m_CSSBits.clearAll();
 }
 
 }  // namespace blink