Plug some memory leaks in parser tests.

Plug some memory leaks in parser tests.

BUG=v8:2763
R=dslomov@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=15490

[earthdok, 2013-07-04 13:45:53]

please take a look

[Michael Starzinger, 2013-07-04 15:06:29]

Dmitry kindly offered to take a look at this one.

[Dmitry Lomov (no reviews), 2013-07-04 15:22:29]

https://codereview.chromium.org/18694004/diff/1/test/cctest/test-parsing.cc
File test/cctest/test-parsing.cc (right):

https://codereview.chromium.org/18694004/diff/1/test/cctest/test-parsing.cc#n...
test/cctest/test-parsing.cc:1076: i::DeleteArray(message);
Not too happy about explicit deletions here - would be nice if BuildArgs and
BuildMessage returned some sort of scoped_ptr, but this out of the scope of this
CL.

https://codereview.chromium.org/18694004/diff/1/test/cctest/test-parsing.cc#n...
test/cctest/test-parsing.cc:1299: i::DeleteArray(program.start());
Replace Vector with ScopedVector here, then explicit deletion is not needed.

[earthdok, 2013-07-04 15:31:13]

https://codereview.chromium.org/18694004/diff/1/test/cctest/test-parsing.cc
File test/cctest/test-parsing.cc (right):

https://codereview.chromium.org/18694004/diff/1/test/cctest/test-parsing.cc#n...
test/cctest/test-parsing.cc:1299: i::DeleteArray(program.start());
On 2013/07/04 15:22:29, Dmitry Lomov (chromium) wrote:
> Replace Vector with ScopedVector here, then explicit deletion is not needed.

done

[Dmitry Lomov (no reviews), 2013-07-04 15:35:22]

lgtm

[earthdok, 2013-07-04 15:45:30]

On 2013/07/04 15:35:22, Dmitry Lomov (chromium) wrote:
> lgtm

I don't have committer access. Can someone please commit this for me?

[Dmitry Lomov (no reviews), 2013-07-04 15:46:18]

On 2013/07/04 15:45:30, earthdok wrote:
> On 2013/07/04 15:35:22, Dmitry Lomov (chromium) wrote:
> > lgtm
> 
> I don't have committer access. Can someone please commit this for me?

I'll land.

[Dmitry Lomov (no reviews), 2013-07-04 15:57:48]

Committed patchset #2 manually as r15490 (presubmit successful).