Title: [90064] trunk/Source/WebCore
Revision
90064
Author
[email protected]
Date
2011-06-29 16:45:52 -0700 (Wed, 29 Jun 2011)

Log Message

2011-06-29  Ryosuke Niwa  <[email protected]>

        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):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (90063 => 90064)


--- trunk/Source/WebCore/ChangeLog	2011-06-29 23:45:28 UTC (rev 90063)
+++ trunk/Source/WebCore/ChangeLog	2011-06-29 23:45:52 UTC (rev 90064)
@@ -1,3 +1,15 @@
+2011-06-29  Ryosuke Niwa  <[email protected]>
+
+        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  <[email protected]>
 
         Speculative WinCE build correction after r90057.

Modified: trunk/Source/WebCore/editing/visible_units.cpp (90063 => 90064)


--- trunk/Source/WebCore/editing/visible_units.cpp	2011-06-29 23:45:28 UTC (rev 90063)
+++ trunk/Source/WebCore/editing/visible_units.cpp	2011-06-29 23:45:52 UTC (rev 90064)
@@ -818,7 +818,7 @@
     }
 
     if (type == Position::PositionIsOffsetInAnchor) {
-        ASSERT(type != Position::PositionIsOffsetInAnchor || !offset);
+        ASSERT(type == Position::PositionIsOffsetInAnchor || !offset);
         return VisiblePosition(Position(node, offset, type), DOWNSTREAM);
     }
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to