Closed
Bug 166608
Opened 23 years ago
Closed 23 years ago
Add xmlextras to embedding manifests
Categories
(Core Graveyard :: Embedding: APIs, defect, P1)
Core Graveyard
Embedding: APIs
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.0.2
People
(Reporter: adamlock, Assigned: adamlock)
References
Details
(Keywords: topembed+, Whiteboard: [adt2] [ETA 09/12])
Attachments
(2 files)
|
2.99 KB,
patch
|
ccarlen
:
review+
hjtoi-bugzilla
:
superreview+
|
Details | Diff | Splinter Review |
|
3.11 KB,
patch
|
ccarlen
:
review+
hjtoi-bugzilla
:
superreview+
chofmann
:
approval+
|
Details | Diff | Splinter Review |
Useful functionality for embedders to have
See related bug: http://bugzilla.mozilla.org/show_bug.cgi?id=143342
*** Bug 143342 has been marked as a duplicate of this bug. ***
branch checkin approval, a=valeski@netscape.com
Keywords: edt1.0.2+
|
||
Updated•23 years ago
|
Keywords: mozilla1.2
Priority: -- → P1
Whiteboard: [adt2] [ETA 09/12]
Target Milestone: --- → mozilla1.0.2
Heikki can you give me an sr on this so I can get it into the trunk?
The readme.html already has a note about how to remove components by writing a
client-win file, but I can add something that specifically mentions XMLExtras.
Priority: P1 → --
Target Milestone: mozilla1.0.2 → ---
|
|
||
Updated•23 years ago
|
Fixed checked in on branch
Comment on attachment 97783 [details] [diff] [review]
Patch to embedding manifests
sr=heikki
Attachment #97783 -
Flags: superreview+
Comment on attachment 98433 [details] [diff] [review]
Same patch against 1.0.x branch
sr=heikki
Attachment #98433 -
Flags: superreview+
|
|
||
Comment 10•23 years ago
|
||
Comment on attachment 97783 [details] [diff] [review]
Patch to embedding manifests
r=ccarlen. I would add a comment to this item that it's optional, not required.
Attachment #97783 -
Flags: review+
|
|
||
Comment 11•23 years ago
|
||
Comment on attachment 98433 [details] [diff] [review]
Same patch against 1.0.x branch
r=ccarlen. I would add a comment to this item that it's optional, not required.
Attachment #98433 -
Flags: review+
|
||
Comment 12•23 years ago
|
||
Comment on attachment 98433 [details] [diff] [review]
Same patch against 1.0.x branch
a=chofmann for 1.0.2
Attachment #98433 -
Flags: approval+
|
|
Assignee | |
Comment 13•23 years ago
|
||
Fix checked into trunk, with a comment in the manifests and readme.html saying
xmlextras is optional
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
|
|
||
Comment 14•23 years ago
|
||
Ashish please verify this on the branch and change keyword to verified1.0.2
QA Contact: mdunn → ashishbhatt
|
|
||
Comment 15•23 years ago
|
||
Marking as verified
Status: RESOLVED → VERIFIED
Keywords: fixed1.0.2 → verified1.0.2
|
||
Updated•7 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•