Changeset 89056 in webkit
- Timestamp:
- Jun 16, 2011 12:11:32 PM (13 years ago)
- Location:
- trunk
- Files:
-
- 2 added
- 4 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r89055 r89056 1 2011-06-16 Ryosuke Niwa <rniwa@webkit.org> 2 3 Reviewed by Eric Seidel. 4 5 Consider padding and border when looking for the next/previous line position 6 https://bugs.webkit.org/show_bug.cgi?id=55481 7 8 Added a test to ensure WebKit can allow vertical caret movements even when 9 inline elements that span multiple lines have paddings, borders, or both. 10 11 * editing/selection/move-vertically-with-paddings-borders-expected.txt: Added. 12 * editing/selection/move-vertically-with-paddings-borders.html: Added. 13 1 14 2011-06-16 Keunsoon Lee <keunsoon.lee@samsung.com> 2 15 -
trunk/Source/WebCore/ChangeLog
r89055 r89056 1 2011-06-16 Ryosuke Niwa <rniwa@webkit.org> 2 3 Reviewed by Eric Seidel. 4 5 Consider padding and border when looking for the next/previous line position 6 https://bugs.webkit.org/show_bug.cgi?id=55481 7 8 The bug was caused by previousLinePosition and nextLinePosition passing y coordinate 9 above the line in some cases. Fixed the bug by passing the larger of selectionTop and logicalTop. 10 11 This patch is based on a patch originally written by Mario Sanchez Prada <msanchez@igalia.com>. 12 13 Test: editing/selection/move-vertically-with-paddings-borders.html 14 15 * editing/visible_units.cpp: 16 (WebCore::previousLinePosition): 17 (WebCore::nextLinePosition): 18 * rendering/RootInlineBox.h: 19 (WebCore::RootInlineBox::blockDirectionPointInLine): 20 1 21 2011-06-16 Keunsoon Lee <keunsoon.lee@samsung.com> 2 22 -
trunk/Source/WebCore/editing/visible_units.cpp
r88359 r89056 574 574 if (node && editingIgnoresContent(node)) 575 575 return positionInParentBeforeNode(node); 576 return renderer->positionForPoint(IntPoint(x - absPos.x(), root-> lineTop()));576 return renderer->positionForPoint(IntPoint(x - absPos.x(), root->blockDirectionPointInLine())); 577 577 } 578 578 … … 681 681 if (node && editingIgnoresContent(node)) 682 682 return positionInParentBeforeNode(node); 683 return renderer->positionForPoint(IntPoint(x - absPos.x(), root-> lineTop()));683 return renderer->positionForPoint(IntPoint(x - absPos.x(), root->blockDirectionPointInLine())); 684 684 } 685 685 -
trunk/Source/WebCore/rendering/RootInlineBox.h
r88319 r89056 57 57 int selectionBottom() const; 58 58 int selectionHeight() const { return max(0, selectionBottom() - selectionTop()); } 59 60 int blockDirectionPointInLine() const { return max(lineTop(), selectionTop()); } 59 61 60 62 int alignBoxesInBlockDirection(int heightOfBlock, GlyphOverflowAndFallbackFontsMap&, VerticalPositionCache&);
Note: See TracChangeset
for help on using the changeset viewer.