Closed
Bug 421857
Opened 17 years ago
Closed 16 years ago
Remove CocoaEmbed test app
Categories
(Core Graveyard :: Embedding: Mac, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.9.1b1
People
(Reporter: alqahira, Assigned: RyanVM)
References
Details
Attachments
(1 file)
|
59.49 KB,
patch
|
jaas
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
When the obsolete CocoaEmbed impl was removed last year in bug 399035, the CocoaEmbed test app in embedding/tests/cocoaEmbed got left behind. People are occasionally modifying or trying to modify those files, and they really just need to be removed.
|
Assignee | |
Comment 1•16 years ago
|
||
Too bad this was missed during the Hg transition. Smokey, you still planning to kill this?
|
Reporter | |
Comment 2•16 years ago
|
||
I don't currently have an Hg tree, so feel free to take it, Ryan.
|
|
Assignee | |
Comment 3•16 years ago
|
||
Smokey, if you're happy with this, feel free to check it in and close the bug.
|
|
Reporter | |
Comment 4•16 years ago
|
||
Comment on attachment 338443 [details] [diff] [review]
Remove tests/embedding/cocoaEmbed
[Checkin: Comment 5]
Josh should review (and land) this, not me ;)
Attachment #338443 -
Flags: review?(alqahira) → review?(joshmoz)
Attachment #338443 -
Flags: review?(joshmoz) → review+
Attachment #338443 -
Flags: superreview?(roc)
Attachment #338443 -
Flags: superreview?(roc) → superreview+
|
|
Assignee | |
Updated•16 years ago
|
Keywords: checkin-needed
|
||
Comment 5•16 years ago
|
||
Comment on attachment 338443 [details] [diff] [review]
Remove tests/embedding/cocoaEmbed
[Checkin: Comment 5]
http://hg.mozilla.org/mozilla-central/rev/ba2c1179f2b9
Attachment #338443 -
Attachment description: Remove tests/embedding/cocoaEmbed → Remove tests/embedding/cocoaEmbed
[Checkin: Comment 5]
|
|
||
Updated•16 years ago
|
Blocks: 399035
Severity: normal → trivial
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1b1
Version: unspecified → Trunk
|
||
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•