Remove "pippki" from embedding packages

RESOLVED FIXED in mozilla1.2beta



16 years ago
16 years ago


(Reporter: kaie, Assigned: kaie)



Firefox Tracking Flags

(Not tracked)



(1 attachment)



16 years ago
pippki is the portion of PSM that provides Mozilla's XUL UI for PSM.

The embedding packages include file pippki.[dll|so]
This should not be necessary, and it's presence actually can cause problems in
some embedding environments.

I suggest to remove that component from the embedding package.
Please object if you know a reason why this shouldn't be done.

Comment 1

16 years ago
cc'ing people who worked on bug 81884, since the pippki package was added with a
patch in that bug.


16 years ago
Priority: -- → P1
Target Milestone: --- → mozilla1.2beta

Comment 3

16 years ago
Adam, can you please review?

Comment 4

16 years ago
Comment on attachment 98028 [details] [diff] [review]
Patch v1

Does pippki only contain the front end and nothing else?

Assuming so, r=adamlock

Comment 5

16 years ago
Comment on attachment 98028 [details] [diff] [review]
Patch v1

Attachment #98028 - Flags: superreview+

Comment 6

16 years ago
fixed on trunk
Last Resolved: 16 years ago
Resolution: --- → FIXED
This seems to have caused bug 169943.

Comment 8

16 years ago
Sorry, but i have to reopen this bug.

pikpki.dll/xpt seem to be needed in certain embedding scenarios as described below. 

We have a certain embedding customer who uses the following code fragment to
display the Cert details using their own native UI:

void ShowCertDetailInfo()
  nsIASN1Object *asn1Obj;
  nsCOMPtr<nsIASN1Tree> asn1Tree = do_GetService(NS_ASN1TREE_CONTRACTID);
  /* Without the pippki.dll/xpt present |asn1tree| var will be null and we fail 
     to proceed with displaying cert details
  //Code Display asn1Obj here
Resolution: FIXED → ---

Comment 9

16 years ago
Adding javi to cc-list.


Should we move NS_ASN1TREE_CONTRACTID implementation into pipnss?

Comment 10

16 years ago
I disagree, unless there is a strong need for it.
David, do you have a need for that, too?

I will look at the code that Chak mentions and supply a way to not use the
implementation. I believe the current code does not really need it.

Comment 11

16 years ago
If the NS_ASN1TREE_CONTRACTID implementation is useful to embedders to write 
cert view code, then I think it should be part of the embedder kit. Of course, 
if there is an easier way for them to achieve the same result without have to 
move implementation and interfaces around, then that's fine too.

Comment 12

16 years ago

I think we should go ahead and put this into the pipnss embeders module simply
because it's useful enough that many other embeders may want to use it. 
Otherwise, every embeder that wants to populate a tree with cert details like
the Mozilla cert viewer will have to write something very similar, if not
exactly, like our nsIASN1Object implementation.


Comment 13

16 years ago
I think one of the arguments against doing that was, nsIASNTreeView is derived
from nsITreeView, which is XUL specific, and most embeddors are not using XUL.

Comment 14

16 years ago
Man this code keeps changing alot.  I thought we were talking about tne

I'm not so sure nsIASN1Tree should become a part of the default embeder module
unless nsITreeView is an interface embeders are free to use as well.

Comment 15

16 years ago
I believe I have the embedding that Chak uses in front of me.
I believe it is a bad idea to use nsIASN1Tree for that embedding code.
What is done is the embedding code abuses Mozilla's tree UI implementation as
its own data structure for mapping "index within ASN1Sequence" to "pointer to

The embedding code has to maintain it's own mapping from embedding UI code to
actual data objects anyway, and the embedding code is already doing that
partially. I see a easy way to get rid of abusing this private Mozilla completely.

I'm closing this bug again and will work with Chak and provide more info.
Last Resolved: 16 years ago16 years ago
Resolution: --- → FIXED

Comment 16

16 years ago
FYI: We forgot something, see new bug 188145.
You need to log in before you can comment on or make changes to this bug.