chrome.platformKeys.getKeyPair: Check requested algorithm against certificate.

chrome/common/extensions/api/platform_keys_internal.idl

Index: chrome/common/extensions/api/platform_keys_internal.idl
diff --git a/chrome/common/extensions/api/platform_keys_internal.idl b/chrome/common/extensions/api/platform_keys_internal.idl
index d3338b4fcabbd652ed07554e817398b031fee03d..463fded1447510fd4e19496324e5c209af3f9107 100644
--- a/chrome/common/extensions/api/platform_keys_internal.idl
+++ b/chrome/common/extensions/api/platform_keys_internal.idl
@@ -4,7 +4,7 @@
 
 // Internal API for to implement the platformKeys and enterprise.platformKeys
 // APIs.
-[ implemented_in = "chrome/browser/extensions/api/platform_keys/platform_keys_api.h" ]
+[implemented_in = "chrome/browser/extensions/api/platform_keys/platform_keys_api.h"]
 namespace platformKeysInternal {
   callback SelectCallback = void (platformKeys.Match[] certs);
 
@@ -48,9 +48,13 @@ namespace platformKeysInternal {
                      ArrayBuffer data,
                      SignCallback callback);
 
-    // Calls back <code>callback</code> with details about the key certified by
-    // <code>certificate</code>.
+    // Checks whether <code>certificate</code> certifies a key that allows usage
+    // of the WebCrypto algorithm <code>algorithmName</code>. If so, calls back

[Andrew T Wilson (Slow), 2015/03/19 07:38:12]

Maybe document what happens if there is no allowable key? I guess the callback
is not invoked and it just silently fails?

[pneubeck (no reviews), 2015/03/23 13:43:12]

Error handling for extension APIs is very standardized: callback is called with
chrome.runtime.lastError set.
I added a comment that an errors is thrown.

+    // <code>callback</code> with the key info and a WebCrypto
+    // <code>KeyAlgorithm</code> dictionary describing the key's algorithm. The
+    // <code>name</code> property will equal <code>algorithmName</code>.
     static void getPublicKey(ArrayBuffer certificate,
+                             DOMString algorithmName,
                              GetPublicKeyCallback callback);
   };
 };