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

Unified Diff: chrome/browser/extensions/api/screenlock_private/screenlock_private_api.cc

Issue 1096293003: Move screenlock_bridge to components/proximity_auth (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/screenlock_private/screenlock_private_api.cc
diff --git a/chrome/browser/extensions/api/screenlock_private/screenlock_private_api.cc b/chrome/browser/extensions/api/screenlock_private/screenlock_private_api.cc
index 78d1c4907185a2790a18718020d7dd57fb664fdb..f5c59e8c58a6cf6ea6e634d329f2b8f740374fe5 100644
--- a/chrome/browser/extensions/api/screenlock_private/screenlock_private_api.cc
+++ b/chrome/browser/extensions/api/screenlock_private/screenlock_private_api.cc
@@ -8,6 +8,7 @@
#include "base/values.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/signin/easy_unlock_service.h"
+#include "chrome/browser/signin/easy_unlock_util.h"
#include "chrome/common/extensions/api/screenlock_private.h"
#include "chrome/common/extensions/extension_constants.h"
#include "extensions/browser/app_window/app_window_registry.h"
@@ -20,23 +21,23 @@ namespace extensions {
namespace {
screenlock::AuthType FromLockHandlerAuthType(
- ScreenlockBridge::LockHandler::AuthType auth_type) {
+ proximity_auth::ScreenlockBridge::LockHandler::AuthType auth_type) {
switch (auth_type) {
- case ScreenlockBridge::LockHandler::OFFLINE_PASSWORD:
+ case proximity_auth::ScreenlockBridge::LockHandler::OFFLINE_PASSWORD:
return screenlock::AUTH_TYPE_OFFLINEPASSWORD;
- case ScreenlockBridge::LockHandler::NUMERIC_PIN:
+ case proximity_auth::ScreenlockBridge::LockHandler::NUMERIC_PIN:
return screenlock::AUTH_TYPE_NUMERICPIN;
- case ScreenlockBridge::LockHandler::USER_CLICK:
+ case proximity_auth::ScreenlockBridge::LockHandler::USER_CLICK:
return screenlock::AUTH_TYPE_USERCLICK;
- case ScreenlockBridge::LockHandler::ONLINE_SIGN_IN:
+ case proximity_auth::ScreenlockBridge::LockHandler::ONLINE_SIGN_IN:
// Apps should treat forced online sign in same as system password.
return screenlock::AUTH_TYPE_OFFLINEPASSWORD;
- case ScreenlockBridge::LockHandler::EXPAND_THEN_USER_CLICK:
+ case proximity_auth::ScreenlockBridge::LockHandler::EXPAND_THEN_USER_CLICK:
// This type is used for public sessions, which do not support screen
// locking.
NOTREACHED();
return screenlock::AUTH_TYPE_NONE;
- case ScreenlockBridge::LockHandler::FORCE_OFFLINE_PASSWORD:
+ case proximity_auth::ScreenlockBridge::LockHandler::FORCE_OFFLINE_PASSWORD:
return screenlock::AUTH_TYPE_OFFLINEPASSWORD;
}
NOTREACHED();
@@ -50,7 +51,8 @@ ScreenlockPrivateGetLockedFunction::ScreenlockPrivateGetLockedFunction() {}
ScreenlockPrivateGetLockedFunction::~ScreenlockPrivateGetLockedFunction() {}
bool ScreenlockPrivateGetLockedFunction::RunAsync() {
- SetResult(new base::FundamentalValue(ScreenlockBridge::Get()->IsLocked()));
+ SetResult(
+ new base::FundamentalValue(GetScreenlockBridgeInstance()->IsLocked()));
SendResponse(error_.empty());
return true;
}
@@ -75,9 +77,9 @@ bool ScreenlockPrivateSetLockedFunction::RunAsync() {
if (service)
service->SetTrialRun();
}
- ScreenlockBridge::Get()->Lock(GetProfile());
+ GetScreenlockBridgeInstance()->Lock(GetProfile());
} else {
- ScreenlockBridge::Get()->Unlock(GetProfile());
+ GetScreenlockBridgeInstance()->Unlock(GetProfile());
}
SendResponse(error_.empty());
return true;
@@ -104,19 +106,19 @@ bool ScreenlockPrivateAcceptAuthAttemptFunction::RunSync() {
ScreenlockPrivateEventRouter::ScreenlockPrivateEventRouter(
content::BrowserContext* context)
: browser_context_(context) {
- ScreenlockBridge::Get()->AddObserver(this);
+ GetScreenlockBridgeInstance()->AddObserver(this);
}
ScreenlockPrivateEventRouter::~ScreenlockPrivateEventRouter() {}
void ScreenlockPrivateEventRouter::OnScreenDidLock(
- ScreenlockBridge::LockHandler::ScreenType screen_type) {
+ proximity_auth::ScreenlockBridge::LockHandler::ScreenType screen_type) {
DispatchEvent(screenlock::OnChanged::kEventName,
new base::FundamentalValue(true));
}
void ScreenlockPrivateEventRouter::OnScreenDidUnlock(
- ScreenlockBridge::LockHandler::ScreenType screen_type) {
+ proximity_auth::ScreenlockBridge::LockHandler::ScreenType screen_type) {
DispatchEvent(screenlock::OnChanged::kEventName,
new base::FundamentalValue(false));
}
@@ -146,11 +148,11 @@ ScreenlockPrivateEventRouter::GetFactoryInstance() {
}
void ScreenlockPrivateEventRouter::Shutdown() {
- ScreenlockBridge::Get()->RemoveObserver(this);
+ GetScreenlockBridgeInstance()->RemoveObserver(this);
}
bool ScreenlockPrivateEventRouter::OnAuthAttempted(
- ScreenlockBridge::LockHandler::AuthType auth_type,
+ proximity_auth::ScreenlockBridge::LockHandler::AuthType auth_type,
const std::string& value) {
extensions::EventRouter* router =
extensions::EventRouter::Get(browser_context_);

Powered by Google App Engine
This is Rietveld 408576698