Add tests for FuncNameInferrer.

Add tests for FuncNameInferrer.

There were no explicit tests for FuncNameInferrer (though, some tests might
implicitly rely on it working properly in the common cases). Afaics, there were
no tests which would exercise inferring two byte function names or escaped
function names.

R=rossberg@chromium.org
BUG=

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

[marja, 2014-06-03 13:31:30]

rossberg, ptal.

This is just adding tests, since FuncNameInferring didn't look like well tested
territory (esp the corner cases) and I want to make sure my "delay string /
value internalization" doesn't break it.

[rossberg, 2014-06-03 13:48:46]

LGTM with comments

https://codereview.chromium.org/310063002/diff/1/test/cctest/cctest.h
File test/cctest/cctest.h (right):

https://codereview.chromium.org/310063002/diff/1/test/cctest/cctest.h#newcode451
test/cctest/cctest.h:451: CHECK(result->Equals(expected));
Not your CL really, but shouldn't this use SameValue, or at least StrictEquals?

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

https://codereview.chromium.org/310063002/diff/1/test/cctest/test-parsing.cc#...
test/cctest/test-parsing.cc:2474: "function not_lazy_not_ctor() { "
How does this differ from the previous?

https://codereview.chromium.org/310063002/diff/1/test/cctest/test-parsing.cc#...
test/cctest/test-parsing.cc:2496: ExpectString("Ctor()", "Ctor.foo5");
Wow...

[marja, 2014-06-03 14:16:35]

thanks for review!

https://codereview.chromium.org/310063002/diff/1/test/cctest/cctest.h
File test/cctest/cctest.h (right):

https://codereview.chromium.org/310063002/diff/1/test/cctest/cctest.h#newcode451
test/cctest/cctest.h:451: CHECK(result->Equals(expected));
On 2014/06/03 13:48:46, rossberg wrote:
> Not your CL really, but shouldn't this use SameValue, or at least
StrictEquals?

Changed this to use SameValue.

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

https://codereview.chromium.org/310063002/diff/1/test/cctest/test-parsing.cc#...
test/cctest/test-parsing.cc:2474: "function not_lazy_not_ctor() { "
On 2014/06/03 13:48:46, rossberg wrote:
> How does this differ from the previous?

Oops yes, this was supposed to be forcibly non-lazy (function foo(){}())();
*but* I cannot anyway make lazy functions work in this test case (laziness is
not allowed if natives syntax is allowed), so I removed this case.

[marja, 2014-06-03 15:02:27]

Committed patchset #3 manually as r21649 (presubmit successful).