471714 - reftests fail when character sets are not detected

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

We have reftests that test things like rtl ordering and hebrew characters.  Specifically there are 2 test cases in layout/reftest/bidi/413928*.html and 1 in layout/reftests/text/444656.html which reference the hebrew character set and these fail.

What happens is we display a box with the unicode number in there instead of the character.  The problem is the box is not the same shape/size in the test vs. the reference.

I am initially filing this under fennec as this is specific to fennec.  We might decide that fixing the boxes around nonprintable unicode characters in specific situations is not what we care about doing and in that case, I will move this to a reftest bug and make it a conditional test.

Comment 1

[:Gavin Sharp [email: [e-mail]]]

You're testing on the device, right? Sounds like it just doesn't fonts to display those characters.

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

correct, this is on device.  Still, what is printed does not match up (probably not a high priority).  If this is a won't fix for fennec, we need to ignore these reftests on fennec (work item for myself)

Comment 3

[Simon Montagu :smontagu]

So the other reftests in layout/reftests/bidi pass? This is possibly similar to or dependent on bug 469005

Comment 4

[Jonathan Kew [:jfkthame]]

Yes, this sounds similar to 469005, though I haven't looked into how fennec is doing font matching, which I think is the source of that issue. I'm working on some restructuring of font management that should give us a better handle on this and other style-matching issues, as well as preparing for Harfbuzz, but that's some way off yet.

Maybe marking the tests as TODO on fennec would be a good option for now? It's surely not a high-priority issue, but ought to eventually get fixed.

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

There are two other bidi tests which fail as per bug 471711.  

This does look very similar to bug 469005.  I will go ahead and work on marking these tests as TODO on fennec until we have a solution for this.

Comment 6

[Mark Finkle (:mfinkle) (use needinfo?)]

Fennec font matching should be using the normal linux code paths in our platform code. There might be device/OS pieces that we depend on which are missing or different (including fonts)

Comment 7

[Joel Maher ( :jmaher ) (UTC -8)]

keep in mind all the bidi tests pass on fennec desktop, but the noted failures are on maemo only.

Comment 8

[Gordon P. Hemsley [:GPHemsley]]

This bug has been buried in the graveyard and has not been updated in over 5 years. It is probably safe to assume that it will never be fixed, so resolving as WONTFIX.

[Mass-change filter: graveyard-wontfix-2014-09-24]