Revision: 22589
Author:   [email protected]
Date:     Thu Jul 24 09:31:45 2014 UTC
Log: Extend the interceptor setter ASSERT to support the JSGlobalProxy case.

BUG=v8:3463
LOG=n
[email protected]

Review URL: https://codereview.chromium.org/415973004
http://code.google.com/p/v8/source/detail?r=22589

Modified:
 /branches/bleeding_edge/src/stub-cache.cc

=======================================
--- /branches/bleeding_edge/src/stub-cache.cc   Wed Jul 23 11:16:29 2014 UTC
+++ /branches/bleeding_edge/src/stub-cache.cc   Thu Jul 24 09:31:45 2014 UTC
@@ -606,7 +606,16 @@
   Handle<JSObject> receiver = args.at<JSObject>(0);
   Handle<Name> name = args.at<Name>(1);
   Handle<Object> value = args.at<Object>(2);
-  ASSERT(receiver->HasNamedInterceptor());
+#ifdef DEBUG
+  if (receiver->IsJSGlobalProxy()) {
+    PrototypeIterator iter(isolate, receiver);
+    ASSERT(iter.IsAtEnd() ||
+ Handle<JSGlobalObject>::cast(PrototypeIterator::GetCurrent(iter))
+               ->HasNamedInterceptor());
+  } else {
+    ASSERT(receiver->HasNamedInterceptor());
+  }
+#endif
   Handle<Object> result;
   ASSIGN_RETURN_FAILURE_ON_EXCEPTION(
       isolate, result,

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to