Removing old prefs storage for BudgetManager

chrome/browser/budget_service/budget_manager.cc

 // Copyright 2016 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/budget_service/budget_manager.h"
 
 #include <stdint.h>
 
 #include "base/callback.h"
 #include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "base/time/time.h"
 #include "chrome/browser/engagement/site_engagement_score.h"
 #include "chrome/browser/profiles/profile.h"
-#include "chrome/common/pref_names.h"
-#include "components/pref_registry/pref_registry_syncable.h"
-#include "components/prefs/pref_service.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/common/origin_util.h"
 #include "third_party/WebKit/public/platform/modules/budget_service/budget_service.mojom.h"
 #include "url/origin.h"
 
 using content::BrowserThread;
 
-namespace {
-
-// Previously, budget information was stored in the prefs. If there is any old
-// information still there, clear it.
-// TODO(harkness): Remove once Chrome 56 has branched.
-void ClearBudgetDataFromPrefs(Profile* profile) {
-  profile->GetPrefs()->ClearPref(prefs::kBackgroundBudgetMap);
-}
-
-}  // namespace
-
 BudgetManager::BudgetManager(Profile* profile)
     : profile_(profile),
       db_(profile,
           profile->GetPath().Append(FILE_PATH_LITERAL("BudgetDatabase")),
           BrowserThread::GetBlockingPool()
               ->GetSequencedTaskRunnerWithShutdownBehavior(
                   base::SequencedWorkerPool::GetSequenceToken(),
                   base::SequencedWorkerPool::CONTINUE_ON_SHUTDOWN)),
       weak_ptr_factory_(this) {
-  ClearBudgetDataFromPrefs(profile);
 }
 
 BudgetManager::~BudgetManager() {}
 
 // static
-void BudgetManager::RegisterProfilePrefs(
-    user_prefs::PrefRegistrySyncable* registry) {
-  registry->RegisterDictionaryPref(prefs::kBackgroundBudgetMap);
-}
-
-// static
 double BudgetManager::GetCost(blink::mojom::BudgetOperationType type) {
   switch (type) {
     case blink::mojom::BudgetOperationType::SILENT_PUSH:
       return 2.0;
     case blink::mojom::BudgetOperationType::INVALID_OPERATION:
       return SiteEngagementScore::kMaxPoints + 1;
       // No default case.
   }
   NOTREACHED();
   return SiteEngagementScore::kMaxPoints + 1.0;
@@ skipping 87 matching lines @@
                                blink::mojom::BudgetServiceErrorType error,
                                bool success) {
   // If the call succeeded, write the new reservation into the map.
   if (success && error == blink::mojom::BudgetServiceErrorType::NONE)
     reservation_map_[origin]++;
 
   UMA_HISTOGRAM_BOOLEAN("Blink.BudgetAPI.Reserve", success);
 
   callback.Run(error, success);
 }