Changeset 111613 in webkit
- Timestamp:
- Mar 21, 2012 3:42:25 PM (12 years ago)
- Location:
- trunk/Source/WebCore
- Files:
-
- 2 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebCore/ChangeLog
r111611 r111613 1 2012-03-21 Levi Weintraub <leviw@chromium.org> 2 3 Correct LayoutUnit usage in VisiblePosition.cpp 4 https://bugs.webkit.org/show_bug.cgi?id=81775 5 6 Reviewed by Eric Seidel. 7 8 Correcting a mismatch between the .h and .cpp for the definition of localCaretRect, 9 and properly using LayoutUnits for local coordinates. 10 11 No new tests. No change in behavior. 12 13 * editing/VisiblePosition.cpp: 14 (WebCore::VisiblePosition::localCaretRect): Switching to return a LayoutRect, as in the 15 header. Local coordinates are in LayoutUnits. 16 (WebCore::VisiblePosition::absoluteCaretBounds): Using LayoutRect for the localCaretRect. 17 (WebCore::VisiblePosition::lineDirectionPointForBlockDirectionNavigation): Ditto. 18 1 19 2012-03-21 Adam Klein <adamk@chromium.org> 2 20 -
trunk/Source/WebCore/editing/VisiblePosition.cpp
r97597 r111613 596 596 } 597 597 598 IntRect VisiblePosition::localCaretRect(RenderObject*& renderer) const598 LayoutRect VisiblePosition::localCaretRect(RenderObject*& renderer) const 599 599 { 600 600 if (m_deepPosition.isNull()) { … … 606 606 renderer = node->renderer(); 607 607 if (!renderer) 608 return IntRect();608 return LayoutRect(); 609 609 610 610 InlineBox* inlineBox; … … 621 621 { 622 622 RenderObject* renderer; 623 IntRect localRect = localCaretRect(renderer);623 LayoutRect localRect = localCaretRect(renderer); 624 624 if (localRect.isEmpty() || !renderer) 625 625 return IntRect(); … … 631 631 { 632 632 RenderObject* renderer; 633 IntRect localRect = localCaretRect(renderer);633 LayoutRect localRect = localCaretRect(renderer); 634 634 if (localRect.isEmpty() || !renderer) 635 635 return 0;
Note: See TracChangeset
for help on using the changeset viewer.