Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4)

Unified Diff: extensions/browser/api/power/power_api.cc

Issue 2073353002: Merge PowerSaveBlockerImpl and PowerSaveBlocker (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove-power-factory
Patch Set: android Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/browser/api/power/power_api.cc
diff --git a/extensions/browser/api/power/power_api.cc b/extensions/browser/api/power/power_api.cc
index 819089699f56e95975e3536caa57e767a7fc232b..62cb73699a0634f45c3a837b16e95c584a61d024 100644
--- a/extensions/browser/api/power/power_api.cc
+++ b/extensions/browser/api/power/power_api.cc
@@ -34,6 +34,16 @@ device::PowerSaveBlocker::PowerSaveBlockerType LevelToPowerSaveBlockerType(
base::LazyInstance<BrowserContextKeyedAPIFactory<PowerAPI>> g_factory =
LAZY_INSTANCE_INITIALIZER;
+std::unique_ptr<device::PowerSaveBlocker> CreatePowerSaveBlocker(
+ device::PowerSaveBlocker::PowerSaveBlockerType type,
+ device::PowerSaveBlocker::Reason reason,
+ const std::string& description,
+ scoped_refptr<base::SequencedTaskRunner> ui_task_runner,
+ scoped_refptr<base::SingleThreadTaskRunner> file_task_runner) {
+ return std::unique_ptr<device::PowerSaveBlocker>(new device::PowerSaveBlocker(
+ type, reason, description, ui_task_runner, file_task_runner));
+}
+
} // namespace
bool PowerRequestKeepAwakeFunction::RunSync() {
@@ -74,9 +84,7 @@ void PowerAPI::RemoveRequest(const std::string& extension_id) {
void PowerAPI::SetCreateBlockerFunctionForTesting(
CreateBlockerFunction function) {
create_blocker_function_ =
- !function.is_null()
- ? function
- : base::Bind(&device::PowerSaveBlocker::CreateWithTaskRunners);
+ !function.is_null() ? function : base::Bind(&CreatePowerSaveBlocker);
}
void PowerAPI::OnExtensionUnloaded(content::BrowserContext* browser_context,
@@ -88,8 +96,7 @@ void PowerAPI::OnExtensionUnloaded(content::BrowserContext* browser_context,
PowerAPI::PowerAPI(content::BrowserContext* context)
: browser_context_(context),
- create_blocker_function_(
- base::Bind(&device::PowerSaveBlocker::CreateWithTaskRunners)),
+ create_blocker_function_(base::Bind(&CreatePowerSaveBlocker)),
current_level_(api::power::LEVEL_SYSTEM) {
ExtensionRegistry::Get(browser_context_)->AddObserver(this);
}
« no previous file with comments | « device/power_save_blocker/power_save_blocker_x11.cc ('k') | extensions/browser/api/power/power_api_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698