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

Side by Side Diff: extensions/browser/api/power/power_api.cc

Issue 1902873002: Convert //extensions/browser/api from scoped_ptr to std::unique_ptr (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "extensions/browser/api/power/power_api.h" 5 #include "extensions/browser/api/power/power_api.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/lazy_instance.h" 8 #include "base/lazy_instance.h"
9 #include "extensions/browser/extension_registry.h" 9 #include "extensions/browser/extension_registry.h"
10 #include "extensions/common/api/power.h" 10 #include "extensions/common/api/power.h"
(...skipping 17 matching lines...) Expand all
28 NOTREACHED() << "Unhandled level " << level; 28 NOTREACHED() << "Unhandled level " << level;
29 return content::PowerSaveBlocker::kPowerSaveBlockPreventDisplaySleep; 29 return content::PowerSaveBlocker::kPowerSaveBlockPreventDisplaySleep;
30 } 30 }
31 31
32 base::LazyInstance<BrowserContextKeyedAPIFactory<PowerAPI>> g_factory = 32 base::LazyInstance<BrowserContextKeyedAPIFactory<PowerAPI>> g_factory =
33 LAZY_INSTANCE_INITIALIZER; 33 LAZY_INSTANCE_INITIALIZER;
34 34
35 } // namespace 35 } // namespace
36 36
37 bool PowerRequestKeepAwakeFunction::RunSync() { 37 bool PowerRequestKeepAwakeFunction::RunSync() {
38 scoped_ptr<api::power::RequestKeepAwake::Params> params( 38 std::unique_ptr<api::power::RequestKeepAwake::Params> params(
39 api::power::RequestKeepAwake::Params::Create(*args_)); 39 api::power::RequestKeepAwake::Params::Create(*args_));
40 EXTENSION_FUNCTION_VALIDATE(params); 40 EXTENSION_FUNCTION_VALIDATE(params);
41 EXTENSION_FUNCTION_VALIDATE(params->level != api::power::LEVEL_NONE); 41 EXTENSION_FUNCTION_VALIDATE(params->level != api::power::LEVEL_NONE);
42 PowerAPI::Get(browser_context())->AddRequest(extension_id(), params->level); 42 PowerAPI::Get(browser_context())->AddRequest(extension_id(), params->level);
43 return true; 43 return true;
44 } 44 }
45 45
46 bool PowerReleaseKeepAwakeFunction::RunSync() { 46 bool PowerReleaseKeepAwakeFunction::RunSync() {
47 PowerAPI::Get(browser_context())->RemoveRequest(extension_id()); 47 PowerAPI::Get(browser_context())->RemoveRequest(extension_id());
48 return true; 48 return true;
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 if (it->second == api::power::LEVEL_DISPLAY) 105 if (it->second == api::power::LEVEL_DISPLAY)
106 new_level = it->second; 106 new_level = it->second;
107 } 107 }
108 108
109 // If the level changed and we need to create a new blocker, do a swap 109 // If the level changed and we need to create a new blocker, do a swap
110 // to ensure that there isn't a brief period where power management is 110 // to ensure that there isn't a brief period where power management is
111 // unblocked. 111 // unblocked.
112 if (!power_save_blocker_ || new_level != current_level_) { 112 if (!power_save_blocker_ || new_level != current_level_) {
113 content::PowerSaveBlocker::PowerSaveBlockerType type = 113 content::PowerSaveBlocker::PowerSaveBlockerType type =
114 LevelToPowerSaveBlockerType(new_level); 114 LevelToPowerSaveBlockerType(new_level);
115 scoped_ptr<content::PowerSaveBlocker> new_blocker( 115 std::unique_ptr<content::PowerSaveBlocker> new_blocker(
116 create_blocker_function_.Run(type, 116 create_blocker_function_.Run(type,
117 content::PowerSaveBlocker::kReasonOther, 117 content::PowerSaveBlocker::kReasonOther,
118 kPowerSaveBlockerDescription)); 118 kPowerSaveBlockerDescription));
119 power_save_blocker_.swap(new_blocker); 119 power_save_blocker_.swap(new_blocker);
120 current_level_ = new_level; 120 current_level_ = new_level;
121 } 121 }
122 } 122 }
123 123
124 void PowerAPI::Shutdown() { 124 void PowerAPI::Shutdown() {
125 // Unregister here rather than in the d'tor; otherwise this call will recreate 125 // Unregister here rather than in the d'tor; otherwise this call will recreate
126 // the already-deleted ExtensionRegistry. 126 // the already-deleted ExtensionRegistry.
127 ExtensionRegistry::Get(browser_context_)->RemoveObserver(this); 127 ExtensionRegistry::Get(browser_context_)->RemoveObserver(this);
128 power_save_blocker_.reset(); 128 power_save_blocker_.reset();
129 } 129 }
130 130
131 } // namespace extensions 131 } // namespace extensions
OLDNEW
« no previous file with comments | « extensions/browser/api/power/power_api.h ('k') | extensions/browser/api/power/power_api_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698