Summary: | [SVG2] Fix SVGSVGElement to conform with SVG2 | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Said Abou-Hallawa <sabouhallawa> | ||||||
Component: | SVG | Assignee: | Said Abou-Hallawa <sabouhallawa> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cdumez, commit-queue, dino, esprehn+autocc, ews-watchlist, fmalita, gyuyoung.kim, kondapallykalyan, pdr, schenney, sergio, simon.fraser, webkit-bug-importer, zimmermann | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 191292, 200143 | ||||||||
Attachments: |
|
Description
Said Abou-Hallawa
2019-10-22 15:45:19 PDT
Created attachment 381638 [details]
Patch
Comment on attachment 381638 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=381638&action=review > Source/WebCore/svg/SVGSVGElement.cpp:266 > + element.document().updateLayoutIgnorePendingStylesheets(); You can't call updateLayoutIgnorePendingStylesheets() and guarantee that the element is still valid. This function should keep a Ref<SVGElement> around. > Source/WebCore/svg/SVGSVGElement.cpp:272 > + element.document().updateLayoutIgnorePendingStylesheets(); Ditto. Created attachment 381758 [details]
Patch
Comment on attachment 381758 [details] Patch Clearing flags on attachment: 381758 Committed r251527: <https://trac.webkit.org/changeset/251527> All reviewed patches have been landed. Closing bug. |