[webkit-reviews] review granted: [Bug 81593] Merge nextRootInlineBox with nextLinePosition : [Attachment 141901] Deleted more code but considerably more complicated patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 16 14:03:30 PDT 2012


Enrica Casucci <enrica at apple.com> has granted Ryosuke Niwa <rniwa at webkit.org>'s
request for review:
Bug 81593: Merge nextRootInlineBox with nextLinePosition
https://bugs.webkit.org/show_bug.cgi?id=81593

Attachment 141901: Deleted more code but considerably more complicated patch
https://bugs.webkit.org/attachment.cgi?id=141901&action=review

------- Additional Comments from Enrica Casucci <enrica at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=141901&action=review


The change looks good.

> Source/WebCore/ChangeLog:9
> +	   to share the code. Moved out the nullity check of startBox and
extraction of the renderer's node from

should be: "and extracted the renderer's node..."


More information about the webkit-reviews mailing list