RESOLVED WONTFIX 66417
[chromium] fast/text/midword-break-before-surrogate-pair-2.html should be redesigned 
https://bugs.webkit.org/show_bug.cgi?id=66417
Summary [chromium] fast/text/midword-break-before-surrogate-pair-2.html should be red...
Tony Chang
Reported 2011-08-17 15:15:26 PDT
The following layout test is failing on Chromium Win and Linux fast/text/midword-break-before-surrogate-pair-2.html Probable cause: We don't have glyphs for the emoji characters used in the test. It would be nice if we could have the test use a different character. http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=fast%2Ftext%2Fmidword-break-before-surrogate-pair-2.html I will mark as failing for now.
Attachments
Add attachment proposed patch, testcase, etc.
Peter Kasting
Comment 1 2012-07-31 22:14:50 PDT
Also fails on Lion.
Peter Kasting
Comment 2 2012-07-31 22:29:29 PDT
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.
Ryosuke Niwa
Comment 3 2012-07-31 22:32:13 PDT
Dan, isn't this test supposed to be ran only on Lion and onwards? We should probably move it to platform/mac.
mitz
Comment 4 2012-07-31 22:37:38 PDT
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.
Ryosuke Niwa
Comment 5 2012-07-31 22:50:21 PDT
(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?
mitz
Comment 6 2012-07-31 23:06:12 PDT
(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.
Ryosuke Niwa
Comment 7 2013-02-03 14:08:34 PST
Levi, maybe you can look into this while you fix the midpoint bug in isolate.
Note You need to log in before you can comment on or make changes to this bug.