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

Unified Diff: chrome/browser/extensions/api/networking_private/networking_private_api_nonchromeos.cc

Issue 102993002: Implement Networking Private API VerifyAndEncryptCredentials method (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address codereview comments. Created 6 years, 10 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/networking_private/networking_private_api_nonchromeos.cc
diff --git a/chrome/browser/extensions/api/networking_private/networking_private_api_nonchromeos.cc b/chrome/browser/extensions/api/networking_private/networking_private_api_nonchromeos.cc
index 74c42b0804819671fe6cbcd8389313db2e73eaad..b0fce0ae6ce94d89a6eb8e54f659353f70056c56 100644
--- a/chrome/browser/extensions/api/networking_private/networking_private_api_nonchromeos.cc
+++ b/chrome/browser/extensions/api/networking_private/networking_private_api_nonchromeos.cc
@@ -410,11 +410,32 @@ bool NetworkingPrivateVerifyAndEncryptCredentialsFunction::RunImpl() {
scoped_ptr<api::VerifyAndEncryptCredentials::Params> params =
api::VerifyAndEncryptCredentials::Params::Create(*args_);
EXTENSION_FUNCTION_VALIDATE(params);
- SetResult(new base::StringValue("encrypted_credentials"));
- SendResponse(true);
+ NetworkingPrivateServiceClient* service_client =
+ NetworkingPrivateServiceClientFactory::GetForProfile(GetProfile());
+ service_client->VerifyAndEncryptCredentials(
+ args_.Pass(),
+ base::Bind(
+ &NetworkingPrivateVerifyAndEncryptCredentialsFunction::ResultCallback,
+ this),
+ base::Bind(
+ &NetworkingPrivateVerifyAndEncryptCredentialsFunction::ErrorCallback,
+ this));
return true;
}
+void NetworkingPrivateVerifyAndEncryptCredentialsFunction::ResultCallback(
+ const std::string& result) {
+ SetResult(new base::StringValue(result));
+ SendResponse(true);
+}
+
+void NetworkingPrivateVerifyAndEncryptCredentialsFunction::ErrorCallback(
+ const std::string& error_name,
+ const std::string& error) {
+ error_ = error_name;
+ SendResponse(false);
+}
+
////////////////////////////////////////////////////////////////////////////////
// NetworkingPrivateVerifyAndEncryptDataFunction

Powered by Google App Engine
This is Rietveld 408576698