diff --git a/shell/browser/badging/badge_manager_factory.cc b/shell/browser/badging/badge_manager_factory.cc index 7f5047cf7d81..71216a0aca97 100644 --- a/shell/browser/badging/badge_manager_factory.cc +++ b/shell/browser/badging/badge_manager_factory.cc @@ -31,9 +31,10 @@ BadgeManagerFactory::BadgeManagerFactory() BadgeManagerFactory::~BadgeManagerFactory() = default; -KeyedService* BadgeManagerFactory::BuildServiceInstanceFor( +std::unique_ptr +BadgeManagerFactory::BuildServiceInstanceForBrowserContext( content::BrowserContext* context) const { - return new BadgeManager(); + return std::make_unique(); } } // namespace badging diff --git a/shell/browser/badging/badge_manager_factory.h b/shell/browser/badging/badge_manager_factory.h index a104bc1c2a7f..3ce918a74915 100644 --- a/shell/browser/badging/badge_manager_factory.h +++ b/shell/browser/badging/badge_manager_factory.h @@ -36,7 +36,7 @@ class BadgeManagerFactory : public BrowserContextKeyedServiceFactory { ~BadgeManagerFactory() override; // BrowserContextKeyedServiceFactory - KeyedService* BuildServiceInstanceFor( + std::unique_ptr BuildServiceInstanceForBrowserContext( content::BrowserContext* context) const override; }; diff --git a/shell/browser/extensions/electron_extension_system_factory.cc b/shell/browser/extensions/electron_extension_system_factory.cc index 72074ec33122..55a654eec8b5 100644 --- a/shell/browser/extensions/electron_extension_system_factory.cc +++ b/shell/browser/extensions/electron_extension_system_factory.cc @@ -35,9 +35,10 @@ ElectronExtensionSystemFactory::ElectronExtensionSystemFactory() ElectronExtensionSystemFactory::~ElectronExtensionSystemFactory() = default; -KeyedService* ElectronExtensionSystemFactory::BuildServiceInstanceFor( +std::unique_ptr +ElectronExtensionSystemFactory::BuildServiceInstanceForBrowserContext( BrowserContext* context) const { - return new ElectronExtensionSystem(context); + return std::make_unique(context); } BrowserContext* ElectronExtensionSystemFactory::GetBrowserContextToUse( diff --git a/shell/browser/extensions/electron_extension_system_factory.h b/shell/browser/extensions/electron_extension_system_factory.h index 449e49f3dd69..f2200dd2f393 100644 --- a/shell/browser/extensions/electron_extension_system_factory.h +++ b/shell/browser/extensions/electron_extension_system_factory.h @@ -5,6 +5,8 @@ #ifndef ELECTRON_SHELL_BROWSER_EXTENSIONS_ELECTRON_EXTENSION_SYSTEM_FACTORY_H_ #define ELECTRON_SHELL_BROWSER_EXTENSIONS_ELECTRON_EXTENSION_SYSTEM_FACTORY_H_ +#include + #include "extensions/browser/extension_system_provider.h" namespace base { @@ -36,7 +38,7 @@ class ElectronExtensionSystemFactory : public ExtensionSystemProvider { ~ElectronExtensionSystemFactory() override; // BrowserContextKeyedServiceFactory implementation: - KeyedService* BuildServiceInstanceFor( + std::unique_ptr BuildServiceInstanceForBrowserContext( content::BrowserContext* context) const override; content::BrowserContext* GetBrowserContextToUse( content::BrowserContext* context) const override; diff --git a/shell/browser/file_system_access/file_system_access_permission_context_factory.cc b/shell/browser/file_system_access/file_system_access_permission_context_factory.cc index ff553fc2dc59..4f01b1cb65b4 100644 --- a/shell/browser/file_system_access/file_system_access_permission_context_factory.cc +++ b/shell/browser/file_system_access/file_system_access_permission_context_factory.cc @@ -36,15 +36,9 @@ FileSystemAccessPermissionContextFactory:: FileSystemAccessPermissionContextFactory:: ~FileSystemAccessPermissionContextFactory() = default; -// static -KeyedService* FileSystemAccessPermissionContextFactory::BuildServiceInstanceFor( - content::BrowserContext* context) const { - return BuildInstanceFor(context).release(); -} - std::unique_ptr -FileSystemAccessPermissionContextFactory::BuildInstanceFor( - content::BrowserContext* context) { +FileSystemAccessPermissionContextFactory::BuildServiceInstanceForBrowserContext( + content::BrowserContext* context) const { return std::make_unique(context); } diff --git a/shell/browser/file_system_access/file_system_access_permission_context_factory.h b/shell/browser/file_system_access/file_system_access_permission_context_factory.h index b08fe99d02fe..62256aa742ed 100644 --- a/shell/browser/file_system_access/file_system_access_permission_context_factory.h +++ b/shell/browser/file_system_access/file_system_access_permission_context_factory.h @@ -5,6 +5,8 @@ #ifndef ELECTRON_SHELL_BROWSER_FILE_SYSTEM_ACCESS_FILE_SYSTEM_ACCESS_PERMISSION_CONTEXT_FACTORY_H_ #define ELECTRON_SHELL_BROWSER_FILE_SYSTEM_ACCESS_FILE_SYSTEM_ACCESS_PERMISSION_CONTEXT_FACTORY_H_ +#include + #include "base/no_destructor.h" #include "components/keyed_service/content/browser_context_keyed_service_factory.h" #include "shell/browser/file_system_access/file_system_access_permission_context.h" @@ -18,9 +20,6 @@ class FileSystemAccessPermissionContextFactory content::BrowserContext* context); static FileSystemAccessPermissionContextFactory* GetInstance(); - static std::unique_ptr BuildInstanceFor( - content::BrowserContext* context); - FileSystemAccessPermissionContextFactory( const FileSystemAccessPermissionContextFactory&) = delete; FileSystemAccessPermissionContextFactory& operator=( @@ -33,7 +32,7 @@ class FileSystemAccessPermissionContextFactory ~FileSystemAccessPermissionContextFactory() override; // BrowserContextKeyedServiceFactory: - KeyedService* BuildServiceInstanceFor( + std::unique_ptr BuildServiceInstanceForBrowserContext( content::BrowserContext* context) const override; }; diff --git a/shell/browser/hid/hid_chooser_context_factory.cc b/shell/browser/hid/hid_chooser_context_factory.cc index 74982208ca88..ab556210003f 100644 --- a/shell/browser/hid/hid_chooser_context_factory.cc +++ b/shell/browser/hid/hid_chooser_context_factory.cc @@ -37,11 +37,11 @@ HidChooserContextFactory::HidChooserContextFactory() HidChooserContextFactory::~HidChooserContextFactory() = default; -KeyedService* HidChooserContextFactory::BuildServiceInstanceFor( +std::unique_ptr +HidChooserContextFactory::BuildServiceInstanceForBrowserContext( content::BrowserContext* context) const { - auto* browser_context = - static_cast(context); - return new HidChooserContext(browser_context); + return std::make_unique( + static_cast(context)); } content::BrowserContext* HidChooserContextFactory::GetBrowserContextToUse( diff --git a/shell/browser/hid/hid_chooser_context_factory.h b/shell/browser/hid/hid_chooser_context_factory.h index 26ea0d0e67d5..3a2ae99c1bfc 100644 --- a/shell/browser/hid/hid_chooser_context_factory.h +++ b/shell/browser/hid/hid_chooser_context_factory.h @@ -5,6 +5,8 @@ #ifndef ELECTRON_SHELL_BROWSER_HID_HID_CHOOSER_CONTEXT_FACTORY_H_ #define ELECTRON_SHELL_BROWSER_HID_HID_CHOOSER_CONTEXT_FACTORY_H_ +#include + #include "base/no_destructor.h" #include "components/keyed_service/content/browser_context_keyed_service_factory.h" @@ -31,7 +33,7 @@ class HidChooserContextFactory : public BrowserContextKeyedServiceFactory { ~HidChooserContextFactory() override; // BrowserContextKeyedServiceFactory: - KeyedService* BuildServiceInstanceFor( + std::unique_ptr BuildServiceInstanceForBrowserContext( content::BrowserContext* profile) const override; content::BrowserContext* GetBrowserContextToUse( content::BrowserContext* context) const override; diff --git a/shell/browser/net/network_context_service_factory.cc b/shell/browser/net/network_context_service_factory.cc index b74c5671b9bb..ecfd842d5d0a 100644 --- a/shell/browser/net/network_context_service_factory.cc +++ b/shell/browser/net/network_context_service_factory.cc @@ -29,9 +29,10 @@ NetworkContextServiceFactory::NetworkContextServiceFactory() NetworkContextServiceFactory::~NetworkContextServiceFactory() = default; -KeyedService* NetworkContextServiceFactory::BuildServiceInstanceFor( +std::unique_ptr +NetworkContextServiceFactory::BuildServiceInstanceForBrowserContext( content::BrowserContext* context) const { - return new NetworkContextService( + return std::make_unique( static_cast(context)); } diff --git a/shell/browser/net/network_context_service_factory.h b/shell/browser/net/network_context_service_factory.h index 5f997f37b20b..da723b80b3ad 100644 --- a/shell/browser/net/network_context_service_factory.h +++ b/shell/browser/net/network_context_service_factory.h @@ -5,6 +5,8 @@ #ifndef ELECTRON_SHELL_BROWSER_NET_NETWORK_CONTEXT_SERVICE_FACTORY_H_ #define ELECTRON_SHELL_BROWSER_NET_NETWORK_CONTEXT_SERVICE_FACTORY_H_ +#include + #include "components/keyed_service/content/browser_context_keyed_service_factory.h" class KeyedService; @@ -43,7 +45,7 @@ class NetworkContextServiceFactory : public BrowserContextKeyedServiceFactory { ~NetworkContextServiceFactory() override; // BrowserContextKeyedServiceFactory implementation: - KeyedService* BuildServiceInstanceFor( + std::unique_ptr BuildServiceInstanceForBrowserContext( content::BrowserContext* context) const override; content::BrowserContext* GetBrowserContextToUse( content::BrowserContext* context) const override; diff --git a/shell/browser/serial/serial_chooser_context_factory.cc b/shell/browser/serial/serial_chooser_context_factory.cc index 5524af7d94a2..25e01287a6e4 100644 --- a/shell/browser/serial/serial_chooser_context_factory.cc +++ b/shell/browser/serial/serial_chooser_context_factory.cc @@ -18,11 +18,11 @@ SerialChooserContextFactory::SerialChooserContextFactory() SerialChooserContextFactory::~SerialChooserContextFactory() = default; -KeyedService* SerialChooserContextFactory::BuildServiceInstanceFor( +std::unique_ptr +SerialChooserContextFactory::BuildServiceInstanceForBrowserContext( content::BrowserContext* context) const { - auto* browser_context = - static_cast(context); - return new SerialChooserContext(browser_context); + return std::make_unique( + static_cast(context)); } // static diff --git a/shell/browser/serial/serial_chooser_context_factory.h b/shell/browser/serial/serial_chooser_context_factory.h index 9e73c428952b..f738de85401c 100644 --- a/shell/browser/serial/serial_chooser_context_factory.h +++ b/shell/browser/serial/serial_chooser_context_factory.h @@ -5,6 +5,8 @@ #ifndef ELECTRON_SHELL_BROWSER_SERIAL_SERIAL_CHOOSER_CONTEXT_FACTORY_H_ #define ELECTRON_SHELL_BROWSER_SERIAL_SERIAL_CHOOSER_CONTEXT_FACTORY_H_ +#include + #include "components/keyed_service/content/browser_context_keyed_service_factory.h" #include "shell/browser/serial/serial_chooser_context.h" @@ -35,7 +37,7 @@ class SerialChooserContextFactory : public BrowserContextKeyedServiceFactory { delete; // BrowserContextKeyedServiceFactory methods: - KeyedService* BuildServiceInstanceFor( + std::unique_ptr BuildServiceInstanceForBrowserContext( content::BrowserContext* context) const override; content::BrowserContext* GetBrowserContextToUse( content::BrowserContext* context) const override; diff --git a/shell/browser/usb/usb_chooser_context_factory.cc b/shell/browser/usb/usb_chooser_context_factory.cc index acbd8949dfac..5b8ae2d7d218 100644 --- a/shell/browser/usb/usb_chooser_context_factory.cc +++ b/shell/browser/usb/usb_chooser_context_factory.cc @@ -18,11 +18,11 @@ UsbChooserContextFactory::UsbChooserContextFactory() UsbChooserContextFactory::~UsbChooserContextFactory() = default; -KeyedService* UsbChooserContextFactory::BuildServiceInstanceFor( +std::unique_ptr +UsbChooserContextFactory::BuildServiceInstanceForBrowserContext( content::BrowserContext* context) const { - auto* browser_context = - static_cast(context); - return new UsbChooserContext(browser_context); + return std::make_unique( + static_cast(context)); } // static diff --git a/shell/browser/usb/usb_chooser_context_factory.h b/shell/browser/usb/usb_chooser_context_factory.h index 731c56d5f8e4..99c09645be81 100644 --- a/shell/browser/usb/usb_chooser_context_factory.h +++ b/shell/browser/usb/usb_chooser_context_factory.h @@ -5,6 +5,8 @@ #ifndef ELECTRON_SHELL_BROWSER_USB_USB_CHOOSER_CONTEXT_FACTORY_H_ #define ELECTRON_SHELL_BROWSER_USB_USB_CHOOSER_CONTEXT_FACTORY_H_ +#include + #include "components/keyed_service/content/browser_context_keyed_service_factory.h" namespace base { @@ -34,7 +36,7 @@ class UsbChooserContextFactory : public BrowserContextKeyedServiceFactory { ~UsbChooserContextFactory() override; // BrowserContextKeyedServiceFactory methods: - KeyedService* BuildServiceInstanceFor( + std::unique_ptr BuildServiceInstanceForBrowserContext( content::BrowserContext* profile) const override; };