Summary: | [chromium] fast/text/midword-break-before-surrogate-pair-2.html should be redesigned | ||
---|---|---|---|
Product: | WebKit | Reporter: | Tony Chang <tony> |
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED WONTFIX | ||
Severity: | Normal | CC: | bashi, eae, leviw, mitz, pkasting, rniwa, schenney |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Description
Tony Chang
2011-08-17 15:15:26 PDT
Also fails on Lion. From eseidel: "I argue the test is poorly designed and needs to be fixed to work better across platforms." From rniwa: "We should just skip these tests." I tend to side with eseidel here but what do I know. The Lion results changed with http://trac.webkit.org/changeset/124295 so this test is pretty sensitive to wrapping issues. Dan, isn't this test supposed to be ran only on Lion and onwards? We should probably move it to platform/mac. There was nothing platform-specific about the code change that this test covers, but I don’t know if non-OS X platforms have a font that would make it possible to test this. It seems OK to move the test to platform/mac. (In reply to comment #4) > There was nothing platform-specific about the code change that this test covers, but I don’t know if non-OS X platforms have a font that would make it possible to test this. It seems OK to move the test to platform/mac. Can we reproduce the bug you were fixing using some other surrogate pairs? (In reply to comment #5) > (In reply to comment #4) > > There was nothing platform-specific about the code change that this test covers, but I don’t know if non-OS X platforms have a font that would make it possible to test this. It seems OK to move the test to platform/mac. > > Can we reproduce the bug you were fixing using some other surrogate pairs? Like I said, I don’t know. I think none of the test fonts cover non-BMP characters. Levi, maybe you can look into this while you fix the midpoint bug in isolate. |