New FormData methods: get, getAll, has, set, delete and iterable

Source/core/html/DOMFormData.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "core/html/DOMFormData.h"
 
+#include "bindings/core/v8/V8Binding.h"
+#include "bindings/core/v8/V8File.h"
+#include "core/dom/Iterator.h"

[Jens Widell, 2015/01/15 07:35:06]

You don't need to include this now, I think?

[jsbell, 2015/01/15 18:57:13]

Done.

 #include "core/fileapi/Blob.h"
+#include "core/fileapi/File.h"
 #include "core/html/HTMLFormElement.h"
 #include "wtf/text/TextEncoding.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
+namespace {
+
+ScriptValue entryValueToScriptValue(ScriptState* scriptState, const FormDataList::Entry& entry)
+{
+    if (entry.isString())
+        return ScriptValue(scriptState, v8String(scriptState->isolate(), entry.string()));
+    if (entry.isFile())
+        return ScriptValue(scriptState, toV8(entry.file(), scriptState->context()->Global(), scriptState->isolate()));
+    ASSERT(entry.isNone());
+    return ScriptValue(scriptState, v8::Null(scriptState->isolate()));
+}
+
+class DOMFormDataIterationSource final : public PairIterable<String, ScriptValue>::IterationSource {
+public:
+    DOMFormDataIterationSource(DOMFormData* formData) : m_formData(formData), m_current(0) { }
+
+    virtual bool next(ScriptState* scriptState, String& key, ScriptValue& value, ExceptionState& exceptionState) override
+    {
+        if (m_current >= m_formData->size())
+            return false;
+
+        const FormDataList::Entry entry = m_formData->getEntry(m_current++);
+        ASSERT(!entry.isNone());
+        key = entry.name();
+        value = entryValueToScriptValue(scriptState, entry);
+        return true;
+    }
+
+    virtual void trace(Visitor* visitor)
+    {
+        PairIterable<String, ScriptValue>::IterationSource::trace(visitor);

[Jens Widell, 2015/01/15 07:35:06]

The convention is to make this a tail call, said haraken@ in
https://codereview.chromium.org/848673002.

[jsbell, 2015/01/15 18:57:13]

Done.

+        visitor->trace(m_formData);
+    }
+
+private:
+    const RefPtrWillBeMember<DOMFormData> m_formData;
+    size_t m_current;
+};
+
+} // namespace
+
 DOMFormData::DOMFormData(const WTF::TextEncoding& encoding)
     : FormDataList(encoding)
 {
 }
 
 DOMFormData::DOMFormData(HTMLFormElement* form)
     : FormDataList(UTF8Encoding())
 {
     if (!form)
         return;
 
     for (unsigned i = 0; i < form->associatedElements().size(); ++i) {
         FormAssociatedElement* element = form->associatedElements()[i];
         if (!toHTMLElement(element)->isDisabledFormControl())
             element->appendFormData(*this, true);
     }
 }
 
 void DOMFormData::append(const String& name, const String& value)
 {
     appendData(name, value);
 }
 
 void DOMFormData::append(const String& name, Blob* blob, const String& filename)
 {
     appendBlob(name, blob, filename);
 }
 
+ScriptValue DOMFormData::get(ScriptState* scriptState, const String& name)
+{
+    return entryValueToScriptValue(scriptState, getEntry(name));
+}
+
+Vector<ScriptValue> DOMFormData::getAll(ScriptState* scriptState, const String& name)
+{
+    Vector<ScriptValue> results;
+    WillBeHeapVector<FormDataList::Entry> entries = FormDataList::getAll(name);
+    for (size_t i = 0; i < entries.size(); ++i) {
+        const FormDataList::Entry& entry = entries[i];
+        ASSERT(!entry.isNone());
+        ASSERT(entry.name() == name);
+        results.append(entryValueToScriptValue(scriptState, entry));
+    }
+    ASSERT(results.size() == entries.size());
+    return results;
+}
+
+void DOMFormData::set(const String& name, const String& value)
+{
+    setData(name, value);
+}
+
+void DOMFormData::set(const String& name, Blob* blob, const String& filename)
+{
+    setBlob(name, blob, filename);
+}
+
+PairIterable<String, ScriptValue>::IterationSource* DOMFormData::startIteration(ScriptState*, ExceptionState&)
+{
+    return new DOMFormDataIterationSource(this);
+}
+
 } // namespace blink