Reviewed by Dave.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 30 Aug 2004 21:50:53 +0000 (21:50 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 30 Aug 2004 21:50:53 +0000 (21:50 +0000)
        - fixed <rdar://problem/3303968> final newline character omitted from <textarea> when parsing a new page

        * khtml/html/htmltokenizer.cpp: (khtml::HTMLTokenizer::processListing): Handle newline pending
        cases as well as space and tab pending spaces at the end of this function. This seems right for
        both scripts and text areas; it's not clear why omittiing the LF is ever needed. Tested by running
        all the layout tests; no regressions.

        - fixed <rdar://problem/3552736> word wrapped text fields that blur/focus when you type move the insertion point strangely (vtext.com)

        * khtml/rendering/render_form.cpp: (RenderTextArea::updateFromElement): Call text() instead of
        calling widget->text() directly. This prevents this function from running in cases where the
        text didn't really change for text areas in wrap mode.

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/html/htmltokenizer.cpp
WebCore/khtml/rendering/render_form.cpp

index 019d80fed291d4f89a73584d659b1ea1299179f5..5edd46b3b56c6050a99df8c4f2aa85ab4d78f29e 100644 (file)
@@ -1,3 +1,20 @@
+2004-08-30  Darin Adler  <darin@apple.com>
+
+        Reviewed by Dave.
+
+        - fixed <rdar://problem/3303968> final newline character omitted from <textarea> when parsing a new page
+
+        * khtml/html/htmltokenizer.cpp: (khtml::HTMLTokenizer::processListing): Handle newline pending
+        cases as well as space and tab pending spaces at the end of this function. This seems right for
+        both scripts and text areas; it's not clear why omittiing the LF is ever needed. Tested by running
+        all the layout tests; no regressions.
+
+        - fixed <rdar://problem/3552736> word wrapped text fields that blur/focus when you type move the insertion point strangely (vtext.com)
+
+        * khtml/rendering/render_form.cpp: (RenderTextArea::updateFromElement): Call text() instead of
+        calling widget->text() directly. This prevents this function from running in cases where the
+        text didn't really change for text areas in wrap mode.
+
 2004-08-30  Darin Adler  <darin@apple.com>
 
         Reviewed by Dave.
index dffbf38474c8066a9333702f3b679cc71bb74666..f3ef9c30baf46f26ec38c7313f84feb9a4640628 100644 (file)
@@ -392,9 +392,8 @@ void HTMLTokenizer::processListing(TokenizerString list)
 
     }
 
-    if ((pending == SpacePending) || (pending == TabPending))
+    if (pending)
         addPending();
-    pending = NonePending;
 
     prePos = 0;
 
index 35c2aac1e43951cd1c1d695ba1fd84eb677e0a5a..efd0780fb9c4dc08ed8e4aeafd185a6eb3c4febe 100644 (file)
@@ -1557,10 +1557,10 @@ void RenderTextArea::updateFromElement()
     w->setDisabled(element()->disabled());
 #endif
     
-    // Call w->text() before calling element()->value(), because in the case of inline
+    // Call text() before calling element()->value(), because in the case of inline
     // input such as Hiragana, w->text() has a side effect of sending the notification
     // that we use in slotTextChanged to update element()->m_value
-    QString widgetText = w->text();
+    QString widgetText = text();
     QString text = element()->value().string();
     text.replace('\\', backslashAsCurrencySymbol());
     if (widgetText != text) {