[es5] Class of object is "Function" if object has [[Call]].

[es5] Class of object is "Function" if object has [[Call]].

The concept of class was mostly removed from ES6, but we still use the
class of objects to distinguish object kinds in our builtins.  So update
this to be in sync with IsCallable (thereby getting rid of the previous
instance type based tests for callable things completely).

R=jarin@chromium.org, jkummerow@chromium.org

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

[Benedikt Meurer, 2015-09-03 12:33:49]

Hey guys,

This is preliminary cleanup for proper bound functions support (and
also proper ES6 style proxies). 
Please take a look.

Thanks,
Benedikt

[Jakob Kummerow, 2015-09-03 13:19:23]

LGTM with nits.

https://codereview.chromium.org/1307943013/diff/1/src/arm/lithium-codegen-arm.cc
File src/arm/lithium-codegen-arm.cc (left):

https://codereview.chromium.org/1307943013/diff/1/src/arm/lithium-codegen-arm...
src/arm/lithium-codegen-arm.cc:2643: __ b(gt, is_false);
For the record: I think this signed comparison was actually a bug. Should've
been unsigned.

https://codereview.chromium.org/1307943013/diff/1/src/full-codegen/mips/full-...
File src/full-codegen/mips/full-codegen-mips.cc (right):

https://codereview.chromium.org/1307943013/diff/1/src/full-codegen/mips/full-...
src/full-codegen/mips/full-codegen-mips.cc:3689: __ GetObjectType(v0, v0, a1); 
// Map is now in v8.
s/v8/v0/ :-)

https://codereview.chromium.org/1307943013/diff/1/src/full-codegen/mips64/ful...
File src/full-codegen/mips64/full-codegen-mips64.cc (right):

https://codereview.chromium.org/1307943013/diff/1/src/full-codegen/mips64/ful...
src/full-codegen/mips64/full-codegen-mips64.cc:3692: __ GetObjectType(v0, v0,
a1);  // Map is now in v8.
s/v8/v0/

[Benedikt Meurer, 2015-09-03 13:20:44]

The CQ bit was checked by bmeurer@chromium.org

[Benedikt Meurer, 2015-09-03 13:20:44]

The patchset sent to the CQ was uploaded after l-g-t-m from
jkummerow@chromium.org
Link to the patchset: https://codereview.chromium.org/1307943013/#ps40001
(title: "Address Jakobs comments.")

[commit-bot: I haz the power, 2015-09-03 13:20:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1307943013/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1307943013/40001

[commit-bot: I haz the power, 2015-09-03 14:10:03]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-03 14:10:20]

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

[Michael Achenbach, 2015-09-03 15:02:09]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/1306303005/ by machenbach@chromium.org.

The reason for reverting is: [Sheriff] Changes several layout test expectations.
Please fix upstream first if intended. E.g.:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...

Several lines change from PASS to FAIL..

[Benedikt Meurer, 2015-09-03 18:33:04]

On 2015/09/03 15:02:09, Michael Achenbach wrote:
> A revert of this CL (patchset #3 id:40001) has been created in
> https://codereview.chromium.org/1306303005/ by mailto:machenbach@chromium.org.
> 
> The reason for reverting is: [Sheriff] Changes several layout test
expectations.
> Please fix upstream first if intended. E.g.:
>
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/bui...
> 
> Several lines change from PASS to FAIL..

Hm, there's indeed some funky interplay with WebIDL/DOM. I guess that'll have to
wait for Adam or bindings people to take a look; seems like this is related to
ongoing @@toStringTag problems.