Title: [99492] trunk/Source/WebCore
- Revision
- 99492
- Author
- [email protected]
- Date
- 2011-11-07 15:43:21 -0800 (Mon, 07 Nov 2011)
Log Message
Remove unnecessary use of function pointer in FrameSelection::modify()
https://bugs.webkit.org/show_bug.cgi?id=71646
Reviewed by Ryosuke Niwa.
* editing/FrameSelection.cpp:
(WebCore::FrameSelection::modify):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (99491 => 99492)
--- trunk/Source/WebCore/ChangeLog 2011-11-07 23:39:29 UTC (rev 99491)
+++ trunk/Source/WebCore/ChangeLog 2011-11-07 23:43:21 UTC (rev 99492)
@@ -1,3 +1,13 @@
+2011-11-07 Alice Boxhall <[email protected]>
+
+ Remove unnecessary use of function pointer in FrameSelection::modify()
+ https://bugs.webkit.org/show_bug.cgi?id=71646
+
+ Reviewed by Ryosuke Niwa.
+
+ * editing/FrameSelection.cpp:
+ (WebCore::FrameSelection::modify):
+
2011-11-07 Andreas Kling <[email protected]>
Unreviewed Windows build fix after r99468.
Modified: trunk/Source/WebCore/editing/FrameSelection.cpp (99491 => 99492)
--- trunk/Source/WebCore/editing/FrameSelection.cpp 2011-11-07 23:39:29 UTC (rev 99491)
+++ trunk/Source/WebCore/editing/FrameSelection.cpp 2011-11-07 23:43:21 UTC (rev 99492)
@@ -1024,7 +1024,11 @@
VisiblePosition result;
VisiblePosition next;
for (VisiblePosition p = pos; ; p = next) {
- next = (direction == DirectionUp ? previousLinePosition : nextLinePosition)(p, xPos);
+ if (direction == DirectionUp)
+ next = previousLinePosition(p, xPos);
+ else
+ next = nextLinePosition(p, xPos);
+
if (next.isNull() || next == p)
break;
LayoutUnit nextY;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
