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 b28c5db13b18ce2fe49cc29f05b9e3c5dcdba794..a1a83ca01c80cb3b1b8fffb31d8fc0aead821083 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -33132,6 +33132,9 @@ Therefore, the affected-histogram name has to have at least one dot in it.
 
 <histogram name="SB2.SideEffectFreeWhitelistStatus"
     enum="SB2SideEffectFreeWhitelistStatus">
+  <obsolete>
+    Deprecated 4/2015.
+  </obsolete>

[Scott Hess - ex-Googler, 2015/04/08 22:32:26]

AFAICT, the practice for marking the field trial obsolete in SafeBrowsingLists
would be to edit the LABEL.

[davidben, 2015/04/09 21:54:38]

I'm not sure I follow. What do you mean by LABEL?

[Scott Hess - ex-Googler, 2015/04/09 23:17:19]

Sorry, I'm away from my main desk, there should be a histogram_suffixes item for
SafeBrowsingLists.  The one for side-effect-free-whitelist should change so the
LABEL includes "obsolete" in the text.  If there is no label, add one for the
appropriate item (I think the rule is that the label is used for display in
preference to the name, but the name is used if there is no label).  Look around
the area, there are examples with an "obsolete" inline like that.

[davidben, 2015/04/09 23:41:23]

Gotcha. I think actually I need to obsolete both this one (which isn't generated
by suffixes) and the SideEffectFreeWhitelist suffix.

It seems you can actually obsolete individual suffixes with the <obsolete> tag.
I've done that since it'll participate in the UI right, but if you prefer
changing the label, I can stick with that.

   <owner>shess@chromium.org</owner>
   <summary>The instantiation status of the SideEffectFreeWhitelist.</summary>
 </histogram>