Reviewed by Ken.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Aug 2004 21:41:23 +0000 (21:41 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Aug 2004 21:41:23 +0000 (21:41 +0000)
- remove assertions that asserted text is non-empty, just avoid
doing anything for the empty case; this is needed because input
methods like to insert empty text in various cases.

        * khtml/editing/htmlediting_impl.cpp:
        (khtml::InsertTextCommandImpl::InsertTextCommandImpl):
        (khtml::InsertTextCommandImpl::doApply):
        (khtml::InsertTextCommandImpl::doUnapply):

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

WebCore/ChangeLog-2005-08-23
WebCore/khtml/editing/htmlediting_impl.cpp

index 90e2ab3..9351736 100644 (file)
@@ -1,3 +1,16 @@
+2004-08-03  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Ken.
+
+       - remove assertions that asserted text is non-empty, just avoid
+       doing anything for the empty case; this is needed because input
+       methods like to insert empty text in various cases.
+
+        * khtml/editing/htmlediting_impl.cpp:
+        (khtml::InsertTextCommandImpl::InsertTextCommandImpl):
+        (khtml::InsertTextCommandImpl::doApply):
+        (khtml::InsertTextCommandImpl::doUnapply):
+
 2004-08-03  Darin Adler  <darin@apple.com>
 
         Reviewed by Ken.
index 9107032..76faf63 100644 (file)
@@ -1846,7 +1846,6 @@ InsertTextCommandImpl::InsertTextCommandImpl(DocumentImpl *document, TextImpl *n
 {
     ASSERT(m_node);
     ASSERT(m_offset >= 0);
-    ASSERT(text.length() > 0);
     
     m_node->ref();
     m_text = text.copy(); // make a copy to ensure that the string never changes
@@ -1866,7 +1865,10 @@ int InsertTextCommandImpl::commandID() const
 void InsertTextCommandImpl::doApply()
 {
     ASSERT(m_node);
-    ASSERT(!m_text.isEmpty());
+
+    if (m_text.isEmpty())
+       return;
+
 
     int exceptionCode = 0;
     m_node->insertData(m_offset, m_text, exceptionCode);
@@ -1878,6 +1880,9 @@ void InsertTextCommandImpl::doUnapply()
     ASSERT(m_node);
     ASSERT(!m_text.isEmpty());
 
+    if (m_text.isEmpty())
+       return;
+
     int exceptionCode = 0;
     m_node->deleteData(m_offset, m_text.length(), exceptionCode);
     ASSERT(exceptionCode == 0);