Java isn't properly disabled in manual installs

RESOLVED FIXED in Camino2.1

Status

Camino Graveyard
Plug-ins
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Smokey Ardisson (offline for a while; not following bugs - do not email), Assigned: Stuart Morgan)

Tracking

1.9.2 Branch
Camino2.1
All
Mac OS X
Bug Flags:
camino2.1b1 +

Details

Attachments

(1 attachment)

Contrary to our belief after timestamp-checking (bug 625220 comment 7), manual installs of 2.1a1 do not actually disable Java.

In my case, the checkbox was unchecked, but Java was still active (checking the checkbox did not kill the active applet, so at least for me there were no sync issues like with offline mode).  I believe philippe had a slightly different experience.

[01:25am] smorgan: So manual updates don't work. I wonder why
[01:26am] smorgan: I guess file a bug; I can add some prodding code for b1
[01:26am] smorgan: We can have PreferenceManager do an explicit check once
Flags: camino2.1b1?

Comment 1

7 years ago
(In reply to comment #0)
> I believe philippe had a slightly different
> experience.

I had the same experience actually. The new pref didn't kick in until I manually checked the box ON (there was a slight flicker on the applet while doing so) and then OFF (and that killed the applet).
(Assignee)

Updated

7 years ago
Assignee: nobody → stuart.morgan+bugzilla
Flags: camino2.1b1? → camino2.1b1+
Target Milestone: --- → Camino2.1
(Assignee)

Comment 2

7 years ago
Created attachment 512700 [details] [diff] [review]
fix

Does a one-time resync of the Gecko state to the prefs.

Also removes our oldest pref migration, since it dates back at least to early 2007, and if someone hasn't updated in the last four years, they probably aren't going to.
Attachment #512700 - Flags: superreview?(mikepinkerton)
(Might we want to hold off on checking this in until bug 633684 and bug 632278 are ready, if pink's sr beats my having those ready, so that we don't have to rev the prefs version from 2 to 6 in the course of Camino 2.1 development?)
(Assignee)

Comment 4

7 years ago
Comment on attachment 512700 [details] [diff] [review]
fix

Removing sr request since this one is trivial.
Attachment #512700 - Flags: superreview?(mikepinkerton)
(Assignee)

Comment 5

7 years ago
Landed http://hg.mozilla.org/camino/rev/94b0aaef1504
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.