Beginning on October 25th, 2016, Persona will no longer be an option for authentication on BMO. For more details see Persona Deprecated.
Last Comment Bug 583566 - Remove support for plugin.scan.SunJRE
: Remove support for plugin.scan.SunJRE
: helpwanted
Product: Core
Classification: Components
Component: Plug-ins (show other bugs)
: 1.9.2 Branch
: x86 Windows XP
: P3 minor with 2 votes (vote)
: mozilla21
Assigned To: :Cykesiopka
: Benjamin Smedberg [:bsmedberg]
: 506719 (view as bug list)
Depends on:
  Show dependency treegraph
Reported: 2010-08-01 06:13 PDT by edggar
Modified: 2013-12-06 05:29 PST (History)
5 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Remove plugin.scan.SunJRE pref and associated code (7.17 KB, patch)
2013-02-15 23:24 PST, :Cykesiopka
benjamin: review+
Details | Diff | Splinter Review
Remove old Java plugin scanning pref and related code (7.19 KB, patch)
2013-02-16 19:19 PST, :Cykesiopka
no flags Details | Diff | Splinter Review

Description edggar 2010-08-01 06:13:20 PDT
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv: Gecko/20100722 Firefox/3.6.8
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv: Gecko/20100722 Firefox/3.6.8

Updating from to the Java plugin now shows up under plugins despite having plugin.scan.SunJRE set to "99.0".

Reproducible: Always

Steps to Reproduce:
1.Set plugin.scan.SunJRE to a high value like 99.0
2.Install Java
Actual Results:  
The Java plugin is being listed.

Expected Results:  
Java plugin should not be listed under plugins.

I am seeing the same thing with Thunderbird/3.1.1 Gecko/20100713 but it should affect all builds of Firefox and Thunderbird using said Java version.
Comment 1 Matthias Versen [:Matti] 2010-08-01 08:51:00 PDT
Oracle changed the vendor strings with from Sun to Oracle
Comment 2 edggar 2010-09-08 14:49:34 PDT
Fixed for me with rv: Gecko/20100824 Firefox/3.6.9.
Comment 3 Matthias Versen [:Matti] 2010-09-08 17:09:58 PDT
wfm without a patch
Comment 4 edggar 2010-10-13 01:59:02 PDT
And the problem is back with the latest Java
Comment 5 Serge Gautherie (:sgautherie) 2012-03-22 22:05:38 PDT
Should setting plugin.scan.SunJRE to "99.0" through about:config then restarting be enough to reproduce?
If so, I can reproduce with a SeaMonkey 2.10a1(pre) build.
Comment 6 Benjamin Smedberg [:bsmedberg] 2013-02-12 10:56:11 PST
We should remove support for this preference. All the current+secure versions of Java put their registry keys correctly into HKLM/Software/MozillaPlugins.
Comment 7 :Cykesiopka 2013-02-15 23:24:53 PST
Created attachment 714757 [details] [diff] [review]
Remove plugin.scan.SunJRE pref and associated code

Hi, I was familiarising myself with Mercurial and patches, and I thought I would try and fix this bug...

Sorry if the patch doesn't so the correct thing...
Comment 8 Benjamin Smedberg [:bsmedberg] 2013-02-16 06:18:29 PST
Comment on attachment 714757 [details] [diff] [review]
Remove plugin.scan.SunJRE pref and associated code

This look perfect. Normally you'd set the "review" "?" "revieweremail" flag, but since I'm the correct reviewer I'll just mark it. I don't think a try run will test anything useful since we don't have tests for out-of-data Java.

Can you upload a final patch for checkin that includes a slightly better commit message (one mentioning Java at least) and includes "r=bsmedberg". After you've done that, add the 'checkin-needed' keyword to the bug.
Comment 9 :Cykesiopka 2013-02-16 19:19:08 PST
Created attachment 714860 [details] [diff] [review]
Remove old Java plugin scanning pref and related code

Updated patch with better commit message
Comment 10 Ryan VanderMeulen [:RyanVM] 2013-02-17 04:00:09 PST
Comment on attachment 714860 [details] [diff] [review]
Remove old Java plugin scanning pref and related code

You don't need to request another review on this.
Comment 11 Ryan VanderMeulen [:RyanVM] 2013-02-17 04:11:06 PST
Comment 12 :Cykesiopka 2013-02-17 13:15:16 PST
Ok, sorry about that...
Comment 13 Ryan VanderMeulen [:RyanVM] 2013-02-17 14:55:47 PST
Comment 14 Michael Lefevre 2013-02-18 12:53:26 PST
(In reply to cykesiopka from comment #12)
> Ok, sorry about that...

No need to apologise - just something to know for the future.

Thanks for the patch.
Comment 15 :Cykesiopka 2013-12-06 05:29:31 PST
*** Bug 506719 has been marked as a duplicate of this bug. ***

Note You need to log in before you can comment on or make changes to this bug.