70847 - LiveConnect uses jni.h that is incompatible with Mac OS Runtime for Java

Status: RESOLVED INCOMPLETE

(Core Graveyard :: Java: Live Connect, defect)

Description

[Patrick C. Beard]

I was doing some tests with LiveConnect and the XPConnect shell, and found that 
certain calls into JNI are causing crashes. I took a look at the access paths for 
the LiveConnect.mcp project and discovered that it is using the jni.h found in 
mozilla/sun-java/stubs/include/jni.h. Changing the access paths to point to the 
jni.h provided in {Compiler}Mac OS Support:JNIHeaders fixes the crash. We should 
definitely retire mozilla/sun-java as soon as possible.

Comment 1

[drapeau]

Patrick: would you please look at Bugzilla bug 15488 and note here whether you
think this bug is a duplicate of that one?

I also don't like the idea of having a jni.h in the mozilla repository; at
least, not the old one that's there now.

Comment 2

[Patrick C. Beard]

This is definitely the same bug. Removing it from the repository will fix this 
bug as well. I'll just make this bug dependent on that bug, so we can track both 
platforms. I can also work around this problem by changing an access path in 
LiveConnect.mcp, which I've done in my own tree.

Comment 3

[drapeau]

Yep, I'm sick of this problem as well.  I'll talk to Ed about taking action
on 15488.

Comment 4

[Patrick C. Beard]

Let's resolve this as soon as possible.

Comment 5

[Patrick C. Beard]

-> 0.9.2

Comment 6

[Patrick C. Beard]

--> mozilla0.9.3

Comment 7

[Blake Ross]

Doesn't look like this is getting fixed before the freeze tonight.
Pushing out a milestone.  Please correct if I'm mistaken.

Comment 8

[Patrick C. Beard]

--> 0.9.6

Comment 9

[Patrick C. Beard]

-> 1.0

Comment 10

[Patrick C. Beard]

This is also going to be a problem in the Mac OS X Java plugin, especially when
we get the mach-o build going. We need to remove the jni.h in our tree and only
build with the one appropriate for each platform.

Comment 11

[thebeastwitheyesthatstared]

what´s the current status on this, is this still an issue on OS9 & OSX?

Comment 12

[Simon Paquet [:sipaq]]

This bug is targeted at a Mac classic platform/OS, which is no longer supported
by mozilla.org. Please re-target it to another platform/OS if this bug applies
there as well or resolve this bug.

I will resolve this bug as WONTFIX in four weeks if no action has been taken.
To filter this and similar messages out, please filter for "mac_cla_reorg".

Comment 13

[Bob Clary [:bc] (inactive)]

-> default assignee for old netscape assigned bugs.

Comment 14

[Benjamin Smedberg]

Firefox code moved from custom Liveconnect code to the NPAPI/NPRuntime bridge a while back. Mass-closing the bugs in the liveconnect component which are likely invalid. If you believe that this bug is still relevant to modern versions of Firefox, please reopen it and move it the "Core" product, component "Plug-Ins".