Reviewers: Jakob,

Message:
PTAL

Description:
Add temporary hack compatible with old delete behavior so we can roll into
blink. We'll need to update the broken test.

BUG=

Please review this at https://codereview.chromium.org/483753003/

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

Affected files (+4, -0 lines):
  M src/objects.cc


Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index d407f74c644c619c71c4af592071901bad26b2b5..897cdadbd138d310868dbc3fce8cf9b96f32cb1a 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -5041,6 +5041,10 @@ MaybeHandle<Object> JSObject::DeleteProperty(Handle<JSObject> object,
                                              ? KEEP_INOBJECT_PROPERTIES
                                              : CLEAR_INOBJECT_PROPERTIES;
         Handle<JSObject> holder = it.GetHolder<JSObject>();
+        // TODO(verwaest): Remove this temporary compatibility hack.
+        if (!holder.is_identical_to(object)) {
+          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