[Webkit-unassigned] [Bug 108881] Canvas fillText and measureText handle ideographic spaces differently

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Aug 12 00:17:51 PDT 2013


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





--- Comment #25 from Rashmi Shyamasundar <rashmi.s2 at samsung.com>  2013-08-12 00:17:25 PST ---
(In reply to comment #24)
> (From update of attachment 208423 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=208423&action=review
> 
> > Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp:98
> > +// Spec : http://www.whatwg.org/specs/web-apps/current-work/multipage/common-microsyntaxes.html#space-character
> > +static inline bool isHTMLSpace(UChar c)
> > +{
> > +    return (c == 0x0009 || c == 0x000A || c == 0x000B || c == 0x000C || c == 0x000D);
> > +}
> 
> This function already exists in HTMLParserIdioms.h and you should use the version from there instead of adding a new one.

The function isHTMLSpace() defined in HTMLParserIdioms.h does not behave exactly as mentioned in Spec : http://www.whatwg.org/specs/web-apps/current-work/multipage/common-microsyntaxes.html#space-character . 

It does not return true for the below characters :-
1. line feed - U+000A
2. line tabulation - U+000B (Please check the discussion at https://bugs.webkit.org/show_bug.cgi?id=119567)  

It returns true for  '\n' ( U+2424) which is not required.

-- 
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