LGTM when the ASSERT is removed.
http://codereview.chromium.org/7618012/diff/3001/src/elements.cc File src/elements.cc (right): http://codereview.chromium.org/7618012/diff/3001/src/elements.cc#newcode142 src/elements.cc:142: ASSERT(value->IsString() || value->IsNumber()); This ASSERT should be removed to make this method more general. http://codereview.chromium.org/7618012/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
