RESOLVED FIXED 192840
SVGUseElement::findTarget should return nullptr when there is a cycle 
https://bugs.webkit.org/show_bug.cgi?id=192840
Summary SVGUseElement::findTarget should return nullptr when there is a cycle
Ryosuke Niwa
Reported 2018-12-18 19:20:53 PST
https://trac.webkit.org/changeset/233366 added an early return to updateShadowTree() when there is a cycle but there is a separate cycle detection for when there is a corresponding element in SVGUseElement::findTarget. Support detecting a cycle in SVGUseElement::findTarget to consolidate these two cycle detection code paths.
Attachments
Cleanup (2.35 KB, patch)
2018-12-18 19:49 PST, Ryosuke Niwa 		thorton: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2018-12-18 19:49:01 PST
Created attachment 357647 [details] Cleanup
Tim Horton
Comment 2 2018-12-19 14:30:39 PST
Comment on attachment 357647 [details] Cleanup View in context: https://bugs.webkit.org/attachment.cgi?id=357647&action=review > Source/WebCore/ChangeLog:12 > + No new tests since there shoul be no behavioral change. s/shoul/should/
Ryosuke Niwa
Comment 3 2018-12-19 15:51:06 PST
Committed r239402: <https://trac.webkit.org/changeset/239402>
Radar WebKit Bug Importer
Comment 4 2018-12-19 15:52:26 PST
<rdar://problem/46855832>
Note You need to log in before you can comment on or make changes to this bug.