[Webkit-unassigned] [Bug 108511] [Chromium] Spelling should ignore punctuation
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Feb 1 12:04:53 PST 2013
https://bugs.webkit.org/show_bug.cgi?id=108511
--- Comment #2 from Tony Chang <tony at chromium.org> 2013-02-01 12:06:55 PST ---
(From update of attachment 186092)
View in context: https://bugs.webkit.org/attachment.cgi?id=186092&action=review
> Source/WebKit/chromium/src/ContextMenuClientImpl.cpp:147
> +// Returns true for whitespace and punctuation characters. Returns false for every other type of character.
Remove this comment.
> Source/WebKit/chromium/src/ContextMenuClientImpl.cpp:150
> + return WTF::Unicode::isSeparatorSpace(c) || WTF::Unicode::isPunct(c);
The old code did WTF::Unicode::direction(c) == WTF::Unicode::WhiteSpaceNeutral instead of WTF::Unicode::isSeparatorSpace(c). Is this difference intentional? Maybe we should #include <wtf/text/StringImpl.h> and use isSpaceOrNewline(c) || WTF::Unicode::isPunct(c) to be more like the old code?
--
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