204184 - Remove unused items from wallet.cpp/wallet.h/singsign.cpp

Status: RESOLVED FIXED

(SeaMonkey :: Passwords & Permissions, defect)

Description

[timeless]

Comment 1

[timeless]

Attachment: patch

Comment 2

[[e-mail]]

Comment on attachment 122293 [details] [diff] [review]
patch

builds fine, warnings are gone now.

what the hell were static vars doing in a header anyway? ;)

Comment 3

[[e-mail]]

mvl also has a list of stuff he found was obsolete in wallet.

he promised to post it here. ;)

Comment 4

[Michiel van Leeuwen (email: mvl+moz@)]

Attachment: more cleanup

With this patch, a few files can be removed: (relative to extensions/wallet)

public/nsIKeyedStreamGenerator.idl
public/nsIPasswordSink.idl
src/nsWalletBuiltInDataSources.h
src/nsBasicStreamGenerator.cpp
src/nsBasicStreamGenerator.h
src/cookie.properties

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 122301 [details] [diff] [review]
more cleanup

Also requesting review for the list of files.

Comment 6

[Michiel van Leeuwen (email: mvl+moz@)]

src/URLFieldSchema.tbl is also unused.
Except for some deleteThisFile statements in installer files.
http://lxr.mozilla.org/seamonkey/search?string=URLFieldSchema
How long must they stay?

More unused files in cookieviewer:

cookieviewer/Makefile.in
cookieviewer/nsCookieViewer.cpp
cookieviewer/nsCookieViewer.h
cookieviewer/nsICookieViewer.idl
cookieviewer/CookieViewer.html

Comment 7

[[e-mail]]

hmm... cookieviewer sounds fine to remove, but not sure about the ones in
comment #4.

from cvs logs:

1.1	sspitzer%netscape.com	Jan 27 2000	 	adding this interfaces, they will be
used when we make the storing
of passwords in single signon "safer".

Seth, can you shed any light on this? is this stuff cruft now, or will it still
one day have some use? is this only relevant to wallet in its current state, or
will it still be relevant to a future (possibly rewritten) wallet?

thanks ;)

Comment 8

[[e-mail]]

Comment on attachment 122301 [details] [diff] [review]
more cleanup

hmm, since I can't seem to generate bugmail to Seth today... i'll just request
review ;)

Seth: please see comment #7.

Comment 9

[(not reading, please use [e-mail] instead)]

so, where is the work we do in nsBasicStreamGenerator::GetByte() being done now?

Comment 10

[Michiel van Leeuwen (email: mvl+moz@)]

It seems nsBasicStreamGenerator is not build at the moment. Nothing is doing
it's work, but nothing needs it.

Thinking of it: Was the use of it to stop handing out the passwords to
consumers, but let the consumers give something the StreamGenerator puts the
password in, so the password is send to the server? (I don't know enough of
necko to know how exactly...)
And was this ever really done? Some way of preventing everything with chrome
privs to get all passwords sounds not too bad :)

Comment 11

[[e-mail]]

i'll ask seth about this on aim sometime, but given that it was more than three
years ago i doubt we'll have much luck.

cc'ing mstoltz just in case... any idea about comment 7? specifically, any idea
why nsIKeyedStreamGenerator.idl was introduced into the tree, and whether it is
still relevant? it appears to have been introduced for a future endeavour, but
never was actually used. but if it might one day be relevant, we may want to
keep it.

in the meantime, let's whack cookieviewer.

Comment 12

[[e-mail]]

Attachment: remove cookieviewer cruft (checked in)

this patches /allmakefiles.sh to stop generation of the cookieviewer/ Makefile.

Comment 13

[[e-mail]]

Comment on attachment 124424 [details] [diff] [review]
remove cookieviewer cruft (checked in)

requesting r/sr for the removal of some cookieviewer cruft (none of this stuff
is built or used).

this also includes the cvs removal of the following files (not included in the
diff, since one of them is large):

extensions/wallet/src/URLFieldSchema.tbl
extensions/wallet/cookieviewer/Makefile.in
extensions/wallet/cookieviewer/nsCookieViewer.cpp
extensions/wallet/cookieviewer/nsCookieViewer.h
extensions/wallet/cookieviewer/nsICookieViewer.idl
extensions/wallet/cookieviewer/CookieViewer.html
extensions/wallet/signonviewer/SignonViewer.html
extensions/wallet/walletpreview/WalletPreview.html

the html files are being removed since I believe the xul/dtd/js fu replaces
them.

Comment 14

[[e-mail]]

for reference, applying the patch in comment 4 reduces libwallet.so by 2.1k (opt
linux build w/ stripping).

we should check there aren't any consumers of this stuff in commercial tree, or
by embeddors.

Comment 15

[[e-mail]]

Comment on attachment 124424 [details] [diff] [review]
remove cookieviewer cruft (checked in)

landed this one.

Comment 16

[Mitchell Stoltz (not reading bugmail)]

Regarding comment 11, I don't know what that was for. Maybe someone from the PSM
team remembers, CCing them.

Comment 17

[[e-mail]]

see bug 26020 (yes, that old) for cleaning up the duplicate files in wallet.

Comment 18

[[e-mail]]

Comment on attachment 122301 [details] [diff] [review]
more cleanup

r=dwitte. i'll switch the sr request over to dveditz since seth doesn't recall
the details, and dveditz is the owner.

Comment 19

[[e-mail]]

dveditz: i forgot to mention... the sr request also covers cvs removing the
related files mentioned in comment 4. (this patch removes some unused cruft from
wallet). thx!

Comment 20

[[e-mail]]

Comment on attachment 122301 [details] [diff] [review]
more cleanup

on second thought... bryner, care to sr this? please see comments 18 & 19.

Comment 21

[[e-mail]]

all checked in on trunk.