Open
Bug 1066833
Opened 10 years ago
Updated 2 years ago
LoadInfo should include the loading Element
Categories
(Core :: DOM: Security, defect)
Core
DOM: Security
Tracking
()
REOPENED
People
(Reporter: tanvi, Unassigned)
References
Details
(Whiteboard: [domsecurity-backlog])
Attachments
(1 file)
3.29 KB,
patch
|
Details | Diff | Splinter Review |
We have had many discussions about whether the "loadingNode" that we are adding in Bug 1038756 should be the actual element that is being added to a document or if it should be the document where the load is happening.
Including the loadingNode gives us more information, since we can just call OwnerDoc() on it to get the document. At least for the TYPE_SCRIPT, TYPE_IMAGE, and TYPE_STYLESHEET cases.
This bug is to figure out whether we should loadingElement to loadInfo or replace loadingNode with the loadingElement in certain cases. And then update the callsites accordingly.
Reporter | ||
Comment 1•10 years ago
|
||
Some rough notes from a discussion on this topic - http://pastebin.mozilla.org/6456755
Comment 2•9 years ago
|
||
Given the way we progressed with the LoadInfo, I think this bug has become invalid over time.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → INVALID
Whiteboard: [domsecurity-backlog]
Reporter | ||
Comment 3•9 years ago
|
||
This is still valid - we could start passing the loadingElement instead of the loadingNode to loadInfo. I believe <video> and <audio> elements are passed instead of the document they are contained in.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Comment 4•9 years ago
|
||
(In reply to Tanvi Vyas [:tanvi] from comment #3)
> This is still valid - we could start passing the loadingElement instead of
> the loadingNode to loadInfo. I believe <video> and <audio> elements are
> passed instead of the document they are contained in.
Oh ok, sorry for closing it prematurely. I thought the way we progressed with loadInfo basically caused this bug never to be finished anyway.
Comment 5•8 years ago
|
||
Basic WIP patch that adds a loadingElement to the idl and sets it if the loadingNode is an element. Works reliably for iframes.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•