Revision: 8708 Author: [email protected] Date: Thu Jul 21 04:28:18 2011 Log: Make proxy fixing GC safe.
[email protected] BUG= TEST= Review URL: http://codereview.chromium.org/7460009 http://code.google.com/p/v8/source/detail?r=8708 Modified: /branches/bleeding_edge/src/objects.cc /branches/bleeding_edge/src/runtime.cc ======================================= --- /branches/bleeding_edge/src/objects.cc Wed Jul 20 08:14:50 2011 +++ /branches/bleeding_edge/src/objects.cc Thu Jul 21 04:28:18 2011 @@ -2331,7 +2331,7 @@ Handle<JSProxy> self(this); isolate->factory()->BecomeJSObject(self); - ASSERT(IsJSObject()); + ASSERT(self->IsJSObject()); // TODO(rossberg): recognize function proxies. } ======================================= --- /branches/bleeding_edge/src/runtime.cc Tue Jul 19 02:38:59 2011 +++ /branches/bleeding_edge/src/runtime.cc Thu Jul 21 04:28:18 2011 @@ -619,7 +619,7 @@ ASSERT(args.length() == 1); CONVERT_CHECKED(JSProxy, proxy, args[0]); proxy->Fix(); - return proxy; + return isolate->heap()->undefined_value(); } -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
