Bug 913990 - Test 'feed:' filter URL as crashtest instead of computed style test. r=dbaron

2014-10-03 10:06:07 -07:00 · 2014-10-03 10:06:07 -07:00 · a2d2bc4354
--- a/layout/style/test/property_database.js
+++ b/layout/style/test/property_database.js
@ -4651,11 +4651,9 @@ if (SpecialPowers.getBoolPref("layout.css.filters.enabled")) {
      "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)",
--- a/layout/style/test/test_value_computation.html
+++ b/layout/style/test/test_value_computation.html
@ -41,11 +41,6 @@
 /** 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" ],
--- a/layout/svg/crashtests/913990.html
+++ 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
@ -176,6 +176,7 @@ 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