Bug 1265611 - Crashtest; r=heycam
authorBrian Birtles <birtles@gmail.com>
Thu, 21 Apr 2016 17:05:46 +0900
changeset 332126 5e6bc73da82bac38f3ddbf090d16cf7174a9c934
parent 332125 147890124190c408f7f052da501c6734de79af48
child 332127 fe27ee21b66bee1f96d686e222bff3a8f654ecae
push id6048
push userkmoir@mozilla.com
push dateMon, 06 Jun 2016 19:02:08 +0000
treeherdermozilla-beta@46d72a56c57d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1265611 - Crashtest; r=heycam I have verified that without the fix in the first patch in this series this test fails, but passes with the fix applied. MozReview-Commit-ID: JmncnapbVLa
new file mode 100644
--- /dev/null
+++ b/layout/style/crashtests/1265611-1.html
@@ -0,0 +1,16 @@
+<!DOCTYPE html>
+  This test relies on triggering a transition on the 'color' property which,
+  at least when this test was written, would trigger a transition on the
+  -webkit-text-fill-color property since its default value is 'currentcolor'.
+  However, in crashtests.list we turn off layout.css.prefixes.webkit so
+  we should not trigger a transition on -webkit-test-fill-color.
+  This test exercises some code that, prior to this bug, would fail because we
+  would initially create the transition on -webkit-test-fill-color (because we
+  forgot to check if it was enabled or not) and then we would call other
+  methods that *do* check for the enabled-ness of the property leaving us
+  in an unexpected state.
+<body style="transition: all 4s" onload="document.body.style.color = 'green';"></body>
--- a/layout/style/crashtests/crashtests.list
+++ b/layout/style/crashtests/crashtests.list
@@ -134,13 +134,16 @@ load 1226400-1.html
 load 1227501-1.html
 load 1230408-1.html
 load 1233135-1.html
 load 1233135-2.html
 load 1238660-1.html
 load 1245260-1.html
 load 1247865-1.html
 load 1264396-1.html
+# The following test relies on -webkit-text-fill-color being behind the
+# layout.css.prefixes.webkit pref
+pref(layout.css.prefixes.webkit,false) load 1265611-1.html
 load border-image-visited-link.html
 load font-face-truncated-src.html 
 load large_border_image_width.html
 load long-url-list-stack-overflow.html
 pref(layout.css.background-clip-text.enabled,true) load 1264949.html