Location OF "failover.jsc" file in "Auto Config" feature

VERIFIED FIXED in mozilla0.9.2

Status

()

Core
Preferences: Backend
P2
normal
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: Mohan Bhamidipati, Assigned: Mitesh Shah)

Tracking

Trunk
mozilla0.9.2
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: waiting for a= for trunk checkin.)

Attachments

(2 attachments)

(Reporter)

Description

17 years ago
"failover.jsc" file is currently being created inside "Program" folder; It has
to be in user specific folder because this file is user specific.

Comment 1

17 years ago
assigning QA contact to luke.
QA Contact: sairuh → lrg
(Reporter)

Updated

17 years ago
Priority: -- → P2
Target Milestone: --- → mozilla0.9.2
(Reporter)

Comment 2

17 years ago
Created attachment 37200 [details] [diff] [review]
Proposed Patch - v1
(Reporter)

Comment 3

17 years ago
CC'ing Brian for review;

Comment 4

17 years ago
Looks right to me. r=bnesse.
(Reporter)

Comment 5

17 years ago
CC'ing Alec for super review;

Comment 6

17 years ago
sr=alecf
(Reporter)

Updated

17 years ago
Whiteboard: waiting for a= for trunk checkin.
a=blizzard on behalf of drivers for the trunk

Updated

17 years ago
Blocks: 83989
(Reporter)

Updated

17 years ago
Blocks: 79480
(Reporter)

Comment 8

17 years ago
reassigning to mitesh;
Assignee: mohanb → mitesh
(Assignee)

Comment 9

17 years ago
Since we are changing the location of failover.jsc to profile directory, we do 
not need to add 'Essential Files' folder for Mac. Attaching updated patch
Status: NEW → ASSIGNED
(Assignee)

Comment 10

17 years ago
Created attachment 37416 [details] [diff] [review]
updated patch - removed appending of 'Essential Files' for mac

Comment 11

17 years ago
Good catch Mitesh... my bad. r=bnesse on the new patch.

Comment 12

17 years ago
oh! good catch
sr=alecf
(Assignee)

Comment 13

17 years ago
code checked in 
(Assignee)

Comment 14

17 years ago
Tinderbox seems fine.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 15

17 years ago
I have verified this with 06/25 branch build and 06/26 trunk build on MacOS, 
Linux, and win32, makring Verified Fixed.

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