131085 2014-04-01 16:59:22 -0700 SVG-filtered anonymous renderers not updated when filter is loaded 2022-10-21 14:34:36 -0700 1 1 1 Unclassified WebKit SVG 528+ (Nightly build) Unspecified Unspecified NEW InRadar P2 Normal --- 1 jhoneycutt webkit-unassigned ahmad.saleem792 bfulgham ddkilzer dino koivisto simon.fraser thorton webkit-bug-importer zimmermann oldest_to_newest 996533 0 228341 jhoneycutt 2014-04-01 16:59:22 -0700 Created attachment 228341 Layout test If an anonymous render object is filtered with an SVG filter, and that filter appears in the document after the element (or if it loads sometime later), the renderer will not be updated/repaint when the filter loads. 996535 1 228342 jhoneycutt 2014-04-01 17:00:12 -0700 Created attachment 228342 Layout test reference 996551 2 jhoneycutt 2014-04-01 17:48:45 -0700 FilterEffectRenderer::buildReferenceFilter() does not handle the case that renderer->element() is false when the filter does not already exist in the document. I tried fixing this in two ways: 1) Using a non-null ancestor Element for anonymous objects. This doesn't work because the object may not yet exist in the RenderTree when this code is reached, so it has no ancestors. 2) Because we only use the renderer from the stored Element in RenderSVGResourceContainer::registerResource(), I tried changing SVGDocumentExtensions to store a set of RenderElements rather than Elements. This led to issues when building the list of pending resources for Elements that do not yet have associated renderers. 996634 3 jhoneycutt 2014-04-02 01:38:26 -0700 (In reply to comment #2) > FilterEffectRenderer::buildReferenceFilter() does not handle the case that renderer->element() is false That is, when renderer->element() is null. 1001989 4 webkit-bug-importer 2014-04-18 13:00:03 -0700 <rdar://problem/16661954> 1001996 5 ddkilzer 2014-04-18 13:07:52 -0700 *** Bug 129757 has been marked as a duplicate of this bug. *** 1907426 6 ahmad.saleem792 2022-10-21 14:34:36 -0700 I am able to reproduce this bug in Safari Technology Preview 156 using "Layout test" and first letter "F" is blurred while it is not the case with Chrome Canary 109 and Firefox Nightly 108. Thanks! 228341 2014-04-01 16:59:22 -0700 2014-04-01 16:59:22 -0700 Layout test first-letter.html text/html 1243 jhoneycutt PGhlYWQ+CiAgICA8c3R5bGU+CiAgICAgICAgI3RvcCB7IGhlaWdodDogMHB4OyB9CiAgICAgICAg ZGl2OmZpcnN0LWxldHRlciB7IC13ZWJraXQtZmlsdGVyOiB1cmwoI2JsdXJGaXJzdExldHRlcik7 IH0KICAgIDwvc3R5bGU+CjwvaGVhZD4KCjxkaXYgaWQ9InRvcCI+PC9kaXY+Cgo8ZGl2PkZpcnN0 IGxldHRlciBibHVyLjwvZGl2PgoKPHNjcmlwdD4KICAgIGlmICh3aW5kb3cudGVzdFJ1bm5lcikK ICAgICAgICB3aW5kb3cudGVzdFJ1bm5lci53YWl0VW50aWxEb25lKCk7CgogICAgd2luZG93LnNl dFRpbWVvdXQoZnVuY3Rpb24oKSB7CiAgICAgICAgdmFyIGRpdiA9IGRvY3VtZW50LmdldEVsZW1l bnRCeUlkKCJ0b3AiKTsKICAgICAgICB2YXIgc3ZnTlMgPSAiaHR0cDovL3d3dy53My5vcmcvMjAw MC9zdmciOwoKICAgICAgICB2YXIgc3ZnID0gZG9jdW1lbnQuY3JlYXRlRWxlbWVudE5TKHN2Z05T LCAic3ZnIik7CiAgICAgICAgc3ZnLnNldEF0dHJpYnV0ZSgieG1sbnMiLCBzdmdOUyk7CiAgICAg ICAgc3ZnLnNldEF0dHJpYnV0ZSgidmVyc2lvbiIsICIxLjEiKTsKICAgICAgICBzdmcuc2V0QXR0 cmlidXRlKCJ3aWR0aCIsICIwIik7CiAgICAgICAgc3ZnLnNldEF0dHJpYnV0ZSgiaGVpZ2h0Iiwg IjAiKTsKCiAgICAgICAgdmFyIGZpbHRlckRlZnMgPSBkb2N1bWVudC5jcmVhdGVFbGVtZW50TlMo c3ZnTlMsICJkZWZzIik7CgogICAgICAgIHZhciBmaWx0ZXIgPSBkb2N1bWVudC5jcmVhdGVFbGVt ZW50TlMoc3ZnTlMsICJmaWx0ZXIiKTsKICAgICAgICBmaWx0ZXIuc2V0QXR0cmlidXRlKCJpZCIs ICJibHVyRmlyc3RMZXR0ZXIiKTsKCiAgICAgICAgdmFyIGJsdXIgPSBkb2N1bWVudC5jcmVhdGVF bGVtZW50TlMoc3ZnTlMsICJmZUdhdXNzaWFuQmx1ciIpOwogICAgICAgIGJsdXIuc2V0QXR0cmli dXRlTlMobnVsbCwgImluIiwgIlNvdXJjZUdyYXBoaWMiKTsKICAgICAgICBibHVyLnNldEF0dHJp YnV0ZU5TKG51bGwsICJzdGREZXZpYXRpb24iLCAiMSIpOwoKICAgICAgICBmaWx0ZXIuYXBwZW5k Q2hpbGQoYmx1cik7CiAgICAgICAgZmlsdGVyRGVmcy5hcHBlbmRDaGlsZChmaWx0ZXIpOwogICAg ICAgIHN2Zy5hcHBlbmRDaGlsZChmaWx0ZXJEZWZzKTsKICAgICAgICBkaXYuYXBwZW5kQ2hpbGQo c3ZnKTsKCiAgICAgICAgaWYgKHdpbmRvdy50ZXN0UnVubmVyKQogICAgICAgICAgICB3aW5kb3cu dGVzdFJ1bm5lci5ub3RpZnlEb25lKCk7CiAgICB9LCAwKTsKPC9zY3JpcHQ+Cg== 228342 2014-04-01 17:00:12 -0700 2014-04-01 17:00:12 -0700 Layout test reference first-letter-expected.html text/html 460 jhoneycutt PGhlYWQ+CiAgICA8c3R5bGU+CiAgICAgICAgI3RvcCB7IGhlaWdodDogMHB4OyB9CiAgICAgICAg I2JsdXIgeyAtd2Via2l0LWZpbHRlcjogdXJsKCNibHVyRmlyc3RMZXR0ZXIpOyB9CiAgICA8L3N0 eWxlPgo8L2hlYWQ+Cgo8ZGl2IGlkPSJ0b3AiPgogICAgPHN2ZyB4bWxucz0iaHR0cDovL3d3dy53 My5vcmcvMjAwMC9zdmciIHZlcnNpb249IjEuMSIgd2lkdGg9IjAiIGhlaWdodD0iMCI+CiAgICAg ICAgPGRlZnM+CiAgICAgICAgICAgIDxmaWx0ZXIgaWQ9ImJsdXJGaXJzdExldHRlciI+CiAgICAg ICAgICAgICAgICA8ZmVHYXVzc2lhbkJsdXIgaW49IlNvdXJjZUdyYXBoaWMiIHN0ZERldmlhdGlv bj0iMSIvPgogICAgICAgICAgICA8L2ZpbHRlcj4KICAgICAgICA8L2RlZnM+CiAgICA8L3N2Zz4K PC9kaXY+Cgo8ZGl2PjxzcGFuIGlkPSJibHVyIj5GPC9zcGFuPmlyc3QgbGV0dGVyIGJsdXIuPC9k aXY+Cg==