The default bug view has changed. See this FAQ.

Use CPP_UNIT_TESTS for TestPoisonArea.cpp

RESOLVED FIXED in mozilla17

Status

()

Core
Layout
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

(Blocks: 1 bug)

Trunk
mozilla17
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Created attachment 640617 [details] [diff] [review]
Patch v1

The comment claims it doesn't work; tryserver thinks it does.

https://tbpl.mozilla.org/?tree=Try&rev=9be04254021c
Attachment #640617 - Flags: review?(khuey)
(Assignee)

Updated

5 years ago
Blocks: 769378
Attachment #640617 - Flags: review?(khuey) → review+
(Assignee)

Comment 1

5 years ago
So I made sure to do a try run... Only it turns out that I sent an empty patch to try and the comment is correct after all.
Ha, ok.  Please move this test into a subdirectory and add that subdirectory to TOOL_DIRS (like editor/txmgr/tests/).
(Assignee)

Comment 3

5 years ago
Created attachment 642194 [details] [diff] [review]
Patch v2

Alright, this should be better.
Attachment #640617 - Attachment is obsolete: true
Attachment #642194 - Flags: review?(khuey)
Attachment #642194 - Flags: review?(khuey) → review+
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/mozilla-central/rev/1968d3930f17
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.