[webkit-reviews] review canceled: [Bug 29292] [HTML5][Forms] Support for <textarea maxlength=N> : [Attachment 39781] Proposed patch (rev.4)

[bugzilla-daemon at webkit.org]

TAMURA, Kent <tkent at chromium.org> has canceled TAMURA, Kent
<tkent at chromium.org>'s request for review:
Bug 29292: [HTML5][Forms] Support for <textarea maxlength=N>
https://bugs.webkit.org/show_bug.cgi?id=29292

Attachment 39781: Proposed patch (rev.4)
https://bugs.webkit.org/attachment.cgi?id=39781&action=review

------- Additional Comments from TAMURA, Kent <tkent at chromium.org>
> Suppose that the string is "ab". numCharactersInGraphemeClusters(1) should
> return 1, not 2. If we specify length(), the result would be 2.

ok, I changed it to:

unsigned String::numCharactersInGraphemeClusters(unsigned numGraphemeClusters)
const
{
    TextBreakIterator* it = characterBreakIterator(characters(), length());
    if (!it)
	return min(length(), numGraphemeClusters);