Make JsNative1JsNative2JsSample cctest work on Win Release

Make JsNative1JsNative2JsSample cctest work on Win Release

MSVC optimization realizes that CallJsFunction2 is just the same as CallJsFunction, so it eliminates the former making the call stack contain two instances of the same function.

The patch makes two functions distinct.

LOG=N
BUG=v8:3055
R=aandrey@chromium.org, jkummerow@chromium.org, yurys@chromium.org

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

[yurys, 2014-07-01 05:31:32]

lgtm

[aandrey, 2014-07-01 08:20:42]

lgtm

https://codereview.chromium.org/357383003/diff/1/test/cctest/cctest.status
File test/cctest/cctest.status (left):

https://codereview.chromium.org/357383003/diff/1/test/cctest/cctest.status#ol...
test/cctest/cctest.status:155: # BUG(3055).
BUG=3055 ?

[alph, 2014-07-01 09:16:03]

Michael, could you please do the owners review.

[alph, 2014-07-01 09:17:08]

https://codereview.chromium.org/357383003/diff/1/test/cctest/cctest.status
File test/cctest/cctest.status (left):

https://codereview.chromium.org/357383003/diff/1/test/cctest/cctest.status#ol...
test/cctest/cctest.status:155: # BUG(3055).
On 2014/07/01 08:20:42, aandrey wrote:
> BUG=3055 ?

Updated the description.

[Jakob Kummerow, 2014-07-01 09:46:19]

lgtm

[alph, 2014-07-01 10:10:22]

Committed patchset #2 manually as r22113 (tree was closed).

[Michael Achenbach, 2014-07-01 10:36:36]

Note that these trybot results are not showing anything. All CPU profiler test
are marked as flaky in line 76 on all platforms. The try servers are not running
tests marked as flaky.

[alph, 2014-07-01 15:09:10]

On 2014/07/01 10:36:36, Michael Achenbach wrote:
> Note that these trybot results are not showing anything. All CPU profiler test
> are marked as flaky in line 76 on all platforms. The try servers are not
running
> tests marked as flaky.

Oh, I see, thanks.
At least I can see it's fixed on the waterfall bots, e.g.
http://build.chromium.org/p/client.v8/builders/V8%20Win32%20-%201 in #2068