Only consider styled SVG elements for fastAttributeLookupAllowed
Created attachment 180386 [details] Patch
A have a more correct patch coming in a sec
Created attachment 180393 [details] Patch
Comment on attachment 180393 [details] Patch Looks reasonable to me. I've cc'd antti/kling who are the real experts for the fastAttr stuff.
(In reply to comment #4) > (From update of attachment 180393 [details]) > Looks reasonable to me. I've cc'd antti/kling who are the real experts for the fastAttr stuff. LGTM!
(In reply to comment #4) > (From update of attachment 180393 [details]) > Looks reasonable to me. I've cc'd antti/kling who are the real experts for the fastAttr stuff. This also seems reasonable to me. I went through all the animatable elements with the class attribute and it looks like you're covered (seriously, <desc> is stylable?)
(In reply to comment #6) > (In reply to comment #4) > > (From update of attachment 180393 [details] [details]) > > Looks reasonable to me. I've cc'd antti/kling who are the real experts for the fastAttr stuff. > > This also seems reasonable to me. I went through all the animatable elements with the class attribute and it looks like you're covered (seriously, <desc> is stylable?) It looks like the good folks working on SVG2 have fixed this so we will be able to remove the special if check in the future: https://svgwg.org/svg2-draft/single-page.html#types-InterfaceSVGElement
Comment on attachment 180393 [details] Patch Clearing flags on attachment: 180393 Committed r138296: <http://trac.webkit.org/changeset/138296>
All reviewed patches have been landed. Closing bug.