RESOLVED FIXED 69656
[Skia on Chromium Mac] Set canExpandAroundIdeographsInComplexText to true 
https://bugs.webkit.org/show_bug.cgi?id=69656
Summary [Skia on Chromium Mac] Set canExpandAroundIdeographsInComplexText to true
Cary Clark
Reported 2011-10-07 12:33:20 PDT
[Skia on Chromium Mac] Set canExpandAroundIdeographsInComplexText to true
Attachments
Patch (1.75 KB, patch)
2011-10-07 12:39 PDT, Cary Clark 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Cary Clark
Comment 1 2011-10-07 12:39:32 PDT
Created attachment 110197 [details] Patch
Stephen White
Comment 2 2011-10-07 12:46:25 PDT
Comment on attachment 110197 [details] Patch Looks good (up to my limited knowledge). Looping in Mike in case he has any comments. As a side note, I see justify-ideograph-complex is failing on all (chrome) platforms currently. Does it simply require rebaselining, or are there further problems?
Mike Reed
Comment 3 2011-10-07 12:58:44 PDT
Hmmm, seems fine, but confusing. The result of this function (which is only called once) seems to only mean "can I call isCJKIdeographOrSymbol() or not". But that guy is purely a unicode lookup function, with no platform specificisms afaik. If it makes more pages look better, lets turn it on and see. From a code-review p.o.v. I can't see why this isn't on for all of our platforms (but time will tell).
Cary Clark
Comment 4 2011-10-07 13:17:40 PDT
justify-ideograph-complex should have a fully justified right side. Since canExpandAroundIdeographsInComplexText has never been enabled for Windows and Linux, this test has never succeeded on those platforms. Oddly, there is a non-justified (i.e. wrong) expected image checked in for chromium-cg-mac.
Cary Clark
Comment 5 2011-10-07 13:19:37 PDT
Stephen, is there anything more you'd like me to do with this patch?
Stephen White
Comment 6 2011-10-07 13:22:33 PDT
Comment on attachment 110197 [details] Patch I'm going with Mike's OK. r=me
Stephen White
Comment 7 2011-10-07 13:23:38 PDT
I've set CQ+ as well, but if you can't be around to watch the canaries when it lands, please clear it until you can.
Cary Clark
Comment 8 2011-10-07 13:27:52 PDT
Comment on attachment 110197 [details] Patch I'll wait 'til Monday morning to commit so I can watch the Canaries.
WebKit Review Bot
Comment 9 2011-10-10 06:07:00 PDT
Comment on attachment 110197 [details] Patch Clearing flags on attachment: 110197 Committed r97058: <http://trac.webkit.org/changeset/97058>
WebKit Review Bot
Comment 10 2011-10-10 06:07:05 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.