Reviewed by Maciej.
authorap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 24 Dec 2005 10:02:23 +0000 (10:02 +0000)
committerap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 24 Dec 2005 10:02:23 +0000 (10:02 +0000)
        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=6221
          REGRESSION: Password field is blank error for secure websites using
          password field (ToT, 12/24/05)

        * khtml/kwq/KWQSlot.mm:
        (KWQSlot::KWQSlot): Changed the parameter type of slotTextChanged to const DOMString &
        * khtml/rendering/render_form.h:
        * khtml/rendering/render_form.cpp:
        (RenderLineEdit::RenderLineEdit): ditto.
        (RenderLineEdit::slotTextChanged): ditto.
        (RenderFileButton::RenderFileButton): ditto.
        (RenderFileButton::slotTextChanged): ditto.

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

WebCore/ChangeLog
WebCore/khtml/rendering/render_form.cpp
WebCore/khtml/rendering/render_form.h
WebCore/kwq/KWQSlot.mm

index 74ab1409110d455d93c5799f5412b227143bbe3d..767944f642a7206d50d221953c94341bf5bd1b3c 100644 (file)
@@ -1,3 +1,20 @@
+2005-12-24  Mitz Pettel  <opendarwin.org@mitzpettel.com>
+
+        Reviewed by Maciej.
+
+        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=6221
+          REGRESSION: Password field is blank error for secure websites using
+          password field (ToT, 12/24/05)
+
+        * khtml/kwq/KWQSlot.mm:
+        (KWQSlot::KWQSlot): Changed the parameter type of slotTextChanged to const DOMString &
+        * khtml/rendering/render_form.h:
+        * khtml/rendering/render_form.cpp:
+        (RenderLineEdit::RenderLineEdit): ditto.
+        (RenderLineEdit::slotTextChanged): ditto.
+        (RenderFileButton::RenderFileButton): ditto.
+        (RenderFileButton::slotTextChanged): ditto.
+
 2005-12-22  Geoffrey Garen  <ggaren@apple.com>
     
         Reviewed by vicki (yay!), eric.
index 11d42dc9b901690027be56c1844ef35da71a59c5..c2fa27c77e1552dff305b32ee81a4a1d78402711 100644 (file)
@@ -202,7 +202,7 @@ RenderLineEdit::RenderLineEdit(HTMLInputElementImpl *element)
         edit->setLiveSearch(false);
     connect(edit,SIGNAL(returnPressed()), this, SLOT(slotReturnPressed()));
     connect(edit, SIGNAL(selectionChanged()), this, SLOT(slotSelectionChanged()));
-    connect(edit,SIGNAL(textChanged(const QString &)),this,SLOT(slotTextChanged(const QString &)));
+    connect(edit,SIGNAL(textChanged(const QString &)),this,SLOT(slotTextChanged(const DOMString &)));
     connect(edit,SIGNAL(clicked()),this,SLOT(slotClicked()));
 
     connect(edit,SIGNAL(performSearch()), this, SLOT(slotPerformSearch()));
@@ -324,7 +324,7 @@ void RenderLineEdit::updateFromElement()
     RenderFormElement::updateFromElement();
 }
 
-void RenderLineEdit::slotTextChanged(const QString &string)
+void RenderLineEdit::slotTextChanged(const DOMString &string)
 {
     if (m_updating) // Don't alter the value if we are in the middle of initing the control, since
         return;     // we are getting the value from the DOM and it's not user input.
@@ -332,7 +332,7 @@ void RenderLineEdit::slotTextChanged(const QString &string)
     // A null string value is used to indicate that the form control has not altered the original
     // default value.  That means that we should never use the null string value when the user
     // empties a textfield, but should always force an empty textfield to use the empty string.
-    QString newText = string.isNull() ? "" : string;
+    QString newText = string.isNull() ? "" : string.qstring();
     newText.replace(backslashAsCurrencySymbol(), QChar('\\'));
     element()->setValueFromRenderer(newText);
 }
@@ -532,7 +532,7 @@ RenderFileButton::RenderFileButton(HTMLInputElementImpl *element)
     : RenderFormElement(element)
 {
     KWQFileButton *w = new KWQFileButton(view()->part());
-    connect(w, SIGNAL(textChanged(const QString &)),this,SLOT(slotTextChanged(const QString &)));
+    connect(w, SIGNAL(textChanged(const QString &)),this,SLOT(slotTextChanged(const DOMString &)));
     connect(w, SIGNAL(clicked()), this, SLOT(slotClicked()));
     setQWidget(w);
 }
@@ -570,9 +570,9 @@ void RenderFileButton::slotReturnPressed()
        element()->form()->prepareSubmit();
 }
 
-void RenderFileButton::slotTextChanged(const QString &string)
+void RenderFileButton::slotTextChanged(const DOMString &string)
 {
-    element()->m_value = DOMString(string);
+    element()->m_value = string;
     element()->onChange();
 }
 
index 9ff028cc3b7d44efcc0ba764546d315f33f13c10..dd74a417723656bf5915a3edfef10c545789ed32 100644 (file)
@@ -150,7 +150,7 @@ public:
 
 public slots:
     void slotReturnPressed();
-    void slotTextChanged(const QString &string);
+    void slotTextChanged(const DOM::DOMString &string);
     void slotSelectionChanged();
     void slotPerformSearch();
 public:
@@ -219,7 +219,7 @@ public:
 public slots:
     virtual void slotClicked();
     virtual void slotReturnPressed();
-    virtual void slotTextChanged(const QString &string);
+    virtual void slotTextChanged(const DOM::DOMString &string);
 
 protected:
     virtual bool isEditable() const { return true; }
index cce78df1493e0f37e4d2541bd279643862fefae6..4667b5a11ee362ed295c02d60e01bd6097b7f272 100644 (file)
@@ -134,7 +134,7 @@ KWQSlot::KWQSlot(QObject *object, const char *member)
         m_function = slotParentDestroyed;
     } else if (KWQNamesMatch(member, SLOT(submitFormAgain()))) {
         m_function = slotSubmitFormAgain;
-    } else if (KWQNamesMatch(member, SLOT(slotTextChanged(const QString &)))) {
+    } else if (KWQNamesMatch(member, SLOT(slotTextChanged(const DOMString &)))) {
         m_function = slotTextChangedWithString;
     } else if (KWQNamesMatch(member, SLOT(slotData(KIO::Job *, const char *, int)))) {
        if (object->isKHTMLLoader()) {