[Arguments] Remove synthetic properties and all code dealing with them.

arguments/src/scopes.cc

Index: arguments/src/scopes.cc
diff --git a/arguments/src/scopes.cc b/arguments/src/scopes.cc
index cf39f8fce5b3bba22e7aabc29e68b0283c4184f2..a0fbb94ac5e07e254d076ae4d9d279f13b3520a2 100644
--- a/arguments/src/scopes.cc
+++ b/arguments/src/scopes.cc
@@ -1,4 +1,4 @@
-// Copyright 2010 the V8 project authors. All rights reserved.
+// Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
@@ -908,8 +908,9 @@ void Scope::AllocateParameterLocals() {
 void Scope::AllocateNonParameterLocal(Variable* var) {
   ASSERT(var->scope() == this);
   ASSERT(var->rewrite() == NULL ||

[Kevin Millikin (Chromium), 2011/03/17 14:44:08]

I honestly don't understand why this assert is here.  It seems kind of silly.

-         (!var->IsVariable(Factory::result_symbol())) ||
-         (var->AsSlot() == NULL || var->AsSlot()->type() != Slot::LOCAL));
+         !var->IsVariable(Factory::result_symbol()) ||
+         var->AsSlot() == NULL ||
+         var->AsSlot()->type() != Slot::LOCAL);
   if (var->rewrite() == NULL && MustAllocate(var)) {
     if (MustAllocateInContext(var)) {
       AllocateHeapSlot(var);