Reviewed by John.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 6 Sep 2004 23:23:15 +0000 (23:23 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 6 Sep 2004 23:23:15 +0000 (23:23 +0000)
        - did some clean-up on form data code

        * khtml/misc/formdata.h: Added more public functions as suggested by Chris when he reviewed
        this class.
        * khtml/misc/formdata.cpp: (khtml::FormData::appendFile): Added. Will be needed soon.
        * kwq/KWQFormData.mm: (arrayFromFormData): Use new cleaned-up FormData API.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@7462 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2005-08-23
WebCore/khtml/misc/formdata.cpp
WebCore/khtml/misc/formdata.h
WebCore/kwq/KWQFormData.mm

index eb2935bfa82796b1494ccb9db4c051259bbd9e34..a1a83d7ec64bc1e2ce99d5e3f60a0520c420b2d3 100644 (file)
@@ -1,3 +1,14 @@
+2004-09-06  Darin Adler  <darin@apple.com>
+
+        Reviewed by John.
+
+        - did some clean-up on form data code
+
+        * khtml/misc/formdata.h: Added more public functions as suggested by Chris when he reviewed
+        this class.
+        * khtml/misc/formdata.cpp: (khtml::FormData::appendFile): Added. Will be needed soon.
+        * kwq/KWQFormData.mm: (arrayFromFormData): Use new cleaned-up FormData API.
+
 2004-09-06  Darin Adler  <darin@apple.com>
 
         Reviewed by John.
index 867eded28f07bce388f5593cba8e9851bc7c3d22..e44a2a777894eddea2e96cc1587fb78c49ef72a9 100644 (file)
@@ -42,6 +42,11 @@ void FormData::appendData(const void *data, size_t size)
     memcpy(e.m_data.data() + oldSize, data, size);
 }
 
+void FormData::appendFile(const QString &filename)
+{
+    m_elements.append(filename);
+}
+
 QByteArray FormData::flatten() const
 {
     // Concatenate all the byte arrays, but omit any files.
index 87dabe9614d58b40aa1243085ff9687ff3e7a395..dd753f622d50a429c5ff3cdf14e40dc2c28c5fe3 100644 (file)
@@ -31,6 +31,8 @@ namespace khtml {
 class FormDataElement {
 public:
     FormDataElement() : m_type(data) { }
+    FormDataElement(const QByteArray &array) : m_type(data), m_data(array) { }
+    FormDataElement(const QString &filename) : m_type(encodedFile), m_filename(filename) { }
 
     enum { data, encodedFile } m_type;
     QByteArray m_data;
@@ -43,10 +45,16 @@ public:
     FormData(const QCString &);
 
     void appendData(const void *data, size_t size);
+    void appendFile(const QString &filename);
 
     QByteArray flatten() const; // omits files
     QString flattenToString() const; // omits files
 
+    size_t count() const { return m_elements.count(); }
+    QValueListConstIterator<FormDataElement> begin() const { return m_elements.begin(); }
+    QValueListConstIterator<FormDataElement> end() const { return m_elements.end(); }
+
+private:
     QValueList<FormDataElement> m_elements;
 };
 
index b662fddda40c0841bd005dd0c18acd525325c3e2..c32df35025e59dbce96aef3f557efa7dad962c4e 100644 (file)
@@ -33,8 +33,8 @@ using khtml::FormDataElement;
 
 NSArray *arrayFromFormData(const FormData &d)
 {
-    NSMutableArray *a = [NSMutableArray arrayWithCapacity:d.m_elements.count()];
-    for (QValueListConstIterator<FormDataElement> it = d.m_elements.begin(); it != d.m_elements.end(); ++it) {
+    NSMutableArray *a = [NSMutableArray arrayWithCapacity:d.count()];
+    for (QValueListConstIterator<FormDataElement> it = d.begin(); it != d.end(); ++it) {
         const FormDataElement &e = *it;
         if (e.m_type == FormDataElement::data) {
             [a addObject:[NSData dataWithBytes:e.m_data.data() length:e.m_data.size()]];