Bug 913990 - Test 'feed:' filter URL as crashtest instead of computed style test. r=dbaron
authorMax Vujovic <mvujovic@adobe.com>
Fri, 03 Oct 2014 10:06:07 -0700
changeset 231981 7fa34faa8330b1258eeefaebdfa94e6f01ae9a9a
parent 231980 9b157630e5abcc12a740021a03a19d8bbd01223f
child 231982 697c4b245de04c10bd712b4d8b8c8becfb28e98e
push id4187
push userbhearsum@mozilla.com
push dateFri, 28 Nov 2014 15:29:12 +0000
treeherdermozilla-beta@f23cc6a30c11 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdbaron
bugs913990
milestone35.0a1
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 913990 - Test 'feed:' filter URL as crashtest instead of computed style test. r=dbaron
layout/style/test/property_database.js
layout/style/test/test_value_computation.html
layout/svg/crashtests/913990.html
layout/svg/crashtests/crashtests.list
--- a/layout/style/test/property_database.js
+++ b/layout/style/test/property_database.js
@@ -4646,21 +4646,19 @@ if (SpecialPowers.getBoolPref("layout.cs
       // Filter functions
       "opacity(50%) saturate(1.0)",
       "invert(50%) sepia(0.1) brightness(90%)",
 
       // Mixed SVG reference filters and filter functions
       "grayscale(1) url(#my-filter-1)",
       "url(#my-filter-1) brightness(50%) contrast(0.9)",
 
-      // The CSS parser will accept these weird URLs. However, we'll fail
-      // to resolve them when computing style, so we'll fall back to the
-      // initial value ("none").
-      "url('feed:javascript:5')",
-      "blur(3px) url('feed:javascript:5') grayscale(50%)",
+      // Bad URLs
+      "url('badscheme:badurl')",
+      "blur(3px) url('badscheme:badurl') grayscale(50%)",
 
       "blur(0)",
       "blur(0px)",
       "blur(0.5px)",
       "blur(3px)",
       "blur(100px)",
       "blur(0.1em)",
       "blur(calc(-1px))", // Parses and becomes blur(0px).
--- a/layout/style/test/test_value_computation.html
+++ b/layout/style/test/test_value_computation.html
@@ -36,21 +36,16 @@
   
 </div>
 <pre id="test">
 <script class="testbody" type="text/javascript">
 
 /** Test for computation of values in property database **/
 
 var gBadComputed = {
-  // The CSS parser will accept these weird URLs. However, we'll fail to
-  // resolve them when computing style, so we'll fall back to the initial
-  // value ("none").
-  "filter": [ "url('feed:javascript:5')", "blur(3px) url('feed:javascript:5') grayscale(50%)" ],
-
   // These values are treated as auto.
   "page-break-after": [ "avoid" ],
   "page-break-before": [ "avoid" ],
 
   // This is the only SVG-length property (i.e., length allowing
   // unitless lengths) whose initial value is zero.
   "stroke-dashoffset": [ "0", "-moz-objectValue" ],
 };
new file mode 100644
--- /dev/null
+++ b/layout/svg/crashtests/913990.html
@@ -0,0 +1,5 @@
+<!DOCTYPE html>
+<html>
+<body style="filter: url('feed:javascript:5');">
+</body>
+</html>
--- a/layout/svg/crashtests/crashtests.list
+++ b/layout/svg/crashtests/crashtests.list
@@ -171,16 +171,17 @@ load 881031-1.svg
 load 885608-1.svg
 load 890782-1.svg
 load 890783-1.svg
 load 893510-1.svg
 load 895311-1.svg
 load 897342-1.svg
 load 898909-1.svg
 load 898951-1.svg
+load 913990.html
 load 919371-1.xhtml
 load 952270-1.svg
 load 963086-1.svg
 load 975773-1.svg
 load 974746-1.svg
 load 979407-1.svg
 load 979407-2.svg
 load 993443.svg