Expose a constructor for WebCryptoAlgorithm.

public/platform/WebCryptoAlgorithm.h

Index: public/platform/WebCryptoAlgorithm.h
diff --git a/public/platform/WebCryptoAlgorithm.h b/public/platform/WebCryptoAlgorithm.h
index b1227328af7bebfd1fae24dc06fc25ec2c357b0b..5ae856a805ae3f576c5c58820dadd7e1cbdd6890 100644
--- a/public/platform/WebCryptoAlgorithm.h
+++ b/public/platform/WebCryptoAlgorithm.h
@@ -75,7 +75,12 @@ class WebCryptoAlgorithm {
 public:
 #if WEBKIT_IMPLEMENTATION
     WebCryptoAlgorithm() { }
-    WebCryptoAlgorithm(WebCryptoAlgorithmId, const char*, PassOwnPtr<WebCryptoAlgorithmParams>);
+    WebCryptoAlgorithm(WebCryptoAlgorithmId, const char* name, PassOwnPtr<WebCryptoAlgorithmParams>);
+#else
+    WebCryptoAlgorithm(WebCryptoAlgorithmId id, const char* name, WebCryptoAlgorithmParams* params)
+    {
+        assign(id, name, params);
+    }

[abarth-chromium, 2013/07/31 00:46:51]

It's not at all obvious that this constructor takes ownership of the
WebCryptoAlgorithmParams object.  Is there some way we could make that more
obvious?  Maybe instead of a constructor it should be a create function who's
name says something about adopting?

[eroman, 2013/07/31 01:49:26]

Good idea. Done

 #endif
 
     ~WebCryptoAlgorithm() { reset(); }
@@ -100,6 +105,7 @@ public:
 
 private:
     WEBKIT_EXPORT void assign(const WebCryptoAlgorithm& other);
+    WEBKIT_EXPORT void assign(WebCryptoAlgorithmId, const char* name, WebCryptoAlgorithmParams*);
     WEBKIT_EXPORT void reset();
 
     WebPrivatePtr<WebCryptoAlgorithmPrivate> m_private;