Expose passwords to JavaScript in Credential Manager API

third_party/WebKit/Source/modules/credentialmanager/PasswordCredential.idl

Index: third_party/WebKit/Source/modules/credentialmanager/PasswordCredential.idl
diff --git a/third_party/WebKit/Source/modules/credentialmanager/PasswordCredential.idl b/third_party/WebKit/Source/modules/credentialmanager/PasswordCredential.idl
index 993920d8a880be288493240522bf4dc9e37d7a7d..672b405cd8f382fa6dcd58a2ca3cc8f83ec22246 100644
--- a/third_party/WebKit/Source/modules/credentialmanager/PasswordCredential.idl
+++ b/third_party/WebKit/Source/modules/credentialmanager/PasswordCredential.idl
@@ -13,8 +13,9 @@ typedef (FormData or URLSearchParams) CredentialBodyType;
     Exposed=Window,
     SecureContext
 ] interface PasswordCredential : Credential {
-    attribute USVString idName;
-    attribute USVString passwordName;
-    attribute CredentialBodyType? additionalData;
+    [DeprecateAs=CredentialManagerIdName] attribute USVString idName;
+    [DeprecateAs=CredentialManagerPasswordName] attribute USVString passwordName;
+    [DeprecateAs=CredentialManagerAdditionalData] attribute CredentialBodyType? additionalData;

[jdoerrie (Google - do not use), 2017/05/16 10:08:00]

Ideally the deprecation messages would only appear when the experimental feature
is enabled, but I couldn't figure out how to do it in the IDL. Simply writing
[DeprecateAs=CredentialManagerAdditionalData,RuntimeEnabled=PasswordCredentialPassword]
wouldn't work, because then the whole method is only available in the
experimental context.

I thought about adding a conditional Deprecation message to the getter and
setter (similarly to the change in RequestInit.cpp), but I don't know how get
hold of the context then which I need to pass to the CountDeprecation message.

What is your opinion here?

+    [RuntimeEnabled=PasswordCredentialPassword] readonly attribute DOMString password;
 };
 PasswordCredential implements CredentialUserData;