Title: [97479] trunk/Source/WebCore
- Revision
- 97479
- Author
- [email protected]
- Date
- 2011-10-14 10:57:52 -0700 (Fri, 14 Oct 2011)
Log Message
Redundant comparison in AccessibilityObject.cpp updateAXLineStartForVisiblePosition
https://bugs.webkit.org/show_bug.cgi?id=70074
Reviewed by Chris Fleizach.
Removed redundant code.
* accessibility/AccessibilityObject.cpp:
(WebCore::updateAXLineStartForVisiblePosition):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (97478 => 97479)
--- trunk/Source/WebCore/ChangeLog 2011-10-14 17:54:42 UTC (rev 97478)
+++ trunk/Source/WebCore/ChangeLog 2011-10-14 17:57:52 UTC (rev 97479)
@@ -1,3 +1,15 @@
+2011-10-14 Ryosuke Niwa <[email protected]>
+
+ Redundant comparison in AccessibilityObject.cpp updateAXLineStartForVisiblePosition
+ https://bugs.webkit.org/show_bug.cgi?id=70074
+
+ Reviewed by Chris Fleizach.
+
+ Removed redundant code.
+
+ * accessibility/AccessibilityObject.cpp:
+ (WebCore::updateAXLineStartForVisiblePosition):
+
2011-10-12 Ryosuke Niwa <[email protected]>
Make the interface of locationAndLengthFromRange and rangeFromLocationAndLength consistent
Modified: trunk/Source/WebCore/accessibility/AccessibilityObject.cpp (97478 => 97479)
--- trunk/Source/WebCore/accessibility/AccessibilityObject.cpp 2011-10-14 17:54:42 UTC (rev 97478)
+++ trunk/Source/WebCore/accessibility/AccessibilityObject.cpp 2011-10-14 17:57:52 UTC (rev 97479)
@@ -486,7 +486,7 @@
VisiblePosition startPosition = visiblePosition;
while (true) {
tempPosition = startPosition.previous();
- if (tempPosition.isNull() || tempPosition.isNull())
+ if (tempPosition.isNull())
break;
Position p = tempPosition.deepEquivalent();
RenderObject* renderer = p.deprecatedNode()->renderer();
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
