[Extensions] Add UMA for the proxy override bubble

chrome/browser/extensions/proxy_overridden_bubble_delegate.cc

Index: chrome/browser/extensions/proxy_overridden_bubble_delegate.cc
diff --git a/chrome/browser/extensions/proxy_overridden_bubble_delegate.cc b/chrome/browser/extensions/proxy_overridden_bubble_delegate.cc
index 684ed5cc7e9e3f70e96421e89b4f6cabf6e65f48..32715d5790f4cc45d64c4a089a07d1d855bfe3f8 100644
--- a/chrome/browser/extensions/proxy_overridden_bubble_delegate.cc
+++ b/chrome/browser/extensions/proxy_overridden_bubble_delegate.cc
@@ -133,12 +133,11 @@ bool ProxyOverriddenBubbleDelegate::ShouldLimitToEnabledExtensions() const {
 }
 
 void ProxyOverriddenBubbleDelegate::LogExtensionCount(size_t count) {
-  UMA_HISTOGRAM_COUNTS_100("ProxyOverriddenBubble.ExtensionCount", count);

[Devlin, 2016/09/08 22:09:32]

This bubble is limited to a single extension, so this histogram is unneeded.

[Mark P, 2016/09/08 22:55:25]

Please add this histogram with an appropriate description (describing its
problems) and an obsolete tag, otherwise perhaps someone hunting for histograms
to investigate something in the future will be misled.

[Devlin, 2016/09/08 23:13:36]

Done.

 }
 
 void ProxyOverriddenBubbleDelegate::LogAction(
     ExtensionMessageBubbleController::BubbleAction action) {
-  UMA_HISTOGRAM_ENUMERATION("ProxyOverriddenBubble.UserSelection",

[Mark P, 2016/09/08 22:55:25]

Please add this old histogram, optionally with an obsolete tag pointing to the
new one.  It's possible a user of the new one will wish they had more history
accessible, which is my the old one should be in histograms.xml.

[Devlin, 2016/09/08 23:13:36]

As discussed offline, since retroactively adding the entry to histograms.xml
will still have the data from before, added this histogram and still using this
name.

+  UMA_HISTOGRAM_ENUMERATION("ExtensionBubble.ProxyUserSelection",
                             action,
                             ExtensionMessageBubbleController::ACTION_BOUNDARY);
 }