Reviewers: Kevin Millikin,

Description:
Remove wrong assert to fix a debug crash with arguments object.

We don't need to assert the existence of a length-property of the
arguments object because it is not a JSArray, but just a normal JSObject.

BUG=v8:1227

Please review this at http://codereview.chromium.org/7064020/

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/contexts.cc


Index: src/contexts.cc
===================================================================
--- src/contexts.cc     (revision 8021)
+++ src/contexts.cc     (working copy)
@@ -162,7 +162,6 @@
ASSERT(index >= 0); // arguments must exist and be in the heap context
         Handle<JSObject> arguments(JSObject::cast(context->get(index)),
                                    isolate);
- ASSERT(arguments->HasLocalProperty(isolate->heap()->length_symbol()));
         if (FLAG_trace_contexts) {
PrintF("=> found parameter %d in arguments object\n", param_index);
         }


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to