1355817 - Fix and re-enable layout/base/tests/test_bug394057.html on Android

Status: NEW

(Core :: Layout: Text and Fonts, defect, P3)

Description

[Ryan VanderMeulen [:RyanVM]]

As part of getting layout/base/tests running on Android again, it was determined that test_bug394057.html is currently permafailing. In the interests of getting the directory running again, the test has been disabled for now until it can be fixed and re-enabled.

241 INFO TEST-START | layout/base/tests/test_bug394057.html
242 INFO TEST-PASS | layout/base/tests/test_bug394057.html | can't find serif and monospace fonts of different width
243 INFO TEST-UNEXPECTED-FAIL | layout/base/tests/test_bug394057.html | changing font pref should change width of table (serif) - got 149, expected 150
    SimpleTest.is@SimpleTest/SimpleTest.js:310:5
    step3@layout/base/tests/test_bug394057.html:77:5
244 INFO TEST-UNEXPECTED-FAIL | layout/base/tests/test_bug394057.html | changing font pref should change width of table (monospace) - got 162, expected 158
    SimpleTest.is@SimpleTest/SimpleTest.js:310:5
    step3@layout/base/tests/test_bug394057.html:79:5
245 INFO TEST-OK | layout/base/tests/test_bug394057.html | took 3479ms

Comment 1

[Ryan VanderMeulen [:RyanVM]]

NI myself to see if we can re-enable these now.

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Still fails:

WARNING -  TEST-UNEXPECTED-FAIL | layout/base/tests/test_bug394057.html | changing font pref should change width of table (monospace) - got 162, expected 152
   INFO -      SimpleTest.is@SimpleTest/SimpleTest.js:500:14
   INFO -      step3@layout/base/tests/test_bug394057.html:79:7

Jonathan, any thoughts on that failure?

Comment 3

[Jonathan Kew [:jfkthame]]

Nothing immediately comes to mind, except that I notice this apparently used to fail on Windows/ASAN as well, which makes me wonder if it's some kind of timing-related issue where slower systems fail to completely handle the pref-change in time.

(It'd be interesting to know if there's a user-visible bug on Android whereby such pref changes don't trigger the necessary reflow, or if it's just an issue in the test environment.)