Closed
Bug 485984
Opened 16 years ago
Closed 16 years ago
remove OJI from the tree
Categories
(Core Graveyard :: Java: OJI, defect)
Core Graveyard
Java: OJI
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.9.2a1
People
(Reporter: jaas, Assigned: jaas)
References
(Blocks 1 open bug)
Details
Attachments
(2 files, 2 obsolete files)
(deleted),
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
benjamin
:
review-
|
Details | Diff | Splinter Review |
We should remove OJI from the tree, as of Sun JRE 6 Update 10 for Windows and Linux it is not necessary (new NPAPI/npruntime Java plugin), and a new Mac OS X Java plugin is on the way.
Flags: wanted1.9.2+
There is a removal patch URL in the URL field for this bug. The patch is ~5.1 MB, too big for posting on bugzilla. This patch builds on Windows, Linux, and Mac OS X, and Java works on Windows and Linux with > JRE 6 Update 10.
Review placeholder attachment, contains URL of actual patch.
Attachment #370269 -
Flags: superreview?(jst)
Comment 3•16 years ago
|
||
(In reply to comment #1)
> There is a removal patch URL in the URL field for this bug. The patch is ~5.1
> MB, too big for posting on bugzilla. This patch builds on Windows, Linux, and
> Mac OS X, and Java works on Windows and Linux with > JRE 6 Update 10.
"Working" might not be completely correct. Kenneth Russel inserted a patch into JRE 6 Update 12 to solve a permission problem that Firefox extension that use Java had with plugin2.
Somebody should check that jar loading actually works without OJI.
The code that worked for years http://simile.mit.edu/wiki/Java_Firefox_Extension does not work with plugin2 (that is without OJI). The example code here https://developer.mozilla.org/en/Java_in_Firefox_Extensions does not work either without OJI.
Comment 4•16 years ago
|
||
The following forum thread indicates that with slight syntactic changes it appears possible to use Java within Firefox extensions using the new Java Plug-In.
http://forums.java.net/jive/thread.jspa?threadID=45933&tstart=0
Comment 5•16 years ago
|
||
Comment on attachment 370269 [details]
fix v1.0
Josh, with the attached patch we appear to be loading the wrong Java plugin on Windows. With a clean mozilla-central tree we list npjp2.dll as the dll we load for Java, and with this patch, I we list npjpi160_10.dll for Java. That should be npjp2.dll in both cases (or alternatively we could use npjpi160_10.dll in a clean mozilla-central tree if the old java plugin is chosen by the user in the Java console). The reason for that looks to be the changes to nsPluginDirServiceProvider.cpp, where you remove the code relating to the function TryToUseNPRuntimeJavaPlugIn(). We need to leave enough of that code in so that the default in our code is to use the new plugin, unconditionally, rather than just removing the code. Actually, we really need to reverse the code there, and in nsPluginHost.cpp, so that we simply refuse to load and/or use the old Java plugin completely.
Besides that, this patch looks good from what I've seen so far, but I still need to spend a bit more time reading through it some more.
Attachment #370269 -
Flags: superreview?(jst) → superreview-
Windows is the one platform I didn't test on myself, I had someone make sure my patch compiled and that Java still worked afterwards. My assumption was that without OJI the old Java plugin couldn't work, thus if Java worked it would be the new plugin working. The person claimed that Java did work with my patch. I'll look into it again.
Comment 7•16 years ago
|
||
Yeah, the old Java plugin will happily work w/o OJI and LiveConnect, you just can't talk to it from JS (or the other way around).
Comment 8•16 years ago
|
||
(In reply to comment #6)
> Windows is the one platform I didn't test on myself, I had someone make sure my
> patch compiled and that Java still worked afterwards. My assumption was that
> without OJI the old Java plugin couldn't work, thus if Java worked it would be
> the new plugin working. The person claimed that Java did work with my patch.
> I'll look into it again.
Please test with some Firefox extensions that use Java too.
fix logic that picks old vs. new plugin on Windows
Attachment #370269 -
Attachment is obsolete: true
Attachment #371564 -
Flags: superreview?(jst)
Assignee | ||
Comment 10•16 years ago
|
||
Add back some AIX code, remove nsJVMAuthTools completely.
Attachment #371564 -
Attachment is obsolete: true
Attachment #371984 -
Flags: superreview?(jst)
Attachment #371564 -
Flags: superreview?(jst)
Updated•16 years ago
|
Attachment #371984 -
Flags: superreview?(jst)
Attachment #371984 -
Flags: superreview+
Attachment #371984 -
Flags: review+
Comment 11•16 years ago
|
||
Comment on attachment 371984 [details] [diff] [review]
fix v1.2
Looks good. I think we'll want to have explicit code for some time to explicitly refuse to use the old Java plugin, but we can add that once we completely remove support for XPCOM plugins.
Assignee | ||
Comment 12•16 years ago
|
||
pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/88e7b02b100a
http://hg.mozilla.org/mozilla-central/rev/1fc65dc2a0c2
Assignee | ||
Comment 13•16 years ago
|
||
another bustage fix for static builds pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/3051d82cb64f
Assignee | ||
Comment 14•16 years ago
|
||
I found some more oji references, I assume we can remove them but I don't know what this is.
Attachment #372015 -
Flags: review?(benjamin)
Updated•16 years ago
|
Blocks: Win16Removal
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Target Milestone: --- → mozilla1.9.2a1
Updated•16 years ago
|
Attachment #372015 -
Flags: review?(benjamin) → review-
Comment 15•16 years ago
|
||
Comment on attachment 372015 [details] [diff] [review]
gre-win fixes, v1.0
Don't bother, this file is older than dead. (We should probably just remove all of embedding/config, I'll file a bug on that.)
Comment 16•16 years ago
|
||
This change made it impossible to run crashtest for me; see bug 487842.
Comment 17•16 years ago
|
||
verified ...FIXED?
Minefield does allow me to load: http://www.time.gov/timezone.cgi?Eastern/d/-5/java
on Build: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090505 Minefield/3.6a1pre ID:20090505030940
Status: RESOLVED → VERIFIED
Comment 18•16 years ago
|
||
That last comment is improper, Java is removed and the browsers shows the "Click here to download plugin" message as well as the popdown when the site is visited.
Updated•15 years ago
|
Blocks: C191ConfSync
Comment 19•15 years ago
|
||
Comment on attachment 371984 [details] [diff] [review]
fix v1.2
>http://joshaas.net/mozilla/oji9.patch
How do you expect anyone to be able to comment on this?
Updated•15 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•