Closed
Bug 121235
Opened 23 years ago
Closed 23 years ago
Freeze nsIDirectoryServiceProvider2
Categories
(Core Graveyard :: Embedding: APIs, defect)
Core Graveyard
Embedding: APIs
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.9
People
(Reporter: ccarlen, Assigned: ccarlen)
References
()
Details
Attachments
(1 file)
708 bytes,
patch
|
dougt
:
review+
jband_mozilla
:
superreview+
|
Details | Diff | Splinter Review |
Summary says it all.
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.9
Updated•23 years ago
|
QA Contact: mdunn → depstein
Comment 1•23 years ago
|
||
changed qa contact to Ashish. nsIDirectoryService is his area.
QA Contact: depstein → ashishbhatt
Comment 2•23 years ago
|
||
conrad, are their there any known issues?
Assignee | ||
Comment 3•23 years ago
|
||
No. nsIDirectoryServiceProvider2 works well for returning multiple locations,
and nsIDirectoryService for single locations. I can't think of any more we need
out of providers.
Assignee | ||
Comment 4•23 years ago
|
||
Ready to freeze.
Doug, if you agree, can you give it an r=?
Comment 5•23 years ago
|
||
Comment on attachment 66446 [details] [diff] [review]
Changes status to frozen
I am happy with freezing this interface.
Attachment #66446 -
Flags: review+
Assignee | ||
Comment 6•23 years ago
|
||
jband, since you've used nsIDirectoryServiceProvider2, do you think it's
freezable (and deserving an sr= here)?
Comment 7•23 years ago
|
||
I'm Ok with this. But isn't freezing a commitee decision?
Comment 8•23 years ago
|
||
I think that there was a group of people including me, valeski, rick, conrad,
and a few others that suggested this approach and defined the interface. Is
there a official commitee that still meets that decides the fate of interface
status. I didn't think so....
Comment 9•23 years ago
|
||
Comment on attachment 66446 [details] [diff] [review]
Changes status to frozen
rs=jband. OK, if this is already approved then here's my rubberstamp.
Attachment #66446 -
Flags: superreview+
Assignee | ||
Comment 10•23 years ago
|
||
Checked in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•