Last Comment Bug 772487 - Use CPP_UNIT_TESTS for TestPoisonArea.cpp
: Use CPP_UNIT_TESTS for TestPoisonArea.cpp
Product: Core
Classification: Components
Component: Layout (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla17
Assigned To: :Ms2ger
Depends on:
Blocks: nomakerules
  Show dependency treegraph
Reported: 2012-07-10 08:49 PDT by :Ms2ger
Modified: 2012-07-18 06:14 PDT (History)
1 user (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch v1 (1.22 KB, patch)
2012-07-10 08:49 PDT, :Ms2ger
khuey: review+
Details | Diff | Review
Patch v2 (2.07 KB, patch)
2012-07-14 02:03 PDT, :Ms2ger
khuey: review+
Details | Diff | Review

Description :Ms2ger 2012-07-10 08:49:54 PDT
Created attachment 640617 [details] [diff] [review]
Patch v1

The comment claims it doesn't work; tryserver thinks it does.
Comment 1 :Ms2ger 2012-07-11 09:54:31 PDT
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.
Comment 2 Kyle Huey [:khuey] ( (Away until 6/13) 2012-07-11 09:58:37 PDT
Ha, ok.  Please move this test into a subdirectory and add that subdirectory to TOOL_DIRS (like editor/txmgr/tests/).
Comment 3 :Ms2ger 2012-07-14 02:03:54 PDT
Created attachment 642194 [details] [diff] [review]
Patch v2

Alright, this should be better.

Note You need to log in before you can comment on or make changes to this bug.