tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 623cacfca61a4f4741f6a30fa2a0c01b2d7cfccd..21d955d424b6f1a822cda5761f7c7504071439c3 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -29532,6 +29532,7 @@ other types of suffix sets.
   <int value="944161065" label="PPB_Flash_DRM;1.1"/>
   <int value="946515854" label="PPB_View(Dev);0.1"/>
   <int value="948969343" label="PPB_OpenGLES2;1.0"/>
+  <int value="961061294" label="PPB_Var;1.2"/>
   <int value="961317980" label="PPB_Fullscreen;1.0"/>
   <int value="964595048" label="PPB_BrokerTrusted;0.2"/>
   <int value="965548627" label="PPB_Audio;1.1"/>
@@ -29570,7 +29571,6 @@ other types of suffix sets.
   <int value="1374976378" label="PPB_OpenGLES2Query;1.0"/>
   <int value="1437724812" label="PPB_AudioConfig;1.0"/>
   <int value="1443771913" label="PPB_NetAddress;1.0"/>
-  <int value="1465557812" label="PPB_VarResource(Dev);0.1"/>

[Alexei Svitkine (slow), 2014/02/10 14:28:53]

Don't remove existing entries from here, since old clients can still report
them. Just add the new ones.

[dmichael (off chromium), 2014/02/10 18:31:56]

The old interface was not being used by any apps aside from internal testing, so
in this case it's safe to remove. It won't appear in the wild.

[Alexei Svitkine (slow), 2014/02/10 18:36:31]

Normally, I'd still push back, but given this whole histogram was added recently
and indeed it seems we don't have any data for that entry[1], I guess it's okay.

[1]https://uma.googleplex.com/histograms/?dayCount=1&endDate=02-10-2014&version=Everything&histograms=Pepper.InterfaceUsed&otherVersions=true&filters

[Matt Giuca, 2014/02/11 02:41:17]

Note that external users can test out this API so we may still get data for this
in the future. But I don't think it's very important seeing as how we didn't
publicise it and it's going away now.

It depends on how critical it is -- UMA is not going to crash is it, if a data
point comes in that we don't have a handler for? I assume it just means we won't
record that data point, and since it will be at best a trickle, it can be safely
removed.

   <int value="1504691399" label="PPB_Flash;13.0"/>
   <int value="1505595424" label="PPB_Crypto(Dev);0.1"/>
   <int value="1508192415" label="PPB_VarDictionary;1.0"/>