Don't apply filter or clip-path in SVGForeignObjectPainter.

chrishtr · chromium-wpt-export-bot · commit 2a0b3d690c0b · 2019-08-08T15:22:03.000-04:00
SVG foreign objects always have PaintLayers now, since they are stacking contexts. The filter and clip-path are applied by PaintLayerPainter. Therefore we shouldn't try to apply them in SVGForeignObjectPainter, nor should we early-out if they "fail" to apply. Bug: 989673 Change-Id: I4a20da8f8ea85d377b25d8376d591f36f172d854 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1743256 Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Reviewed-by: Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#685285}
diff --git a/svg/extensibility/foreignObject/filter-repaint-ref.html b/svg/extensibility/foreignObject/filter-repaint-ref.html
@@ -0,0 +1,15 @@
+<html>
+  <svg>
+  <foreignObject style="overflow: visible;"
+      width="400" height="400" filter="url(#blur)">
+    <div id=target>test</div>
+  </foreignObject>
+  <g>
+    <defs>
+        <filter id="blur">
+          <feGaussianBlur stdDeviation="0.1"/>
+        </filter>
+    </defs>
+    </g>
+  </svg>
+</html>
diff --git a/svg/extensibility/foreignObject/filter-repaint.html b/svg/extensibility/foreignObject/filter-repaint.html
@@ -0,0 +1,24 @@
+<!doctype HTML>
+<title>Test that re-painting a foreignObject element with a filter works.</title>
+<link rel="match" href="filter-repaint-ref.html">
+<link rel="author" title="Chris Harrelson" href="mailto:chrishtr@chromium.org">
+<link rel="help" href="https://svgwg.org/svg2-draft/single-page.html#embedded-ForeignObjectElement"/>
+<svg>
+  <foreignObject style="overflow: visible;"
+        width="400" height="400" filter="url(#blur)">
+    <div id=target>test</div>
+  </foreignObject>
+  <g>
+    <defs>
+      <filter id="blur">
+        <feGaussianBlur stdDeviation="0.1"/>
+      </filter>
+  </defs>
+  </g>
+</svg>
+<script>
+onload = () => {
+  requestAnimationFrame(() => requestAnimationFrame(() =>
+    target.style.width = '11px'));
+}
+</script>
