1149535 - test_unicode_range_loading.html is going to permafail when Gecko 39 merges to Beta

Status: VERIFIED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Ryan VanderMeulen [:RyanVM]]

https://treeherder.mozilla.org/logviewer.html#?job_id=6108363&repo=try

07:01:31 INFO - 824 INFO TEST-UNEXPECTED-FAIL | layout/style/test/test_unicode_range_loading.html | TEST 2 simple load sanity check, font for an unused character not loaded (@font-face rules) after load - TEST 2 simple load sanity check, font for an unused character not loaded (@font-face rules) after load: assert_equals: font loaded - font 0 markA {"unicodeRange":"u+41"} for content BBB expected "unloaded" but got "loaded"
07:01:31 INFO - 825 INFO TEST-UNEXPECTED-FAIL | layout/style/test/test_unicode_range_loading.html | TEST 2 simple load sanity check, font for an unused character not loaded (@font-face rules) after load and timeout - TEST 2 simple load sanity check, font for an unused character not loaded (@font-face rules) after load and timeout: assert_equals: font loaded - font 0 markA {"unicodeRange":"u+41"} for content BBB expected "unloaded" but got "loaded"

etc...

Comment 1

[John Daggett (:jtd)]

Attachment: patch, enable unicode-range pref for testing

tryserver build:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c06905717f1b

Comment 2

[John Daggett (:jtd)]

Pushed to inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a8f3297ddd22

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/a8f3297ddd22

Comment 4

[John Daggett (:jtd)]

Comment on attachment 8586548 [details] [diff] [review]
patch, enable unicode-range pref for testing

Approval Request Comment
[Feature/regressing bug #]: testcase on bug 1094571 will fail unless unicode-range pref is enabled
[User impact if declined]: permafail for unicode-range load test
[Describe test coverage new/current, TreeHerder]: test landed last week on trunk
[Risks and why]: low risk, simply enabling unicode-range for testing
[String/UUID change made/needed]: none

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8586548 [details] [diff] [review]
patch, enable unicode-range pref for testing

This is already on my radar to land a=test-only :)

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/e7bb84d23921