Simplify and optimize ToBoolean handling.

Simplify and optimize ToBoolean handling.

Changing our builtin JavaScript code slightly, we can make sure that we never
see internal objects as arguments for ToBoolean at runtime. Removing that case
from the stub generator and crankshaft makes things a lot easier.

Heap numbers can never be undetectable (only strings and spec objects can), so
we can leave out a useless test.

Try to re-use a non-null register value when returning 'true' in some cases.

Removed special handling of the 'handle all' case, it will very probably never
happen in real code and only makes things more complicated.

Improved naming of the ToBoolean stubs a bit, reflecting the order in which
cases are handled in the code itself.
Committed: http://code.google.com/p/v8/source/detail?r=8886

[Erik Corry, 2011-08-10 16:44:35]

LGTM

http://codereview.chromium.org/7497063/diff/1/src/apinatives.js
File src/apinatives.js (right):

http://codereview.chromium.org/7497063/diff/1/src/apinatives.js#newcode82
src/apinatives.js:82: fun.prototype = typeof prototype ==='undefined' ?
Missing space after ===

http://codereview.chromium.org/7497063/diff/1/src/apinatives.js#newcode91
src/apinatives.js:91: if (!(typeof parent ==='undefined')) {
Ditto

[Sven Panne, 2011-08-11 07:19:24]

http://codereview.chromium.org/7497063/diff/1/src/apinatives.js
File src/apinatives.js (right):

http://codereview.chromium.org/7497063/diff/1/src/apinatives.js#newcode82
src/apinatives.js:82: fun.prototype = typeof prototype ==='undefined' ?
On 2011/08/10 16:44:36, Erik Corry wrote:
> Missing space after ===

Done.

http://codereview.chromium.org/7497063/diff/1/src/apinatives.js#newcode91
src/apinatives.js:91: if (!(typeof parent ==='undefined')) {
On 2011/08/10 16:44:36, Erik Corry wrote:
> Ditto

Done.