[FIX]about: protocol directory needs clean up

CLOSED FIXED in mozilla0.9.9

Status

SeaMonkey
General
P2
normal
CLOSED FIXED
16 years ago
13 years ago

People

(Reporter: Alexey Chernyak, Assigned: bz)

Tracking

Trunk
mozilla0.9.9

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

16 years ago
According to bug 68086, the following files should be removed from
mozilla/netwerk/protocol/about/src/

mzAboutMozilla.cpp
mzAboutMozilla.h
nsAboutCredits.cpp
nsAboutCredits.h
nsAboutPlugins.cpp
nsAboutPlugins.h

Some of these files are still referenced by nsAboutProtocolModule.cpp
However I could not find references to this file itself, so maybe it should be
removed as well, I'm not sure.
(Reporter)

Comment 1

16 years ago
CCing some people from bug 68086
Zach, would you be willing to make the mac build changes necessary for this, if
any?
Priority: -- → P2
Summary: about: protocol directory needs clean up → [FIX]about: protocol directory needs clean up
Target Milestone: --- → mozilla0.9.9
Created attachment 68086 [details] [diff] [review]
Patch removing the dead files.

Tested on Linux, I've asked Zach to test on Mac.

Comment 4

16 years ago
Tested on mac, all is ok
leaf?  The patch builds on Linux/Mac.  Visual inspection shows the files are not
referenced no Win.  Would you review?
Keywords: patch, review

Comment 6

16 years ago
Comment on attachment 68086 [details] [diff] [review]
Patch removing the dead files.

looks good, verified that about: urls still work without these files :)
Attachment #68086 - Flags: review+
Keywords: mozilla0.9.9+
Files removed on trunk (pre-branch, so this made 0.9.9 :) ).
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
(Reporter)

Comment 9

15 years ago
CLOSED
Status: RESOLVED → CLOSED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.