Rename BackgroundBudgetService to BudgetManager

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 bcf6775fb93b6cf25c1bdb96cdeba481eef20deb..00c194c606e79a7abe8c95a6482436aaed0a43b9 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -45708,7 +45708,7 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
-<histogram name="PushMessaging.BackgroundBudget">
+<histogram name="PushMessaging.BudgetManager">

[Peter Beverloo, 2016/08/12 12:39:47]

We've been recording this already, so we can't modify this UMA.

We can either keep it as-is, or create a new one. I don't see a strong reason to
rename it, and I don't feel "BudgetManager" is a clear name for a metric, so I
propose keeping this as-is.

[harkness, 2016/08/15 08:05:41]

Sounds good, I've switched back to the original name.

   <owner>harkness@chromium.org</owner>
   <summary>
     Whenever a Service Worker receives a push message, this records the budget
@@ -98618,7 +98618,7 @@ To add a new entry, add it with any value and run test to compute valid value.
   <suffix name="UsageReportsBufferBackend"
       label="The result of the first attempt to open the usage reports buffer
              backend database."/>
-  <suffix name="BackgroundBudgetService"
+  <suffix name="BudgetManager"

[Peter Beverloo, 2016/08/12 12:39:47]

Since this hasn't ran outside of tests yet, this is fine to rename. Please do
clarify this to the UMA reviewer when sending it out to them.

[harkness, 2016/08/15 08:05:41]

Acknowledged.

       label="Database for storing budget information for origins."/>
   <affected-histogram name="LevelDB.Open"/>
 </histogram_suffixes>