Bug 1341372 - Part 5: Update mochitest expectations. draft
authorBoris Chiou <boris.chiou@gmail.com>
Thu, 06 Apr 2017 13:23:42 +0800
changeset 563576 4168e45f82fb2aa1826eebff8e6b18db6f39dfb5
parent 563575 7e4a8bc1159720cef16f0a377e71ae4def039099
child 563577 00501cb1ec8580d4bda3c5a34f66b021cb66b380
push id54353
push userbmo:boris.chiou@gmail.com
push dateMon, 17 Apr 2017 09:07:14 +0000
bugs1341372
milestone55.0a1
Bug 1341372 - Part 5: 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
@@ -268,27 +268,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
@@ -67,28 +67,23 @@ to mochitest command.
     * test_supports_rules.html [1]
 * test_bug357614.html: case-insensitivity for old attrs in attr selector servo/servo#15006 [2]
 * test_bug387615.html: servo/servo#15006 [1]
 * test_bug397427.html: @import issue bug 1331291 and CSSOM support of @import [1]
 * console support bug 1352669
   * test_bug413958.html `monitorConsole` [3]
   * test_parser_diagnostics_unprintables.html [550]
 * 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_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 [1]
   * test_bug829816.html [8]
 * test_compute_data_with_start_struct.html `timing-function`: incorrectly computing keywords to bezier function servo/servo#15086 [2]
--- 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";