Reviewed by Vicki.
authorharrison <harrison@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 2 May 2005 17:11:26 +0000 (17:11 +0000)
committerharrison <harrison@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 2 May 2005 17:11:26 +0000 (17:11 +0000)
        Fix isStartOfEditableContent and isEndOfEditableContent to return actual, rather than inverted, answers.
        No Radar.  Found this when trying to use isEndOfEditableContent() in some new code.

        * khtml/editing/visible_units.cpp:
        (khtml::isStartOfEditableContent):
        (khtml::isEndOfEditableContent):
        Flip expressions.

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

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

index ce959597c72889f6767a04f4f0321f4e35f0575e..80c8159df874e2e9956ac9d901ea8408a0c229bd 100644 (file)
@@ -1,3 +1,15 @@
+2005-05-02  David Harrison  <harrison@apple.com>
+
+        Reviewed by Vicki.
+
+        Fix isStartOfEditableContent and isEndOfEditableContent to return actual, rather than inverted, answers.
+        No Radar.  Found this when trying to use isEndOfEditableContent() in some new code.
+        
+        * khtml/editing/visible_units.cpp:
+        (khtml::isStartOfEditableContent):
+        (khtml::isEndOfEditableContent):
+        Flip expressions.
+
 2005-04-29  Darin Adler  <darin@apple.com>
 
         Reviewed by David Harrison.
 2005-04-29  Darin Adler  <darin@apple.com>
 
         Reviewed by David Harrison.
index 7bb9e23d1b075906dc789bd0cb2425589c6f623b..04d84efb26dcdab4f9176aa02bb6b2afa2ab08ed 100644 (file)
@@ -864,12 +864,12 @@ bool inSameEditableContent(const VisiblePosition &a, const VisiblePosition &b)
 
 bool isStartOfEditableContent(const VisiblePosition &p)
 {
 
 bool isStartOfEditableContent(const VisiblePosition &p)
 {
-    return inSameEditableContent(p, p.previous());
+    return !inSameEditableContent(p, p.previous());
 }
 
 bool isEndOfEditableContent(const VisiblePosition &p)
 {
 }
 
 bool isEndOfEditableContent(const VisiblePosition &p)
 {
-    return inSameEditableContent(p, p.next());
+    return !inSameEditableContent(p, p.next());
 }
 
 } // namespace khtml
 }
 
 } // namespace khtml