[webkit-reviews] review granted: [Bug 135631] Cleanup InlineTextBox::paintSelection and ::localSelectionRect. : [Attachment 236074] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Aug 5 20:45:23 PDT 2014

Darin Adler <darin at apple.com> has granted Zalan Bujtas <zalan at apple.com>'s
request for review:
Bug 135631: Cleanup InlineTextBox::paintSelection and ::localSelectionRect.

Attachment 236074: Patch

------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=236074&action=review

> Source/WebCore/ChangeLog:8
> +	   No change in functionality.

But we removed the snapping from localSelectionRect. That seems like a bug fix,
not a “no change” refactoring fix.

> Source/WebCore/rendering/InlineTextBox.cpp:740
> +    LayoutUnit selectionTop =

No need to put this into local variable any more if we are only using this

More information about the webkit-reviews mailing list