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

Unified Diff: chrome/browser/extensions/api/identity/identity_api.cc

Issue 274853002: Identity API: add chrome.identity.getAccounts (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 7 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/identity/identity_api.cc
diff --git a/chrome/browser/extensions/api/identity/identity_api.cc b/chrome/browser/extensions/api/identity/identity_api.cc
index df6fdbd65824437d0fedae3b5c68460664692143..ae7461257933992a88cd0081b763c333746963b4 100644
--- a/chrome/browser/extensions/api/identity/identity_api.cc
+++ b/chrome/browser/extensions/api/identity/identity_api.cc
@@ -28,6 +28,7 @@
#include "chrome/common/url_constants.h"
#include "components/signin/core/browser/profile_oauth2_token_service.h"
#include "components/signin/core/browser/signin_manager.h"
+#include "components/signin/core/common/profile_management_switches.h"
#include "extensions/browser/event_router.h"
#include "extensions/browser/extension_function_dispatcher.h"
#include "extensions/common/extension.h"
@@ -168,6 +169,23 @@ const IdentityAPI::CachedTokens& IdentityAPI::GetAllCachedTokens() {
return token_cache_;
}
+std::vector<std::string> IdentityAPI::GetAccounts() const {
+ const std::vector<AccountIds> ids = account_tracker_.GetAccounts();
+ std::vector<std::string> gaia_ids;
+
+ if (switches::IsExtensionsMultiAccount()) {
+ for (std::vector<AccountIds>::const_iterator it = ids.begin();
+ it != ids.end();
+ ++it) {
+ gaia_ids.push_back(it->gaia);
+ }
+ } else if (ids.size() >= 1) {
+ gaia_ids.push_back(ids[0].gaia);
+ }
+
+ return gaia_ids;
+}
+
void IdentityAPI::ReportAuthError(const GoogleServiceAuthError& error) {
account_tracker_.ReportAuthError(GetPrimaryAccountId(browser_context_),
error);
@@ -197,6 +215,12 @@ void IdentityAPI::OnAccountRemoved(const AccountIds& ids) {}
void IdentityAPI::OnAccountSignInChanged(const AccountIds& ids,
bool is_signed_in) {
+ const std::string primary_account_id = GetPrimaryAccountId(browser_context_);
+ if (primary_account_id != ids.account_key &&
+ !switches::IsExtensionsMultiAccount()) {
+ return;
+ }
+
api::identity::AccountInfo account_info;
account_info.id = ids.gaia;
@@ -217,12 +241,44 @@ void IdentityAPI::RemoveShutdownObserver(ShutdownObserver* observer) {
shutdown_observer_list_.RemoveObserver(observer);
}
+void IdentityAPI::SetAccountStateForTest(AccountIds ids, bool is_signed_in) {
+ account_tracker_.SetAccountStateForTest(ids, is_signed_in);
+}
+
template <>
void BrowserContextKeyedAPIFactory<IdentityAPI>::DeclareFactoryDependencies() {
DependsOn(ExtensionsBrowserClient::Get()->GetExtensionSystemFactory());
DependsOn(ProfileOAuth2TokenServiceFactory::GetInstance());
}
+IdentityGetAccountsFunction::IdentityGetAccountsFunction() {
+}
+
+IdentityGetAccountsFunction::~IdentityGetAccountsFunction() {
+}
+
+ExtensionFunction::ResponseAction IdentityGetAccountsFunction::Run() {
+ if (GetProfile()->IsOffTheRecord()) {
+ return RespondNow(Error(identity_constants::kOffTheRecord));
+ }
+
+ std::vector<std::string> gaia_ids =
+ IdentityAPI::GetFactoryInstance()->Get(GetProfile())->GetAccounts();
+ DCHECK(gaia_ids.size() < 2 || switches::IsExtensionsMultiAccount());
+
+ base::ListValue* infos = new base::ListValue();
+
+ for (std::vector<std::string>::const_iterator it = gaia_ids.begin();
+ it != gaia_ids.end();
+ ++it) {
+ api::identity::AccountInfo account_info;
+ account_info.id = *it;
+ infos->Append(account_info.ToValue().release());
+ }
+
+ return RespondNow(MultipleArguments(infos));
+}
+
IdentityGetAuthTokenFunction::IdentityGetAuthTokenFunction()
: OAuth2TokenService::Consumer("extensions_identity_api"),
should_prompt_for_scopes_(false),
« no previous file with comments | « chrome/browser/extensions/api/identity/identity_api.h ('k') | chrome/browser/extensions/api/identity/identity_apitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698