Closed Bug 1495941 Opened 6 years ago Closed 3 years ago

<embed> referencing SVG needs to access potentially cross-process layout information (intrinsic size)

Categories

(Core :: Layout: Images, Video, and HTML Frames, enhancement, P3)

Product:
Core
Component:
Layout: Images, Video, and HTML Frames
Type:
enhancement

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1658343
Project Flags:
Fission Milestone Future

People

(Reporter: heycam, Assigned: jwatt)

References

(Blocks 1 open bug)

Details

When an <embed> references an SVG document, nsSubdocumentFrame::ObtainIntrinsicSizeFrame pokes into the frame tree of the SVG document to find the frame that provides the intrinsic size of the containing <embed>.  This will need to change with Fission, but much of nsSubdocumentFrame is dealing with views and nsFrameLoader, so that probably all needs to be handled first anyway.
Blocks: rendering-fission
Priority: -- → P3

I think the way to handle this is to have the document in the subprocess message the embedding element with the information that it needs (the values that should be returned from GetIntrinsicSize, GetIntrinsicRatio, GetMinISize and GetPrefISize) and trigger a reflow.

Assignee: nobody → jwatt
Fission Milestone: --- → ?
Fission Milestone: ? → M4
Summary: <embed> referencing SVG needs to access potentially cross-process layout information → <embed> referencing SVG needs to access potentially cross-process layout information (intrinsic size)
Fission Milestone: M4 → M5
Fission Milestone: M5 → Future

I think this was fixed in bug 1658343. That replaced all of our ObtainIntrinsicSizeFrame calls with an API that uses IPC to communicate between the inner & outer document.

--> Duping.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE

(ni=jwatt to be sure I'm not missing something that he was intending to do here)

Flags: needinfo?(jwatt)

Nope. Thanks for dup'ing.

Flags: needinfo?(jwatt)
You need to log in before you can comment on or make changes to this bug.