Fonts with no vertical information should still render CJK glyphs upright. See: http://people.mozilla.org/~jdaggett/tests/verticalmargins.html The font is Times-Roman, and it has Japanese glyphs but no vertical metrics. The glyphs should still render upright. Basically our code that is trying to make this decision purely based off the entire font is too simplistic. We need to do better.
Created attachment 73770 [details] Patch Looking for feedback. Not ready to land yet.
Attachment 73770 [details] did not build on chromium: Build output: http://queues.webkit.org/results/5730023
Attachment 73770 [details] did not build on qt: Build output: http://queues.webkit.org/results/5740035
I am not sure that this is an issue with missing vertical metric. Times font does not have Japanese glyphs and you are seeing them through the font fallback. The font used in the fallback is Hiragino, which has vertical metric. I guess the issue is around the handling of fallback fonts.
Created attachment 73781 [details] Patch
Attachment 73770 [details] did not build on chromium: Build output: http://queues.webkit.org/results/5750034
Comment on attachment 73781 [details] Patch Looks ok but would be good to test with a font that actually has CJK but not vertical support.
Attachment 73781 [details] did not build on qt: Build output: http://queues.webkit.org/results/5774035
It seems Font::isCJKIdeograph doesn't check CJK Extension B, C, D and CJK Compatibility Ideographs Supplement. They are mapped U+20000 and up. We need to count surrogate pairs here...
Attachment 73781 [details] did not build on chromium: Build output: http://queues.webkit.org/results/5840003
Created attachment 73807 [details] Patch to address Takano's concerns and that fixes @font-face bugs as well.
Attachment 73807 [details] did not build on chromium: Build output: http://queues.webkit.org/results/5985004
Comment on attachment 73807 [details] Patch to address Takano's concerns and that fixes @font-face bugs as well. r=me, but try not to break cr-linux and efl
Attachment 73807 [details] did not build on qt: Build output: http://queues.webkit.org/results/6007004
Going to keep using the bug to post patches to get the other ports building. This is just a matter of getting orientation() into everyone's FontPlatformData object, even if it just returns Horizontal on non-Mac platforms for now.
Created attachment 73808 [details] Patch
Fixed in r71970.
http://trac.webkit.org/changeset/71970 might have broken SnowLeopard Intel Release (Tests) The following tests are not passing: fast/blockflow/broken-ideographic-font.html