[Webkit-unassigned] [Bug 52926] Add segmentation test for range.expand

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jan 24 14:20:41 PST 2011


https://bugs.webkit.org/show_bug.cgi?id=52926


Ryosuke Niwa <rniwa at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #79976|review?                     |review+
               Flag|                            |




--- Comment #10 from Ryosuke Niwa <rniwa at webkit.org>  2011-01-24 14:20:41 PST ---
(From update of attachment 79976)
View in context: https://bugs.webkit.org/attachment.cgi?id=79976&action=review

> LayoutTests/fast/text/international/cjk-segmentation.html:5
> +<title>Test for CJK segmentation.</title>

Nit: Don't put period in the title.

> LayoutTests/fast/text/international/cjk-segmentation.html:50
> +    if (pass == true) {
> +        log("ALL PASS");
> +    }

Nit: no { } around a single line statement.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list