Navigation.prefPane's Info.plist has the wrong NSPrincipalClass

RESOLVED FIXED in Camino1.5

Status

defect
RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: stuart.morgan+bugzilla, Assigned: stuart.morgan+bugzilla)

Tracking

({fixed1.8.0.9, fixed1.8.1})

Bug Flags:
camino1.0.4 +

Details

Attachments

(1 attachment)

(Assignee)

Description

13 years ago
Somehow Navigation's Info.plist missed the boat when names were upgraded; the NSPrincipalClass entry lists OrgMozillaNavigatorPreferenceNavigation, but the actual class is now OrgMozillaChimeraPreferenceNavigation.  The fact that it still loads now that it includes more than one class (due to RSS) is probably just luck.
Target Milestone: --- → Camino1.1
(Assignee)

Comment 1

13 years ago
Posted patch fixSplinter Review
Assignee: nobody → stuart.morgan
Status: NEW → ASSIGNED
Attachment #241767 - Flags: superreview?(mikepinkerton)
Comment on attachment 241767 [details] [diff] [review]
fix

rs=pink

shouldn't we change all these to Camino?
Attachment #241767 - Flags: superreview?(mikepinkerton) → superreview+
Checked in on 1.8branch and trunk.  Flagging 1.0.4? for smorgan - it's a very safe stability fix, so we should consider it for 1.0.x.

Comment 2 is now filed as bug 356204.
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Flags: camino1.0.4?
Resolution: --- → FIXED
Whiteboard: [needs checkin]
Flags: camino1.0.4? → camino1.0.4+
Whiteboard: [needs checkin 180 branch]
Checked in on 1.8.0branch for 1.0.4.
Whiteboard: [needs checkin 180 branch]
You need to log in before you can comment on or make changes to this bug.