Closed
Bug 168452
Opened 22 years ago
Closed 22 years ago
PSM embedding freeze/ final step/ mark interfaces as frozen
Categories
(Core Graveyard :: Security: UI, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
psm2.4
People
(Reporter: KaiE, Assigned: KaiE)
References
Details
(Keywords: topembed+)
Attachments
(1 file)
(deleted),
patch
|
javi
:
review+
alecf
:
superreview+
sspitzer
:
approval1.4+
|
Details | Diff | Splinter Review |
Juan, if my list is correct, the following is the full list of interfaces we want to freeze. Please comment in the bug if the list is not complete. nsICRLInfo nsIX509Cert nsIX509CertDB nsIX509CertValidity nsIASN1Sequence nsIASN1Object nsICertificateDialogs nsIBadCertListener nsISecurityWarningDialogs
Assignee | ||
Updated•22 years ago
|
Assignee | ||
Updated•22 years ago
|
Assignee | ||
Comment 2•22 years ago
|
||
What we have to do is produce a simple patch that adds the appropriate comments that will mark the interfaces as frozen. We should do this for 1.3 final.
Keywords: mozilla1.3
Assignee | ||
Comment 3•22 years ago
|
||
Luckily we have not yet frozen the interfaces. I would like to make a reasonable change to nsIX509Cert. The change will not create additional work for embeddors, since nsIX509Cert is used by embeddors, but not implemented by them. Since a cert can store multiple email addresses, we are adding a way to obtain the list of all email addresses contained in the cert. Bug 50823 implements the change.
Comment 5•22 years ago
|
||
removing topembed nomination as this is a meta bug. Please nominate the individual bugs (although they all look to be fixed)
Keywords: topembed
Assignee | ||
Comment 6•22 years ago
|
||
Adding back nomination. It's not a meta bug, we need to produce a patch that marks the interfaces as frozen, as required by the embedding docs.
Keywords: topembed
Updated•22 years ago
|
Comment 7•22 years ago
|
||
Discussed in bug triage. Does the milestone relflect reality and when do we expect this to happen?
Assignee | ||
Comment 9•22 years ago
|
||
Assignee | ||
Comment 10•22 years ago
|
||
Comment on attachment 122567 [details] [diff] [review] Patch v1 This is to finalize the freeze process we started a while ago. No modifications to the interfaces happened since we talked about it (with the exception of the small addition in bug 50823, as previously announced). Can you please review?
Attachment #122567 -
Flags: superreview?(alecf)
Attachment #122567 -
Flags: review?(javi)
Comment 11•22 years ago
|
||
Comment on attachment 122567 [details] [diff] [review] Patch v1 r=javi
Attachment #122567 -
Flags: review?(javi) → review+
Comment 12•22 years ago
|
||
Comment on attachment 122567 [details] [diff] [review] Patch v1 sr=alecf
Attachment #122567 -
Flags: superreview?(alecf) → superreview+
Assignee | ||
Comment 13•22 years ago
|
||
Comment on attachment 122567 [details] [diff] [review] Patch v1 Requesting approval for freezing interfaces. This patch only adds the @status FROZEN comments.
Attachment #122567 -
Flags: approval1.4?
Comment 14•22 years ago
|
||
Comment on attachment 122567 [details] [diff] [review] Patch v1 a=sspitzer
Attachment #122567 -
Flags: approval1.4? → approval1.4+
Comment 15•22 years ago
|
||
Has this landed yet? I don't see anything in bonsai.
Whiteboard: landed1.4?
Assignee | ||
Comment 17•22 years ago
|
||
fixed on trunk
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Whiteboard: landed1.4?
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•