Credential manager: Introduce platform interface.

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 "platform/weborigin/SecurityOrigin.h"
 #include "public/platform/Platform.h"
+#include "public/platform/WebCredential.h"
+#include "public/platform/WebCredentialManager.h"
+#include "public/platform/WebCredentialManagerError.h"
 
 namespace blink {
 
+static void rejectDueToCredentialManagerError(PassRefPtr<ScriptPromiseResolver> resolver, WebCredentialManagerError* reason)
+{
+
+    switch (reason->errorType) {
+    case WebCredentialManagerError::DisabledErrorType:
+        resolver->reject(DOMException::create(InvalidStateError, "The credential manager is disabled."));
+        break;
+    case WebCredentialManagerError::UnknownErrorType:
+        resolver->reject(DOMException::create(NotReadableError, "An unknown error occured while talking to the credential manager."));
+        break;

[tkent, 2014/08/11 08:43:50]

Will the break-then-ASSERT_NO_REACHED work well?

[Mike West, 2014/08/11 09:13:40]

You're right, this is still prototypey-enough that it's not entirely clear what
kind of errors we'll need to support yet. I'll switch it to `default:` for the
moment, and move back to the break/assert mode once this code solidified.

+    };

[tkent, 2014/08/11 08:43:50]

; is unnecessary.

[Mike West, 2014/08/11 09:13:40]

Done.

+    ASSERT_NOT_REACHED();
+    resolver->reject(DOMException::create(InvalidStateError));
+}
+
+class NotificationCallbacks : public WebCredentialManager::NotificationCallbacks {

[yhirano, 2014/08/11 08:58:48]

How about enclosing this class with an unnamed namespace?

+    WTF_MAKE_NONCOPYABLE(NotificationCallbacks);
+public:
+    explicit NotificationCallbacks(PassRefPtr<ScriptPromiseResolver> resolver) : m_resolver(resolver) { }
+    virtual ~NotificationCallbacks() { }
+
+    virtual void onSuccess() OVERRIDE
+    {
+        m_resolver->resolve();
+    }
+
+    virtual void onError(WebCredentialManagerError* reason) OVERRIDE

[yhirano, 2014/08/11 08:58:48]

IIRC WebCallbacks handler parameter ownership is transferred to the callee (e.g.
[1]). Shouldn't you release |reason| here?

https://code.google.com/p/chromium/codesearch#chromium/src/content/renderer/p...

[Mike West, 2014/08/11 09:13:40]

Hrm. My impression was that CallbackPromiseAdaptor had this relationship, but it
wasn't clear from the code whether WebCallbacks directly took ownership. Similar
code in ServiceWorker doesn't delete, for instance:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

[yhirano, 2014/08/12 06:55:28]

OK, I see.

+    {
+        rejectDueToCredentialManagerError(m_resolver, reason);
+    }
+
+private:
+    const RefPtr<ScriptPromiseResolver> m_resolver;
+};
+
+class RequestCallbacks : public WebCredentialManager::RequestCallbacks {

[yhirano, 2014/08/11 08:58:48]

ditto

+    WTF_MAKE_NONCOPYABLE(RequestCallbacks);
+public:
+    explicit RequestCallbacks(PassRefPtr<ScriptPromiseResolver> resolver) : m_resolver(resolver) { }
+    virtual ~RequestCallbacks() { }
+
+    virtual void onSuccess(WebCredential* credential) OVERRIDE

[yhirano, 2014/08/11 08:58:48]

ditto

+    {
+        m_resolver->resolve(Credential::create(credential->id(), credential->name(), credential->avatarURL()));
+    }
+
+    virtual void onError(WebCredentialManagerError* reason) OVERRIDE

[yhirano, 2014/08/11 08:58:48]

ditto

+    {
+        rejectDueToCredentialManagerError(m_resolver, reason);
+    }
+
+private:
+    const RefPtr<ScriptPromiseResolver> m_resolver;
+};
+
+
 CredentialsContainer* CredentialsContainer::create()
 {
     return new CredentialsContainer();
 }
 
 CredentialsContainer::CredentialsContainer()
 {
     ScriptWrappable::init(this);
 }
 
@@ skipping 34 matching lines @@
 {
     return stubImplementation(scriptState);
 }
 
 ScriptPromise CredentialsContainer::notifySignedOut(ScriptState* scriptState)
 {
     return stubImplementation(scriptState);
 }
 
 } // namespace blink