369894 - Minimize button stops working after loading Java applet (JEP 0.9.6, Carbon-only, Tiger and above)

Status: RESOLVED FIXED

(Plugins Graveyard :: Java (Java Embedding Plugin), defect)

Description

[Steven Michaud [:smichaud] (Retired)]

In all Carbon-based browsers (Firefox and Seamonkey) that bundle JEP
0.9.6, the minimize button (aka the collapse button) stops working in
a browser window (on OS X 10.4.X and above) after at least one Java
applet has been loaded in it.

This doesn't happen with earlier versions of the JEP.

This doesn't happen with Cocoa-based browsers (i.e. Camino).

This doesn't happen on OS X 10.3.X and 10.2.8.

I already have a 2-3 line fix for this problem, which I'll post in my
next message.  If fixing this is urgent enough (and I think it
probably is), I can release a new JEP "nightly" over the weekend that
contains nothing but this fix.  I'll call it JEP 0.9.6+a and post it
on my SourceForge site (http://javaplugin.sourceforge.net/).  The plan
would be to bundle it with Firefox 2.0.0.2 (and Firefox 1.5.0.10 and
so forth).

I don't think it's an option to revert back to JEP 0.9.5+g+2 -- 0.9.6
contains too many important fixes (see bug 364158).

I'm really embarrassed that I missed this ... though it makes me feel
better that everyone else seems to have missed it too.  I guess I've
become an orthodox Mac user -- I use "hide" much more often than I use
"minimize".

Comment 1

[Samuel Sidler (old account; do not CC)]

I think it might be too late for 1.8.1.2/1.8.0.10, but requesting blocking anyway.

Comment 2

[Steven Michaud [:smichaud] (Retired)]

Attachment: Fix that would be implemented in JEP 0.9.6+a

To fix this problem I had to change the workaround for a different
problem.  That workaround is old (it was present in earlier versions
of the JEP), and used to work just fine.  But it was (partially)
broken when I stopped using Apple's buggy Carbon event handlers to
translate Carbon events to Cocoa events.

Now I've found another (better) way to solve the (old) problem (one
that wouldn't have worked if I were still using Apple's handlers).

Comment 3

[Steven Michaud [:smichaud] (Retired)]

I'm going to postpone creating a JEP 0.9.6+a (one that contains the
fix in attachment 254561 [details] [diff] [review]) until Mozilla.org decides to block Firefox
2.0.0.2 and 1.5.0.10 for a fix to this problem.

If I don't do a JEP 0.9.6+a release now, I'll probably do one in
couple of weeks that also contains a fix for a very-seldom-encountered
crash bug (reported at
http://sourceforge.net/tracker/index.php?func=detail&aid=1638030&group_id=107955&atid=649116)
and a workaround for an Apple bug that (to a large extent) breaks Java
applet garbage-collection in Java 6.0 and on Leopard.

Comment 4

[Jay Patel [:jay]]

Sorry, it is a bit late for 1.8.1.2/1.8.0.10, but let's try to get this ready for next time.  

Comment 6

[James Bengard]

I was going to file a bug, but I am not sure if this issue is relative to the problem.  I have noticed with the latest release of firefox, and I admit I haven't noticed whether leads to it, but the minimize button initially works, then just the right very outer circle can be clicked to minimize.  I didn't have this problem in the previous release, or at least I didn't notice it.  I am using a new Mac Pro, with the latest patches and OS X.

Comment 7

[Steven Michaud [:smichaud] (Retired)]

> then just the right very outer circle can be clicked to minimize

I don't understand what you mean by this.

Comment 8

[Daniel Veditz [:dveditz]]

Maybe I live in a different world, but why is the inability to minimize an app an "urgent" issue or a "blocker"? Especially if you guys didn't even notice for a while. Can't we just wait until the next time we need a new JEP for a more critical issue?

Comment 10

[Steven Michaud [:smichaud] (Retired)]

I've just released a new version (0.9.6.1) of the Java Embedding
Plugin that fixes this problem.  For more information see bug 376395.

Comment 11

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Since bug 376395 landed, this should be FIXED, right?

Comment 12

[Steven Michaud [:smichaud] (Retired)]

Yes.

Comment 14

[Daniel Veditz [:dveditz]]

fixed on branch by bug 376395