Function#toMethod: check `newHome` before `this`

Function#toMethod: check `newHome` before `this`

`Function.prototype.toMethod` should check `newHome` before `this` as
per the spec. V8 currently does it the other way around, resulting in
the wrong error message being thrown.

TEST=mjsunit/harmony
BUG=v8:3749
LOG=Y

[mathiasb, 2014-12-09 21:56:21]

mathiasb@opera.com changed reviewers:
+ arv@chromium.org

[mathiasb, 2014-12-09 21:56:22]

PTAL

[mathiasb, 2014-12-09 21:59:11]

https://codereview.chromium.org/785303002/diff/1/test/mjsunit/harmony/toMetho...
File test/mjsunit/harmony/toMethod.js (right):

https://codereview.chromium.org/785303002/diff/1/test/mjsunit/harmony/toMetho...
test/mjsunit/harmony/toMethod.js:75: }
I’d love to use `assertThrows` here, but its optional third argument checks the
non-existent `exception.type` property rather than `exception.message`, and I
wasn’t sure if it would be okay to change this.

https://code.google.com/p/v8/source/browse/branches/bleeding_edge/test/mjsuni...

[arv (Not doing code reviews), 2014-12-09 21:59:21]

LGTM with nits.

https://codereview.chromium.org/785303002/diff/1/test/mjsunit/harmony/toMetho...
File test/mjsunit/harmony/toMethod.js (right):

https://codereview.chromium.org/785303002/diff/1/test/mjsunit/harmony/toMetho...
test/mjsunit/harmony/toMethod.js:52: assertEquals(true, false);
assertUnreachable()

https://codereview.chromium.org/785303002/diff/1/test/mjsunit/harmony/toMetho...
test/mjsunit/harmony/toMethod.js:56: 'is not an object', exception.message);
+2 indent

[Dmitry Lomov (no reviews), 2014-12-09 22:14:19]

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

[Dmitry Lomov (no reviews), 2014-12-09 22:14:19]

let's not land until https://bugs.ecmascript.org/show_bug.cgi?id=3430 is
resolved, but thanks for tha patch