[webkit-reviews] review requested: [Bug 16844] RenderText::addLineBoxRects erroneously includes last char for boundingBox : [Attachment 18397] Patch taking Peter's comments into account

[bugzilla-daemon at webkit.org]

Finnur Thorarinsson <finnur.webkit at gmail.com> has asked  for review:
Bug 16844: RenderText::addLineBoxRects erroneously includes last char for
boundingBox
http://bugs.webkit.org/show_bug.cgi?id=16844

Attachment 18397: Patch taking Peter's comments into account
http://bugs.webkit.org/attachment.cgi?id=18397&action=edit

------- Additional Comments from Finnur Thorarinsson <finnur.webkit at gmail.com>
I changed:

box->end() + 1 <= end	    to
box->end() < end

... and moved the comment in the else clause above the if (since it applies to
both the if clause and the else clause now).