Bug 1341372 - Part 9: Update mochitest expectations. draft
authorBoris Chiou <boris.chiou@gmail.com>
Thu, 06 Apr 2017 13:23:42 +0800
changeset 563353 c4de49e4fa010067a31d269f22a51a3dc68b8bbe
parent 563352 95749c12bdd6da38df977e57d6adbc557dadf177
child 563354 ea9b83be8450902c736931604aea3efc2608d658
push id54259
push userbmo:boris.chiou@gmail.com
push dateSun, 16 Apr 2017 06:29:51 +0000
bugs1341372
milestone55.0a1
Bug 1341372 - Part 9: Update mochitest expectations. Also update test_transitions_dynamic_changes.html because we don't support "font" property for stylo now. According to its context and purpose, we only need a property other than "text-indent", so it is not necessary to use "font". Besides, we don't support shorthand properties for transitions, so test_transitions.html have two intermittents. Mark the expect number as '*' until we fix shorthand properties for transitions. MozReview-Commit-ID: HyuOV29AJ9a
layout/style/test/mochitest.ini
layout/style/test/stylo-failures.md
layout/style/test/test_transitions_dynamic_changes.html
--- a/layout/style/test/mochitest.ini
+++ b/layout/style/test/mochitest.ini
@@ -269,27 +269,25 @@ skip-if = toolkit == 'android' #bug 7752
 [test_style_attribute_standards.html]
 [test_style_struct_copy_constructors.html]
 [test_supports_rules.html]
 [test_system_font_serialization.html]
 [test_text_decoration_shorthands.html]
 [test_transitions_and_reframes.html]
 [test_transitions_and_restyles.html]
 [test_transitions_and_zoom.html]
-skip-if = stylo # timeout bug 1328499
 [test_transitions_cancel_near_end.html]
 skip-if = stylo # timeout bug 1328499
 [test_transitions_computed_values.html]
 [test_transitions_computed_value_combinations.html]
 [test_transitions_events.html]
 skip-if = stylo # timeout bug 1328499
 [test_transitions.html]
 skip-if = (android_version == '18' && debug) # bug 1159532
 [test_transitions_bug537151.html]
-skip-if = stylo # timeout bug 1328499
 [test_transitions_dynamic_changes.html]
 [test_transitions_per_property.html]
 skip-if = (toolkit == 'android' || stylo) # bug 775227 for android, bug 1292283 for stylo
 [test_transitions_replacement_on_busy_frame.html]
 skip-if = stylo # timeout bug 1328503
 support-files = file_transitions_replacement_on_busy_frame.html
 [test_transitions_step_functions.html]
 [test_transitions_with_disabled_properties.html]
--- a/layout/style/test/stylo-failures.md
+++ b/layout/style/test/stylo-failures.md
@@ -75,28 +75,23 @@ to mochitest command.
   * test_html_attribute_computed_values.html: also list-style-type [8]
 * test_bug387615.html: servo/servo#15006 [1]
 * test_bug397427.html: @import issue bug 1331291 and CSSOM support of @import [1]
 * console support:
   * test_bug413958.html `monitorConsole` [3]
   * test_parser_diagnostics_unprintables.html [550]
 * test_bug511909.html: @-moz-document and @media support [4]
 * Transition support:
-  * test_bug621351.html [4]
+  * test_bug621351.html: shorthand properties [4]
   * test_compute_data_with_start_struct.html `transition` [2]
-  * test_transitions.html [63]
-  * test_transitions_and_reframes.html [16]
-  * test_transitions_and_restyles.html [3]
+  * test_transitions.html: pseudo elements and shorthand properties [*]
   * test_transitions_computed_value_combinations.html [145]
-  * test_transitions_dynamic_changes.html [10]
-  * test_transitions_step_functions.html [24]
   * test_value_storage.html `transition` [596]
   * Events:
-    * test_animations_event_handler_attribute.html [10]
-    * test_animations_event_order.html [11]
+    * test_animations_event_order.html [2]
 * test_bug798843_pref.html: conditional opentype svg support [7]
 * test_computed_style.html `gradient`: -moz-prefixed radient value [9]
 * ... `mask`: mask-image isn't set properly bug 1341667 [10]
 * ... `fill`: svg paint should distinguish whether there is fallback bug 1347409 [2]
 * ... `stroke`: svg paint should distinguish whether there is fallback bug 1347409 [2]
 * character not properly escaped servo/servo#15947
   * test_parse_url.html [4]
   * test_bug829816.html [8]
--- a/layout/style/test/test_transitions_dynamic_changes.html
+++ b/layout/style/test/test_transitions_dynamic_changes.html
@@ -31,17 +31,17 @@ p.style.textIndent = "0";
 is(cs.textIndent, "50px", "transition is halfway");
 p.style.transitionDuration = "0s";
 is(cs.textIndent, "50px", "changing duration doesn't change transitioning");
 p.style.transitionDelay = "0s";
 is(cs.textIndent, "50px", "changing delay doesn't change transitioning");
 p.style.transitionProperty = "text-indent";
 is(cs.textIndent, "50px",
    "irrelevant change to transition property doesn't change transitioning");
-p.style.transitionProperty = "font";
+p.style.transitionProperty = "letter-spacing";
 is(cs.textIndent, "0px",
    "relevant change to transition property does change transitioning");
 
 /** Test for Bug 522643 */
 p.style.transitionDuration = "4s";
 p.style.transitionDelay = "-2s";
 p.style.transitionProperty = "text-indent";
 p.style.textIndent = "100px";