Avoid object slicing in WebCredential::create

third_party/WebKit/Source/platform/exported/WebCredential.cpp

Index: third_party/WebKit/Source/platform/exported/WebCredential.cpp
diff --git a/third_party/WebKit/Source/platform/exported/WebCredential.cpp b/third_party/WebKit/Source/platform/exported/WebCredential.cpp
index 11125ecd1d40121ef026aa466818c24856aa1fee..808ce81e13ffa9a4b04149f4ad600129eab96ae7 100644
--- a/third_party/WebKit/Source/platform/exported/WebCredential.cpp
+++ b/third_party/WebKit/Source/platform/exported/WebCredential.cpp
@@ -4,26 +4,26 @@
 
 #include "public/platform/WebCredential.h"
 
+#include "base/memory/ptr_util.h"
+
 #include "platform/credentialmanager/PlatformCredential.h"
 #include "public/platform/WebFederatedCredential.h"
 #include "public/platform/WebPasswordCredential.h"
 
 namespace blink {
 
-WebCredential WebCredential::create(PlatformCredential* credential)
+std::unique_ptr<WebCredential> WebCredential::create(PlatformCredential* credential)
 {
     if (credential->isPassword()) {
-        WebPasswordCredential password(credential);
-        return password;
+        return base::WrapUnique(new WebPasswordCredential(credential));

[haraken, 2016/05/11 14:16:11]

yutak@: We don't want to use base::WrapUnique, right?

std::unique_ptr(new WebPasswordCredential(credential));

[vabr (Chromium), 2016/05/11 14:42:58]

What about using wrapUnique from Source/wtf/PtrUtil.h, introduced in
https://codereview.chromium.org/1865913005?
That would avoid the type duplication in 
std::unique_ptr<X>(new X)

     }
 
     if (credential->isFederated()) {
-        WebFederatedCredential federated(credential);
-        return federated;
+        return base::WrapUnique(new WebFederatedCredential(credential));

[haraken, 2016/05/11 14:16:11]

Ditto.

     }
 
     ASSERT_NOT_REACHED();
-    return WebCredential(credential);
+    return nullptr;
 }
 
 WebCredential::WebCredential(const WebString& id, const WebString& name, const WebURL& iconURL)