CREDENTIAL: Implement the 'PasswordCredential(HTMLFormElement)' constructor.

third_party/WebKit/Source/modules/credentialmanager/PasswordCredential.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 "modules/credentialmanager/PasswordCredential.h"
 
 #include "bindings/core/v8/Dictionary.h"
 #include "bindings/core/v8/ExceptionState.h"
+#include "core/HTMLNames.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/dom/URLSearchParams.h"
+#include "core/html/FormAssociatedElement.h"
 #include "core/html/FormData.h"
+#include "core/html/HTMLFormElement.h"
 #include "modules/credentialmanager/FormDataOptions.h"
 #include "modules/credentialmanager/PasswordCredentialData.h"
 #include "platform/credentialmanager/PlatformPasswordCredential.h"
 #include "platform/weborigin/SecurityOrigin.h"
 #include "public/platform/WebCredential.h"
 #include "public/platform/WebPasswordCredential.h"
 
 namespace blink {
 
 PasswordCredential* PasswordCredential::create(WebPasswordCredential* webPasswordCredential)
@@ skipping 12 matching lines @@
         return nullptr;
     }
 
     KURL iconURL = parseStringAsURL(data.iconURL(), exceptionState);
     if (exceptionState.hadException())
         return nullptr;
 
     return new PasswordCredential(data.id(), data.password(), data.name(), iconURL);
 }
 
+PasswordCredential* PasswordCredential::create(HTMLFormElement* form, ExceptionState& exceptionState)
+{
+    // Extract data from the form, then use the extracted |formData| object's
+    // value to populate |data|.
+    FormData* formData = FormData::create(form);
+    PasswordCredentialData data;
+
+    AtomicString idName;
+    AtomicString passwordName;
+    for (FormAssociatedElement* element : form->associatedElements()) {

[philipj_slow, 2016/03/29 05:45:54]

The spec wants the "submittable elements" which is a subset of the
"form-associated elements". Is it the early return below that squares the
difference? If so, documenting this would be nice.

[Mike West, 2016/03/31 08:31:41]

Done.

+        FileOrUSVString result;
+        formData->get(element->name(), result);
+        if (!result.isUSVString())
+            continue;
+
+        AtomicString autocomplete = toHTMLElement(element)->fastGetAttribute(HTMLNames::autocompleteAttr);
+        if (equalIgnoringCase(autocomplete, "photo"))
+            data.setIconURL(result.getAsUSVString());
+        if (equalIgnoringCase(autocomplete, "name") || equalIgnoringCase(autocomplete, "nickname"))

[philipj_slow, 2016/03/29 05:45:54]

Maybe more else all the way down? Unless there's a lot of inlining the compiler
probably can't tell that the cases are mutually exclusive.

And same order as in the spec?

[Mike West, 2016/03/31 08:31:41]

Done and done.

+            data.setName(result.getAsUSVString());
+        if (equalIgnoringCase(autocomplete, "current-password") || equalIgnoringCase(autocomplete, "new-password")) {
+            data.setPassword(result.getAsUSVString());
+            passwordName = element->name();
+        }
+        if (equalIgnoringCase(autocomplete, "username")) {
+            data.setId(result.getAsUSVString());
+            idName = element->name();
+        }
+    }
+
+    // Create a PasswordCredential using the data gathered above.
+    PasswordCredential* credential = PasswordCredential::create(data, exceptionState);
+    if (exceptionState.hadException())
+        return nullptr;
+    ASSERT(credential);
+
+    // After creating the Credential, populate its 'additionalData', 'idName', and 'passwordName' attributes.
+    FormDataOrURLSearchParams additionalData;
+    additionalData.setFormData(formData);
+    credential->setAdditionalData(additionalData);
+    credential->setIdName(idName);
+    credential->setPasswordName(passwordName);
+    return credential;
+}
+
 PasswordCredential::PasswordCredential(WebPasswordCredential* webPasswordCredential)
     : Credential(webPasswordCredential->getPlatformCredential())
     , m_idName("username")
     , m_passwordName("password")
 {
 }
 
 PasswordCredential::PasswordCredential(const String& id, const String& password, const String& name, const KURL& icon)
     : Credential(PlatformPasswordCredential::create(id, password, name, icon))
     , m_idName("username")
@@ skipping 48 matching lines @@
     return static_cast<PlatformPasswordCredential*>(m_platformCredential.get())->password();
 }
 
 DEFINE_TRACE(PasswordCredential)
 {
     Credential::trace(visitor);
     visitor->trace(m_additionalData);
 }
 
 } // namespace blink