Revert of Fix stability and data racing issues, coalesce more updates for JumpList

chrome/browser/win/jumplist_factory.cc

Index: chrome/browser/win/jumplist_factory.cc
diff --git a/chrome/browser/win/jumplist_factory.cc b/chrome/browser/win/jumplist_factory.cc
index 66e6fe0b89987eaf598d8af7bd0f61fab900389c..92b10bbe0c95e48e73c8b4f435409870896cb5d4 100644
--- a/chrome/browser/win/jumplist_factory.cc
+++ b/chrome/browser/win/jumplist_factory.cc
@@ -8,13 +8,12 @@
 #include "chrome/browser/history/top_sites_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sessions/tab_restore_service_factory.h"
-#include "chrome/browser/win/jumplist.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 
 // static
-JumpList* JumpListFactory::GetForProfile(Profile* profile) {
+scoped_refptr<JumpList> JumpListFactory::GetForProfile(Profile* profile) {
   return static_cast<JumpList*>(
-      GetInstance()->GetServiceForBrowserContext(profile, true));
+      GetInstance()->GetServiceForBrowserContext(profile, true).get());
 }
 
 // static
@@ -23,7 +22,7 @@
 }
 
 JumpListFactory::JumpListFactory()
-    : BrowserContextKeyedServiceFactory(
+    : RefcountedBrowserContextKeyedServiceFactory(
           "JumpList",
           BrowserContextDependencyManager::GetInstance()) {
   DependsOn(TabRestoreServiceFactory::GetInstance());
@@ -33,7 +32,7 @@
 
 JumpListFactory::~JumpListFactory() = default;
 
-KeyedService* JumpListFactory::BuildServiceInstanceFor(
+scoped_refptr<RefcountedKeyedService> JumpListFactory::BuildServiceInstanceFor(
     content::BrowserContext* context) const {
   return new JumpList(Profile::FromBrowserContext(context));
 }