Clean up some messiness in Scopes.

Clean up some messiness in Scopes.

For some reason, the scope's arguments and arguments shadow were
variable proxies, which resulted in all references to the arguments
shadow being shared in the AST.  This makes it hard to put per-node
state on the AST nodes.

I took the opportunity to remove Variable::AsVariable which has
confused people in the past, and to rename Variable::slot to the more
accurate Variable::AsSlot.

Committed: http://code.google.com/p/v8/source/detail?r=5517

[Kevin Millikin (Chromium), 2010-09-23 14:18:17]

I did not try to clean up the whole VariableProxy::AsVariable mess because it
touches too many call sites.

[fschneider, 2010-09-23 14:31:04]

LGTM.