Fix bug that meant that dependent tests were never reported as...

Fix bug that meant that dependent tests were never reported as
failing (though they could still crash).
(Cache the result of the test in the output object, not in the
test object which is reused from the prerequisite to the dependent.)

Committed: http://code.google.com/p/v8/source/detail?r=3115

[Kasper Lund, 2009-10-22 13:33:28]

LGTM.

http://codereview.chromium.org/321001/diff/1/2
File test/cctest/cctest.status (right):

http://codereview.chromium.org/321001/diff/1/2#newcode36
Line 36: # This is about to go away anyway.
Maybe elaborate (a bit) on why they are going away?

http://codereview.chromium.org/321001/diff/1/3
File test/cctest/test-serialize.cc (right):

http://codereview.chromium.org/321001/diff/1/3#newcode291
Line 291: extern "C" void V8_Fatal(const char* file, int line, const char*
format, ...);
Is that really necessary? You trust V8_Fatal over CHECK, I guess? It looks
pretty ugly.

[Tobias Kaes, 2009-10-23 09:11:40]

On 2009/10/22 13:33:28, Kasper Lund wrote:
> LGTM.
> 
> http://codereview.chromium.org/321001/diff/1/2
> File test/cctest/cctest.status (right):
> 
> http://codereview.chromium.org/321001/diff/1/2#newcode36
> Line 36: # This is about to go away anyway.
> Maybe elaborate (a bit) on why they are going away?
> 
> http://codereview.chromium.org/321001/diff/1/3
> File test/cctest/test-serialize.cc (right):
> 
> http://codereview.chromium.org/321001/diff/1/3#newcode291
> Line 291: extern "C" void V8_Fatal(const char* file, int line, const char*
> format, ...);
> Is that really necessary? You trust V8_Fatal over CHECK, I guess? It looks
> pretty ugly.

In the committed version (r3115) it still has the V8_Fatal import but doesn't
seem to use it ... probably could be removed?