[es6] Bound function names

[es6] Bound function names

https://people.mozilla.org/~jorendorff/es6-draft.html#sec-function.prototype.bind

Bound functions should have a name based on the function that was
bound.

This reverts the revert f2747ed9b48d0e62c7a30da69825ff926aeedbd2. The original
CL was reverted because the Blink layout test broke. I have a CL that disables
these tests at: https://codereview.chromium.org/1196753003/

BUG=N
LOG=N
R=adamk
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel

Committed: https://crrev.com/b6d950c979f4348138de0ec54e40dcc48d833926
Cr-Commit-Position: refs/heads/master@{#29193}

[arv (Not doing code reviews), 2015-06-19 22:13:15]

PTAL

[adamk, 2015-06-19 22:17:16]

lgtm

Can you say a little bit more in the CL description about why you're reverting
the revert?

[arv (Not doing code reviews), 2015-06-19 22:40:50]

On 2015/06/19 22:17:16, adamk wrote:
> lgtm
> 
> Can you say a little bit more in the CL description about why you're reverting
> the revert?

Done

[adamk, 2015-06-19 22:41:50]

Thanks, still lgtm

[arv (Not doing code reviews), 2015-06-19 22:42:20]

The CQ bit was checked by arv@chromium.org

[commit-bot: I haz the power, 2015-06-19 22:42:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1195983002/1

[commit-bot: I haz the power, 2015-06-19 22:54:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-19 22:54:35]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/6...)

[arv (Not doing code reviews), 2015-06-22 13:47:49]

The CQ bit was checked by arv@chromium.org

[commit-bot: I haz the power, 2015-06-22 13:47:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1195983002/1

[commit-bot: I haz the power, 2015-06-22 14:29:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-06-22 14:30:11]

Patchset 1 (id:??) landed as
https://crrev.com/b6d950c979f4348138de0ec54e40dcc48d833926
Cr-Commit-Position: refs/heads/master@{#29193}

[Toon Verwaest, 2015-07-06 15:23:00]

verwaest@chromium.org changed reviewers:
+ verwaest@chromium.org

[Toon Verwaest, 2015-07-06 15:23:00]

https://codereview.chromium.org/1195983002/diff/1/src/v8natives.js
File src/v8natives.js (right):

https://codereview.chromium.org/1195983002/diff/1/src/v8natives.js#newcode1770
src/v8natives.js:1770: SetFunctionName(result, bound_name, "bound");
This does not work since it sets the name on the SFI, and there's only 1 such
SFI for all bound functions.

function f() {};
var o = f.bind();
function g() {};
g.bind();
assertEquals("bound f", o.name)

[arv (Not doing code reviews), 2015-07-06 15:25:12]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1225793002/ by arv@chromium.org.

The reason for reverting is: Incorrect behavior.