401053 - Hang initializing scriptable Java applet

Status: VERIFIED FIXED

(Core Graveyard :: Plug-ins, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

I hope I filed this in the correct component.

Build ID: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a9pre) Gecko/2007102404 Minefield/3.0a9pre

Summary: 

Steps to Reproduce:

1. Load http://www.nintendo8.com/game/314/super_mario_brothers_3/
2. Click in the applet's area (middle of the page)

Expected Results:

Applet initializes

Actual Results:

Once you click the applet's area, Minefield "hangs"--stops responding to
mouse/keyboard events--but doesn't enter the "Not Responding" state, according
to the OS.

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

I should mention that it has to be the first time you've tried to initialize Java in a given browsing session (it matters).  If it's not, you don't freeze, the applet just doesn't load.

Comment 2

[Johnny Stenback (:jst)]

The key here is that the site in the testcase uses an <object> element instead of an <applet> element to instantiate the applet that's causing problems. I need to move the fix for bug 399524 to a place where it applies for all plugin loading elements, but only when loading a Java plugin...

Comment 3

[Johnny Stenback (:jst)]

Attachment: Fix.

This basically moves the fix for bug 399406 to a place in the plugin code that we'll hit no matter what markup the page uses for the plugin.

Comment 4

[Daniel Veditz [:dveditz]]

Comment on attachment 286761 [details] [diff] [review]
Fix.

r/sr=dveditz

Comment 5

[Johnny Stenback (:jst)]

Comment on attachment 286761 [details] [diff] [review]
Fix.

This would be good to get in for the beta, fixes a recent Java plugin regression on the mac on pages that use <object> tags to load Java applets. Not sure how common that is. The fix is really safe and is really just moving a hunk of code (even though that's not entirely obvious from a quick look at the patch).

Comment 6

[Damon Sicore (:damons)]

Comment on attachment 286761 [details] [diff] [review]
Fix.

a=damons for beta.  Let's get this in ASAP.  Thanks for knocking this out.

Comment 7

[Johnny Stenback (:jst)]

Fixed.

Comment 8

[Carsten Book [:Tomcat]]

verified fixed using Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9a9pre) Gecko/2007110105 Minefield/3.0a9pre using the testurl from comment #0. The applet/game works fine 

-> Verified