Revision: 23171
Author:   [email protected]
Date:     Mon Aug 18 16:52:23 2014 UTC
Log: Add temporary hack compatible with old delete behavior so we can roll into blink. We'll need to update the broken test.

BUG=
[email protected]

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

Modified:
 /branches/bleeding_edge/src/objects.cc

=======================================
--- /branches/bleeding_edge/src/objects.cc      Mon Aug 18 15:08:14 2014 UTC
+++ /branches/bleeding_edge/src/objects.cc      Mon Aug 18 16:52:23 2014 UTC
@@ -5041,6 +5041,12 @@
                                              ? KEEP_INOBJECT_PROPERTIES
                                              : CLEAR_INOBJECT_PROPERTIES;
         Handle<JSObject> holder = it.GetHolder<JSObject>();
+ // TODO(verwaest): Remove this temporary compatibility hack when blink
+        // tests are updated.
+        if (!holder.is_identical_to(object) &&
+            !(object->IsJSGlobalProxy() && holder->IsJSGlobalObject())) {
+          return it.isolate()->factory()->true_value();
+        }
         NormalizeProperties(holder, mode, 0);
         Handle<Object> result =
             DeleteNormalizedProperty(holder, name, delete_mode);

--
--
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