Closed Bug 843086 Opened 7 years ago Closed 4 years ago

<applet> root causes "ASSERTION: Should not be initializing plugin without a frame"

Categories

(Core :: Plug-ins, defect, P3)

x86_64
macOS
defect

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: jruderman, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: assertion, testcase)

Attachments

(2 files)

48 bytes, application/xhtml+xml
Details
4.04 KB, text/plain
Details
Attached file testcase
###!!! ASSERTION: Should not be initializing plugin without a frame: 'Not Reached', file dom/plugins/base/nsPluginInstanceOwner.cpp, line 2967
Attached file stack
I have the "Java Plugin" shim that comes with Mac OS X 10.8, but not Java.
Priority: -- → P3
So we're creating a frame for the document that is not an object frame:

http://dxr.mozilla.org/mozilla-central/dom/plugins/base/nsPluginInstanceOwner.cpp#l2956

GetPrimaryFrame() succeeds, so we pass the nsObjectLoadingFrame has-frame checks, but bail out because it's not an object frame. This should be harmless due to the bail out, but I'm not sure what the right fix is - other than just refusing to let plugins be the root.
Why is the plugin the root? Don't we automatically add body frames and such? Or is this a problem only because this is xhtml+xml, so we don't follow normal HTML rules?
(In reply to Benjamin Smedberg  [:bsmedberg] from comment #4)
> Why is the plugin the root? Don't we automatically add body frames and such?
> Or is this a problem only because this is xhtml+xml, so we don't follow
> normal HTML rules?

I have no idea.

I tested and this does not happen with object or embed:

> <object type="application/x-test" xmlns="http://www.w3.org/1999/xhtml" />
> <embed type="application/x-test" xmlns="http://www.w3.org/1999/xhtml" />

So something about how we make the frame tree for applet is wrong
No longer reproduces.
Status: NEW → RESOLVED
Closed: 4 years ago
Flags: in-testsuite+
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.