Credential Manager: Wire up the client.

Source/modules/credentialmanager/CredentialsContainer.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "modules/credentialmanager/CredentialsContainer.h"
 
 #include "bindings/core/v8/Dictionary.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "core/dom/DOMException.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/dom/ExecutionContext.h"
 #include "modules/credentialmanager/Credential.h"
+#include "modules/credentialmanager/CredentialManagerClient.h"
+#include "modules/credentialmanager/LocalCredential.h"
 #include "platform/weborigin/SecurityOrigin.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebCredential.h"
 #include "public/platform/WebCredentialManagerClient.h"
 #include "public/platform/WebCredentialManagerError.h"
+#include "public/platform/WebLocalCredential.h"
 
 namespace blink {
 
 static void rejectDueToCredentialManagerError(PassRefPtr<ScriptPromiseResolver> resolver, WebCredentialManagerError* reason)
 {
 
     switch (reason->errorType) {
     case WebCredentialManagerError::ErrorTypeDisabled:
         resolver->reject(DOMException::create(InvalidStateError, "The credential manager is disabled."));
         break;
@@ skipping 25 matching lines @@
 };
 
 class RequestCallbacks : public WebCredentialManagerClient::RequestCallbacks {
     WTF_MAKE_NONCOPYABLE(RequestCallbacks);
 public:
     explicit RequestCallbacks(PassRefPtr<ScriptPromiseResolver> resolver) : m_resolver(resolver) { }
     virtual ~RequestCallbacks() { }
 
     virtual void onSuccess(WebCredential* credential) OVERRIDE
     {
-        // FIXME: Split this into Local/Federated types.
-        m_resolver->resolve(Credential::create(credential->id(), credential->name(), credential->avatarURL()));
+        if (!credential) {
+            m_resolver->resolve();
+            return;
+        }
+
+        // FIXME: Split this into Local/Federated types. Right now it's hard-coded to be a LocalCredential. :(
+        m_resolver->resolve(LocalCredential::create(credential->id(), credential->name(), credential->avatarURL(), static_cast<WebLocalCredential*>(credential)->password()));
     }
 
     virtual void onError(WebCredentialManagerError* reason) OVERRIDE
     {
         rejectDueToCredentialManagerError(m_resolver, reason);
     }
 
 private:
     const RefPtr<ScriptPromiseResolver> m_resolver;
 };
@@ skipping 12 matching lines @@
 {
     SecurityOrigin* securityOrigin = scriptState->executionContext()->securityOrigin();
     String errorMessage;
     if (!securityOrigin->canAccessFeatureRequiringSecureOrigin(errorMessage)) {
         resolver->reject(DOMException::create(SecurityError, errorMessage));
         return false;
     }
     return true;
 }
 
-static ScriptPromise stubImplementation(ScriptState* scriptState)
+ScriptPromise CredentialsContainer::request(ScriptState* scriptState, const Dictionary&)
 {
+    CredentialManagerClient* client = CredentialManagerClient::from(scriptState);
+    if (!client)
+        return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(InvalidStateError, "Could not establish connection to the credential manager."));
+
     RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);
     ScriptPromise promise = resolver->promise();
 
     if (!canAccessCredentialManagerAPI(scriptState, resolver))
         return promise;
 
-    resolver->resolve();
+    WebVector<WebURL> tempVector;
+    client->dispatchRequest(false, tempVector, new RequestCallbacks(resolver));
     return promise;
 }
 
-ScriptPromise CredentialsContainer::request(ScriptState* scriptState, const Dictionary&)
-{
-    return stubImplementation(scriptState);
-}
-
 ScriptPromise CredentialsContainer::notifySignedIn(ScriptState* scriptState, Credential* credential)
 {
-    return stubImplementation(scriptState);
+    CredentialManagerClient* client = CredentialManagerClient::from(scriptState);

[sof, 2014/09/15 07:57:17]

There's a fair bit of duplicated code in these methods & notifications; possible
to reduce?

+    if (!client)
+        return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(InvalidStateError, "Could not establish connection to the credential manager."));
+
+    RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);
+    ScriptPromise promise = resolver->promise();
+
+    if (!canAccessCredentialManagerAPI(scriptState, resolver))
+        return promise;
+
+    client->dispatchSignedIn(WebCredential(credential->platformCredential()), new NotificationCallbacks(resolver));
+    return promise;
 }
 
 ScriptPromise CredentialsContainer::notifyFailedSignIn(ScriptState* scriptState, Credential* credential)
 {
-    return stubImplementation(scriptState);
+    CredentialManagerClient* client = CredentialManagerClient::from(scriptState);
+    if (!client)
+        return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(InvalidStateError, "Could not establish connection to the credential manager."));
+
+    RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);
+    ScriptPromise promise = resolver->promise();
+
+    if (!canAccessCredentialManagerAPI(scriptState, resolver))
+        return promise;
+
+    client->dispatchFailedSignIn(WebCredential(credential->platformCredential()), new NotificationCallbacks(resolver));
+    return promise;
 }
 
 ScriptPromise CredentialsContainer::notifySignedOut(ScriptState* scriptState)
 {
-    return stubImplementation(scriptState);
+    CredentialManagerClient* client = CredentialManagerClient::from(scriptState);
+    if (!client)
+        return ScriptPromise::rejectWithDOMException(scriptState, DOMException::create(InvalidStateError, "Could not establish connection to the credential manager."));
+
+    RefPtr<ScriptPromiseResolver> resolver = ScriptPromiseResolver::create(scriptState);
+    ScriptPromise promise = resolver->promise();
+
+    if (!canAccessCredentialManagerAPI(scriptState, resolver))
+        return promise;
+
+    client->dispatchSignedOut(new NotificationCallbacks(resolver));
+    return promise;
 }
 
 } // namespace blink