Bug 259313
| Summary: | Only special-case document element outermost <svg> embedded via a frame | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | SVG | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | sabouhallawa, simon.fraser, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | BrowserCompat, InRadar, WPTImpact |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| URL: | http://wpt.live/svg/extensibility/foreignObject/foreign-object-containing-svg-in-svg-in-object.html | ||
Ahmad Saleem
Hi Team,
This is just merge of Blink commit:
Commit: https://chromium-review.googlesource.com/c/chromium/src/+/4681730
It is to modify following:
https://github.com/WebKit/WebKit/blob/3dfb58beb2151d8039d5e56f6ddccddc9cd9d1ce/Source/WebCore/rendering/svg/LegacyRenderSVGRoot.cpp#L127
Change:
if (!frame().ownerRenderer())
to
if (!frame().ownerRenderer() || !isDocumentElementRenderer())
______
Similar in LBSE to fix both in one go. It compiles and work.
_____
Just raising to fix it.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
EWS
Committed 266251@main (4d88a0a85b71): <https://commits.webkit.org/266251@main>
Reviewed commits have been landed. Closing PR #16012 and removing active labels.
Radar WebKit Bug Importer
<rdar://problem/112779569>