[FIX]object/embed should implement nsIChromeEventHandler

RESOLVED FIXED

Status

()

Core
DOM
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: bz, Assigned: smaug)

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 253807 [details] [diff] [review]
Do what frames do

That way an SVG <object> in a chrome document, say, won't assert.
Attachment #253807 - Flags: superreview?(jst)
Attachment #253807 - Flags: review?(jst)
(Assignee)

Comment 1

11 years ago
But I will kill nsIChromeEventHandler in Bug 363067.
Depends on: 363067
Comment on attachment 253807 [details] [diff] [review]
Do what frames do

Ah, good point.  ;)
Attachment #253807 - Flags: superreview?(jst)
Attachment #253807 - Flags: superreview-
Attachment #253807 - Flags: review?(jst)
Attachment #253807 - Flags: review-
(Assignee)

Comment 3

11 years ago
Is there a test case for this? Does this work now?
The testcase is an <object> containing SVG in a chrome document.  The symptom is an assert in nsFrameLoader::EnsureDocShell.

And this should be fixed, since <object>s do QI to nsIDOMEventTarget.

Thank you!
Assignee: bzbarsky → Olli.Pettay
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.