2011-06-29 Ryosuke Niwa <rniwa@webkit.org>
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 29 Jun 2011 23:45:52 +0000 (23:45 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 29 Jun 2011 23:45:52 +0000 (23:45 +0000)
        Reviewed by Darin Adler.

        REGRESSION(r89952): r89952 added a wrong assertion in visible_units.cpp
        https://bugs.webkit.org/show_bug.cgi?id=63668

        We need to assert the offset to be zero only when the position is NOT offset in node.

        * editing/visible_units.cpp:
        (WebCore::startOfParagraph):

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

Source/WebCore/ChangeLog
Source/WebCore/editing/visible_units.cpp

index a58308f..321cf18 100644 (file)
@@ -1,3 +1,15 @@
+2011-06-29  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Reviewed by Darin Adler.
+
+        REGRESSION(r89952): r89952 added a wrong assertion in visible_units.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=63668
+
+        We need to assert the offset to be zero only when the position is NOT offset in node.
+
+        * editing/visible_units.cpp:
+        (WebCore::startOfParagraph):
+
 2011-06-29  Brent Fulgham  <bfulgham@webkit.org>
 
         Speculative WinCE build correction after r90057.
index 1520066..464e5f1 100644 (file)
@@ -818,7 +818,7 @@ VisiblePosition startOfParagraph(const VisiblePosition& c, EditingBoundaryCrossi
     }
 
     if (type == Position::PositionIsOffsetInAnchor) {
-        ASSERT(type != Position::PositionIsOffsetInAnchor || !offset);
+        ASSERT(type == Position::PositionIsOffsetInAnchor || !offset);
         return VisiblePosition(Position(node, offset, type), DOWNSTREAM);
     }