Rename BackgroundBudgetService to BudgetManager

chrome/browser/push_messaging/push_messaging_notification_manager.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/push_messaging/push_messaging_notification_manager.h"
 
 #include <stddef.h>
 
 #include <bitset>
 
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
-#include "chrome/browser/budget_service/background_budget_service.h"
-#include "chrome/browser/budget_service/background_budget_service_factory.h"
+#include "chrome/browser/budget_service/budget_manager.h"
+#include "chrome/browser/budget_service/budget_manager_factory.h"
 #include "chrome/browser/engagement/site_engagement_service.h"
 #include "chrome/browser/notifications/platform_notification_service_impl.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/push_messaging/push_messaging_constants.h"
 #include "chrome/common/features.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/rappor/rappor_utils.h"
 #include "components/url_formatter/elide_url.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
@@ skipping 153 matching lines @@
       platform_notification_service->ClosePersistentNotification(
           profile_, notification_database_data.notification_id);
       platform_notification_service->OnPersistentNotificationClose(
           profile_, notification_database_data.notification_id,
           notification_database_data.origin, false /* by_user */);
 
       break;
     }
   }
 
   // Get the budget for the service worker.

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

nit: origin

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

Done.

-  BackgroundBudgetService* service =
-      BackgroundBudgetServiceFactory::GetForProfile(profile_);
+  BudgetManager* service = BudgetManagerFactory::GetForProfile(profile_);

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

nit: s/service/manager/?

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

Done.

   service->GetBudget(
       origin,
       base::Bind(&PushMessagingNotificationManager::DidGetBudget,
                  weak_factory_.GetWeakPtr(), origin,
                  service_worker_registration_id, message_handled_closure,
                  notification_needed, notification_shown));
 }
 
 void PushMessagingNotificationManager::DidGetBudget(
     const GURL& origin,
     int64_t service_worker_registration_id,
     const base::Closure& message_handled_closure,
     bool notification_needed,
     bool notification_shown,
     const double budget) {
   // Record the budget available any time the budget is queried.
-  UMA_HISTOGRAM_COUNTS_100("PushMessaging.BackgroundBudget", budget);
+  UMA_HISTOGRAM_COUNTS_100("PushMessaging.BudgetManager", budget);
 
   // Get the site engagement score. Only used for UMA recording.
   SiteEngagementService* ses_service = SiteEngagementService::Get(profile_);
   double ses_score = ses_service->GetScore(origin);
 
   // Generate histograms for the GetBudget calls which would return "no budget"
   // or "low budget" if an API was available to app developers.
-  double cost = BackgroundBudgetService::GetCost(
-      BackgroundBudgetService::CostType::SILENT_PUSH);
+  double cost = BudgetManager::GetCost(BudgetManager::CostType::SILENT_PUSH);
   if (budget < cost)
     UMA_HISTOGRAM_COUNTS_100("PushMessaging.SESForNoBudgetOrigin", ses_score);
   else if (budget < 2.0 * cost)
     UMA_HISTOGRAM_COUNTS_100("PushMessaging.SESForLowBudgetOrigin", ses_score);
 
   if (notification_needed && !notification_shown) {
     // If the worker needed to show a notification and didn't, check the budget
     // and take appropriate action.
     CheckForMissedNotification(origin, service_worker_registration_id,
                                message_handled_closure, budget);
@@ skipping 49 matching lines @@
 
 void PushMessagingNotificationManager::CheckForMissedNotification(
     const GURL& origin,
     int64_t service_worker_registration_id,
     const base::Closure& message_handled_closure,
     const double budget) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   // If the service needed to show a notification but did not, update the
   // budget.
-  double cost = BackgroundBudgetService::GetCost(
-      BackgroundBudgetService::CostType::SILENT_PUSH);
+  double cost = BudgetManager::GetCost(BudgetManager::CostType::SILENT_PUSH);
   if (budget >= cost) {
     RecordUserVisibleStatus(
         content::PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_USED_GRACE);
 
-    BackgroundBudgetService* service =
-        BackgroundBudgetServiceFactory::GetForProfile(profile_);
+    BudgetManager* service = BudgetManagerFactory::GetForProfile(profile_);

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

nit: s/service/manager/

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

Done.

     // Update the stored budget.
     service->StoreBudget(origin, budget - cost, message_handled_closure);
 
     return;
   }
 
   RecordUserVisibleStatus(
       content::PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_GRACE_EXCEEDED);
   rappor::SampleDomainAndRegistryFromGURL(
       g_browser_process->rappor_service(),
@@ skipping 49 matching lines @@
   // Do not pass service worker scope. The origin will be used instead of the
   // service worker scope to determine whether a notification should be
   // attributed to a WebAPK on Android. This is OK because this code path is hit
   // rarely.
   PlatformNotificationServiceImpl::GetInstance()->DisplayPersistentNotification(
       profile_, persistent_notification_id, GURL() /* service_worker_scope */,
       origin, notification_data, NotificationResources());
 
   message_handled_closure.Run();
 }