Move passwordmgr/resources/content/* to passwordmgr/content/*

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: Dolske, Assigned: Dolske)

Tracking

Trunk
x86
macOS
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Assignee

Description

12 years ago
As part of the password manager thwacking, I'd like to have the component use a directory structure like other components. I'm basically replacing everything else under passwordmgr/, except for this one bit.

Existing structure:
-------------------
toolkit/components/passwordmgr/resources/Makefile.in
toolkit/components/passwordmgr/resources/jar.mn
toolkit/components/passwordmgr/resources/content/passwordManager.js
toolkit/components/passwordmgr/resources/content/passwordManager.xul
toolkit/components/passwordmgr/resources/content/passwordManagerCommon.js
toolkit/components/passwordmgr/resources/content/passwordManagerExceptions.js
toolkit/components/passwordmgr/resources/content/passwordManagerExceptions.xul

Move to:
--------
toolkit/components/passwordmgr/content/Makefile.in
toolkit/components/passwordmgr/jar.mn
toolkit/components/passwordmgr/content/passwordManager.js
toolkit/components/passwordmgr/content/passwordManager.xul
toolkit/components/passwordmgr/content/passwordManagerCommon.js
toolkit/components/passwordmgr/content/passwordManagerExceptions.js
toolkit/components/passwordmgr/content/passwordManagerExceptions.xul

A couple of files need tweaked to account for the path change, I'll attach a patch.
Assignee

Updated

12 years ago
Blocks: 374723
Assignee

Comment 1

12 years ago
This patch doesn't alter files being moved, so it can be applied either before or after the move.

Also, just to be clear, the movement of the above files should be performed as CVS copies to retain the version history.
Assignee: nobody → dolske
Status: NEW → ASSIGNED
Attachment #264772 - Flags: review?(gavin.sharp)
Assignee

Updated

12 years ago
Depends on: 380663
Attachment #264772 - Flags: review?(gavin.sharp) → review+
Checking in allmakefiles.sh;
/cvsroot/mozilla/allmakefiles.sh,v  <--  allmakefiles.sh
new revision: 1.662; previous revision: 1.661
done
Checking in toolkit/components/passwordmgr/Makefile.in;
/cvsroot/mozilla/toolkit/components/passwordmgr/Makefile.in,v  <--  Makefile.in
new revision: 1.5; previous revision: 1.4
done
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee

Comment 3

12 years ago
Reopening for a bit. Still need to remove the old directory tree [/toolkit/components/passwordmgr/resources], but will wait a bit to do that since other changes are going on. Less to back out in case of disaster. :-) Forgot to mention that to Colin.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
I checked in the removal of these old files, because I needed to convince one of the unit tests machine to build again.
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.