If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

CFG Activation broken on latest trunk builds

VERIFIED FIXED in mozilla0.9.8

Status

()

Core
Preferences: Backend
--
critical
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: Rodney Velasco, Assigned: Brian Nesse (gone))

Tracking

Trunk
mozilla0.9.8
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

16 years ago
Customized CFG will not load if activated in the all.js file.  Problem started
on the December 21, 2001 night builds.  Critical bug that affects AutoConfig and
preference locking.

Steps to Reproduce:
1. Download attached mozilla.cfg file to computer, download and name the file to
mozilla.cfg.

*hashed file that contains the following preferences*
lockPref("general.startup.browser", true ); /* Navigator */
lockPref("general.startup.mail", true); /* Mail */

2. Place mozilla.cfg file in the program directory containing the mozilla
executable (for mac place in the Essential Files Folder)
3. Add these lines to your all.js file located in the bin\defaults\pref directory.

pref("general.config.filename", "mozilla.cfg");
pref("general.config.vendor", "mozilla");

4. save all.js file and start Mozilla

Expected Result:
Mozilla should startup with Both the browser and mail client.  
Open General Prefences panel choose the 'Appearance' Category, in the Mozilla
should start up field:

1) Navigator checkbox should be checked and greyed out.  User should not be able
to modify checkbox.
2) Mail checkbox should be checked and greyed out.  User should not be able to
modify checkbox.

Actual result:
Browser starts up mail doesn't.  Both fields in preference panels can be
modified by user even though I specified a lockPref() for both preferences.

Might be a regression of bug 89137.
(Reporter)

Updated

16 years ago
QA Contact: sairuh → rvelasco
(Reporter)

Comment 1

16 years ago
Created attachment 64077 [details]
CFG file containing lockPref() functions to start browser and mail client.
(Reporter)

Comment 2

16 years ago
adding nsbeta1 keyword, very bad bug for us Customization Folks that require CFG
activation and AutoConfig for our testing.
Keywords: nsbeta1

Comment 3

16 years ago
Hi, Rodney:

Is this a QA blocker for us? My impression is that QA couldn't test
auto-config-ed features at all with the recen build?

Hi, Brian/Jud: any chance we treat this as a p0? Is there an ETA on this?

-thx
(Assignee)

Comment 4

16 years ago
Created attachment 64192 [details] [diff] [review]
Proposed patch.

There are 2 problems here. First, the windows build system apparently changed
out from under the patch in bug 89137 while it was waiting to land. Second,
none of the autoconfig stuff was ever added to the installer packages so
autoconfig only works in a debug build.
(Assignee)

Comment 5

16 years ago
I have also added the appropriate bits to the static build packages. The only 
thing I don't know for certain here is if the autoconfig module is built as part 
of the static build process.

Adding some folks I hope have some packaging knowledge to the cc list. Alec, JJ, 
Simon, can you guys r/sr these changes in attachment 64192 [details] [diff] [review] please?
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.8

Comment 6

16 years ago
Great. 

Brian - thanks for the quick response.
QA - would you notify Yumi to deliver a build when the patches checked in.

thx
(Reporter)

Comment 7

16 years ago
Tao,yes this is a blocker for QA. No way to test autoconfig and pref locking of
new preferences added to the trunk.  

Comment 8

16 years ago
Comment on attachment 64192 [details] [diff] [review]
Proposed patch.

sr=sfraser
Attachment #64192 - Flags: superreview+

Comment 9

16 years ago
Comment on attachment 64192 [details] [diff] [review]
Proposed patch.

sr=alecf
Attachment #64192 - Flags: review+
(Assignee)

Comment 10

16 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
(Reporter)

Comment 11

16 years ago
Tested CFG activation on

Linux: 2002012208 trunk
Win2K: 2002012203 trunk
Mac OSX: 2002012208 trunk

Marking bug verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.