Closed
Bug 369153
Opened 18 years ago
Closed 17 years ago
[FIX]object/embed should implement nsIChromeEventHandler
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bzbarsky, Assigned: smaug)
References
Details
Attachments
(1 file)
|
6.66 KB,
patch
|
bzbarsky
:
review-
bzbarsky
:
superreview-
|
Details | Diff | Splinter Review |
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•18 years ago
|
||
But I will kill nsIChromeEventHandler in Bug 363067.
Depends on: 363067
|
Reporter | |
Comment 2•18 years ago
|
||
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•17 years ago
|
||
Is there a test case for this? Does this work now?
|
|
Reporter | |
Comment 4•17 years ago
|
||
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
|
|
Reporter | |
Updated•17 years ago
|
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
|
||
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•