Credential Manager: Pass the correct credential type to the embedder.

Source/platform/exported/WebCredential.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 "public/platform/WebCredential.h"
 
 #include "platform/credentialmanager/PlatformCredential.h"
+#include "public/platform/WebFederatedCredential.h"
+#include "public/platform/WebLocalCredential.h"
 
 namespace blink {
 
+WebCredential WebCredential::create(PlatformCredential* credential)
+{
+    ASSERT(credential->isLocal() || credential->isFederated());
+
+    if (credential->isLocal()) {
+        WebLocalCredential local(credential);
+        return local;
+    }
+
+    WebFederatedCredential federated(credential);
+    return federated;

[kenneth.r.christiansen, 2014/10/09 13:34:02]

Wouldn´t it be better to add this in an if as well with a return and then add an
assert after if nothing was catched?

+}
+
 WebCredential::WebCredential(const WebString& id, const WebString& name, const WebURL& avatarURL)
     : m_platformCredential(PlatformCredential::create(id, name, avatarURL))
 {
 }
 
+WebCredential::WebCredential(const WebCredential& other)
+{
+    assign(other);
+}
+
 void WebCredential::assign(const WebCredential& other)
 {
     m_platformCredential = other.m_platformCredential;
 }
 
 WebCredential::WebCredential(PlatformCredential* credential)
     : m_platformCredential(credential)
 {
 }
 
@@ skipping 16 matching lines @@
 WebString WebCredential::name() const
 {
     return m_platformCredential->name();
 }
 
 WebURL WebCredential::avatarURL() const
 {
     return m_platformCredential->avatarURL();
 }
 
+bool WebCredential::isLocalCredential() const
+{
+    return m_platformCredential->isLocal();
+}
+
+bool WebCredential::isFederatedCredential() const
+{
+    return m_platformCredential->isFederated();
+}
+
 } // namespace blink