Bob Relyea has produced several patches. I have reviewed them, made suggestions to change them and applied my requested changes myself.

Original patch as created by Bob Relyea

Comment on attachment 203261 [details] [diff] [review] sorry - wrong file attached I'm fine with your changes in general, but I'm requesting some changes, as we have discussed by email.

Bob, I have worked on the changes that I had suggested. Here is the updated patch.

I would like to explain the code that we are adding. - provide a new interface nsIX509Cert2 to eliminate the need to cast to non-IDL pointer, make use of it in nsCMS and other places. - provide a new interface nsIX509CertList to deal with a list of certificates - clean up the certificate cache implemenentation, use the new cert list - enable the cert cache implementation to cache any given list of certificates - in nsCertTree, a data structure used by certificate manager, improve speed by caching all strings intended for display - to speed up, in nsNSSCertificate, only calculate the logical type of the assicoated NSS certificate once. Remove the not required ability to explicitly set the logical type. - add backend code that allows a cert manager user interface to display a cert type information column - for display purposes, when a cert has no isser org, fall back to common name The changes between Bob's patch v1 and my patch v2 are: - make nsNSSCertList not adopt the given list by default, to be consistent with our default behaviour "copy on construction". However, the possibility to use the optimized adopt technique is still available. - do not introduce nsISupportsString and nsTreeString, but make use of existing implementations - removed unnecessary nsCertTree::mNumColumns

*** Bug 299094 has been marked as a duplicate of this bug. ***

Patch checked in. Bob, should we have this on 1.8 branch for Firefox 2.0 ?

What is Camino based off of, the trunk or 1.8?

Comment on attachment 203263 [details] [diff] [review] Patch v2 This patch implements several requests from Camino developers. To answer Bob's question: Camino 1.x will be based on the 1.8 branch.

Comment on attachment 203263 [details] [diff] [review] Patch v2 looks like nsIX509CertDB2 needs a new IID...

Comment on attachment 203263 [details] [diff] [review] Patch v2 Biesi, thanks a lot for pointing this out. I just learned, that although we just add a method to nsIX509CertDB2, this does count as an API change and is not an appropriate change for 1.8 branch. As the camino people want this functionality on the 1.8 branch, we might consider reverting the change and introducing yet another DB interface?

Simon, if you really need or wants this on the 1.8 branch, please speak up.

(In reply to comment #12) > Simon, if you really need or wants this on the 1.8 branch, please speak up. I don't think we actually need any of this stuff on the 1.8 branch.

Bob, could you please review this one line patch?

Comment on attachment 208225 [details] [diff] [review] Additional patch to change uuid of nsIX509CertDB2 r+ relyea

new IID checked in

Comment on attachment 203263 [details] [diff] [review] Patch v2 not for 1.8 branch

this caused regression 352867