Created attachment 401561 [details] [diff] [review] v1.0 This will make writing certain native code tests much easier. Having each test have to make a profile directory is painful. We already do this for free in our other test harnesses, so we should here too.
Comment on attachment 401561 [details] [diff] [review] v1.0 >diff --git a/xpcom/tests/TestHarness.h b/xpcom/tests/TestHarness.h >-static PRBool >+PRBool > StartProfiling(const char* profileName) This was in some other bug. Please make these inline as you remove the static.
Addressed that comment before I pushed. http://hg.mozilla.org/mozilla-central/rev/467f14a11325
Backed out for possible breakage.
Created attachment 403939 [details] [diff] [review] v1.1 Need to also implement GetFiles for TestRegistrationOrder to pass. Not sure how make check worked without this before...
Not sure if this needs to be documented, but I'll leave that up to sheppy.
From what I gather, all this is doing is making it possible for tests to have a profile directory automatically created and destroyed? Or am I misunderstanding the patch here?
(In reply to comment #7) > From what I gather, all this is doing is making it possible for tests to have a > profile directory automatically created and destroyed? Or am I misunderstanding > the patch here? that is it
After looking at this for a while, this doesn't look like it impacts any existing documentation. We do need more and better documentation for creating tests and whatnot, but that's beyond the scope of this particular bug, so I'm removing the doc needed flag here.