Add new UseCounter metric

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

 /*
  * 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 41 matching lines @@
 // anonymously through the Histogram recording system in Chrome
 // for users who opt-in to "Usage Statistics" submission
 // during their install of Google Chrome:
 // http://www.google.com/chrome/intl/en/privacy.html
 
 class CORE_EXPORT UseCounter {
     DISALLOW_NEW();
     WTF_MAKE_NONCOPYABLE(UseCounter);
 public:
     UseCounter();
-    ~UseCounter();
 
     enum Feature {
         // Do not change assigned numbers of existing items: add new features
         // to the end of the list.
-        PageDestruction = 0,
+        OBSOLETE_PageDestruction = 0,
         PrefixedIndexedDB = 3,
         WorkerStart = 4,
         SharedWorkerStart = 5,
         UnprefixedIndexedDB = 9,
         OpenWebDatabase = 10,
         UnprefixedRequestAnimationFrame = 13,
         PrefixedRequestAnimationFrame = 14,
         ContentSecurityPolicy = 15,
         ContentSecurityPolicyReportOnly = 16,
         PrefixedTransitionEndEvent = 18,
@@ skipping 1253 matching lines @@
 
     void muteForInspector();
     void unmuteForInspector();
 
     void recordMeasurement(Feature);
     void updateMeasurements();
 
     bool hasRecordedMeasurement(Feature) const;
 
 protected:
-    friend class UseCounterTest;
+    // Encapsulates the work to preserve the old "FeatureObserver" histogram with original semantics
+    // TODO(rbyers): remove this - http://crbug.com/597963
+    class LegacyCounter {

[Nico, 2016/09/13 14:16:47]

this needs a CORE_EXPORT, else release component builds won't link (at least
with clang-cl):

FAILED: webkit_unit_tests.exe webkit_unit_tests.exe.pdb 
E:/b/depot_tools/python276_bin/python.exe
../../build/toolchain/win/tool_wrapper.py link-wrapper environment.x64 False
link.exe /nologo /OUT:./webkit_unit_tests.exe /PDB:./webkit_unit_tests.exe.pdb
@./webkit_unit_tests.exe.rsp
UseCounterTest.obj : error LNK2019: unresolved external symbol "public: __cdecl
blink::UseCounter::LegacyCounter::~LegacyCounter(void)"
(??1LegacyCounter@UseCounter@blink@@QEAA@XZ) referenced in function "private:
virtual void __cdecl
blink::UseCounterTest_RecordingFeatures_Test::TestBody(void)"
(?TestBody@UseCounterTest_RecordingFeatures_Test@blink@@EEAAXXZ)
./webkit_unit_tests.exe : fatal error LNK1120: 1 unresolved externals

https://build.chromium.org/p/chromium.fyi/builders/CrWinClang%28shared%29/bui...

+    public:
+        LegacyCounter();
+        ~LegacyCounter();
+        void countFeature(Feature);
+        void countCSS(CSSPropertyID);
+        void updateMeasurements();
+    private:
+        // Tracks what features/properties need to be reported to the legacy histograms.
+        BitVector m_featureBits;
+        BitVector m_CSSBits;
+    } m_legacyCounter;
+
     unsigned m_muteCount;
 
-    BitVector m_featureBits;
-    BitVector m_CSSFeatureBits;
+    // Track what features/properties have been reported to the (non-legacy) histograms.
+    BitVector m_featuresRecorded;
+    BitVector m_CSSRecorded;
 };
 
 } // namespace blink
 
 #endif // UseCounter_h