14660 - javascript.enabled not set

Status: VERIFIED FIXED

(SeaMonkey :: Preferences, defect, P1)

Description

[Norris Boyd]

Under "Edit/Preferences..." select "Advanced". Toggling checkboxes for "Enable
JavaScript" doesn't change any setting on the preference "javascript.enabled".

(Also should check to see that "javascript.allow.mailnews" is handled properly
as well.)

Comment 1

[shuang (gone)]

mcafee, would you like to have this bug?

Comment 2

[Chris McAfee]

Enabling javascript sets "javascript.allow", not "javascript.enabled".
Still a bug?

Comment 3

[Chris McAfee]

We should be using javascript.enabled, norris says.
Fix in hand, awaiting approval.

Comment 4

[Chris McAfee]

Fixed.  Turning JS off should now write out javascript.enabled as "false",
it was writing out javascript.allow as "false" which was incorrect.

Comment 5

[cpratt]

In the 1999111016 build under NT, unticking the boxes for Enable JavaScript in
the Advanced prefs seems to do nothing. When I view the contents of the all.js
files on my hard drive, they all say:

pref("javascript.enabled",                  true);
pref("javascript.allow.mailnews",           true);

When I restart the browser and look at the Advanced pane again, the boxes remain
unticked. So... this doesn't seem as if it's been fixed. Reopening.

(In the mailnews.js file, it also says
pref("mailnews.javascript_enabled",         true);
by the way.)

Comment 6

[chris hofmann]

moving off to m12 for more investigation

Comment 7

[Chris McAfee]

worksforme on linux, over to matt to test NT.

Comment 8

[matt]

works for me

user_pref("javascript.allow.mailnews", false);
user_pref("javascript.enabled", false);

Comment 9

[Norris Boyd]

clear status whiteboard

Comment 10

[leger]

Bulk move of all Pref UI component bugs to new Preferences component.  Pref UI 
component will be deleted.

Comment 11

[Tom Everingham]

over to sarah for QA

Comment 12

[sairuh (rarely reading bugmail)]

verif on linux, winNT and mac w/opt comm bits [2000.03.07.09].