Bug 1328522 - Disable test_restyles_in_smil_animation.html; r=hiro
This is failing very frequently so until we have time to analyze it, we should
disable it on Android.
This patch also makes the failure message more informative so that if it fails
on other platforms we might have some idea how it is failing.
MozReview-Commit-ID: KYNDKYAh10c
--- a/layout/style/test/mochitest.ini
+++ b/layout/style/test/mochitest.ini
@@ -249,16 +249,17 @@ skip-if = android_version == '18' #debug
[test_property_database.html]
[test_property_syntax_errors.html]
[test_pseudoelement_state.html]
[test_pseudoelement_parsing.html]
[test_redundant_font_download.html]
support-files = redundant_font_download.sjs
[test_rem_unit.html]
[test_restyles_in_smil_animation.html]
+skip-if = toolkit == 'android' # bug 1328522
[test_root_node_display.html]
[test_rule_insertion.html]
[test_rule_serialization.html]
[test_rules_out_of_sheets.html]
[test_selectors.html]
skip-if = toolkit == 'android' #bug 775227
[test_selectors_on_anonymous_content.html]
[test_setPropertyWithNull.html]
--- a/layout/style/test/test_restyles_in_smil_animation.html
+++ b/layout/style/test/test_restyles_in_smil_animation.html
@@ -81,17 +81,18 @@ add_task(function* smil_is_in_display_no
document.getElementById("svg-rect").appendChild(animate);
yield waitForPaintFlushed();
var displayMarkers = yield observeStyling(5);
// FIXME: Bug 866411: SMIL animations sometimes skip restyles when the target
// element is newly associated with an nsIFrame.
ok(displayMarkers.length == 5 || displayMarkers.length == 4,
- "should restyle in most frames");
+ `should restyle in most frames (got ${displayMarkers.length} restyles`
+ + ' over five frames, expected 4~5)');
var div = document.getElementById("target-div");
div.style.display = "none";
getComputedStyle(div).display;
yield waitForPaintFlushed();
var displayNoneMarkers = yield observeStyling(5);
@@ -100,14 +101,15 @@ add_task(function* smil_is_in_display_no
div.style.display = "";
getComputedStyle(div).display;
yield waitForPaintFlushed();
var displayAgainMarkers = yield observeStyling(5);
// FIXME: Bug 866411: SMIL animations sometimes skip restyles when the target
// element is newly associated with an nsIFrame.
ok(displayMarkers.length == 5 || displayMarkers.length == 4,
- "should restyle again");
+ `should restyle again (got ${displayMarkers.length} restyles over`
+ + ' five frames, expected 4~5)');
yield ensureElementRemoval(animate);
});
</script>
</body>