44 bytes, text/x-github-pull-request
|Details | Review|
revert reftest manifest change now that the test here (and in bug 990627) should no longer fail on b2g
2.56 KB, patch
|Details | Diff | Splinter Review|
https://github.com/mozilla-b2g/moztt/commit/a425b5eaf4949ab17a1670442fc2b17a82c84a74 (bug 987582) caused failures in text/wordbreak-4a.html: On rev3s: https://tbpl.mozilla.org/php/getParsedLog.php?id=37076783&tree=B2g-Inbound On EC2: https://tbpl.mozilla.org/php/getParsedLog.php?id=37076511&tree=B2g-Inbound https://tbpl.mozilla.org/?tree=B2g-Inbound&jobname=emu.*reft&tochange=15c185155786&fromchange=ccea983731fd 16:25 <jfkthame> edmorley|sheriffduty, i think it's likely those oranges were triggered by the font changes in a425b5eaf4949ab17a1670442fc2b17a82c84a74, though i don't totally understand why yet 16:25 <jfkthame> edmorley|sheriffduty, and i suspect the real bug is elsewhere, and just happens to be exposed by the font change 16:27 <jfkthame> edmorley|sheriffduty, would it be an option to mark them as failing on that platform and file a bug (cc'ing me and :mwu)? ... 16:29 <jfkthame> edmorley|sheriffduty, yeah, it's only two reftests, both involving korean chars and missing glyphs... i think there's a real code bug involved but it'll take a bit of time to investigate
Hmm.. the use of compressed fonts is disabled by default though. I would expect the normal font set on the emulator.
(In reply to Michael Wu [:mwu] from comment #2) > Hmm.. the use of compressed fonts is disabled by default though. I would > expect the normal font set on the emulator. It turns out on examining the Android build stuff that the "DroidSansFallback.ttf" package was actually giving us the font called DroidSansFallbackFull.ttf, which has more complete Korean support than DroidSansFallback.ttf. So we should restore that. I just opened https://github.com/mozilla-b2g/moztt/pull/37 for this, but it appears to include several previous commits that AFAIK are already merged, so it needs some kind of cleaning up. I'll see if I can make that happen, though my git knowledge is minimal...
This should fix the test failure here and in bug 990627. We'll also need to revert the changes to the reftest manifests.
Assignee: nobody → jfkthame
Attachment #8400176 - Flags: review?(mwu)
This will need to be pushed when the DroidSansFallback update hits our builds, otherwise the tests will go orange with UNEXPECTED-PASS.
Attachment #8400179 - Flags: review?(mwu)
(In reply to Carsten Book [:Tomcat] from comment #7) > landed on https://hg.mozilla.org/integration/mozilla-inbound/rev/b07bd3a55770 Backed out since this needed to land alongside the moztt PR on b2g-inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/53faa7fe92a6 Relanded on b2g-inbound: remote: https://hg.mozilla.org/integration/b2g-inbound/rev/b6575e357b53
(In reply to Jonathan Kew (:jfkthame) from comment #5) > Created attachment 8400176 [details] [review] > Link to https://github.com/mozilla-b2g/moztt/pull/37 Merged earlier today: https://github.com/mozilla-b2g/moztt/commit/72043feddffa2bb787f6003ae952efe2d89fd818
All green: https://tbpl.mozilla.org/?tree=B2g-Inbound&rev=744e6df22a6e Thank you :-)
The following changeset is now in Firefox Nightly: > 35b1ed05daa1 Bug 990628 - Disable text/wordbreak-4a.html for failures after the landing of bug 987582; CLOSED TREE Nightly Build Information: ID: 20140402030201 Changeset: 4941a2ac0786109b08856738019b016a6c5a66a6 Version: 31.0a1 TBPL: https://tbpl.mozilla.org/?rev=4941a2ac0786 URL: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central Download Links: > Linux x86: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.linux-i686.tar.bz2 > Linux x86_64: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.linux-x86_64.tar.bz2 > Linux x86_64 ASAN: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.linux-x86_64-asan.tar.bz2 > Mac: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.mac.dmg > Win32: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.win32.installer.exe > Win64: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.win64-x86_64.installer.exe Previous Nightly Build Information: ID: 20140401030203 Changeset: 1417d180a1d8665b1a91b897d1cc4cc31e7980d4 Version: 31.0a1 TBPL: https://tbpl.mozilla.org/?rev=1417d180a1d8 URL: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-01-03-02-03-mozilla-central
AFAICS, this eventually landed and merged successfully, but the bug never got resolved. Closing.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.