Changeset 130125 in webkit

Timestamp:

Oct 1, 2012, 11:32:08 PM (13 years ago)

Author:

abarth@webkit.org

Message:

Unreviewed. Fix ASSERT introduced in
​http://trac.webkit.org/changeset/130103. It turns out this case can
occur. This patch causes us to handle it the same way we did
previously.

• bindings/v8/IntrusiveDOMWrapperMap.h:

(WebCore::IntrusiveDOMWrapperMap::removeIfPresent):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• bindings/v8/IntrusiveDOMWrapperMap.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-10-01  Adam Barth  <abarth@webkit.org>
+
+        Unreviewed. Fix ASSERT introduced in
+        http://trac.webkit.org/changeset/130103. It turns out this case can
+        occur. This patch causes us to handle it the same way we did
+        previously.
+
+        * bindings/v8/IntrusiveDOMWrapperMap.h:
+        (WebCore::IntrusiveDOMWrapperMap::removeIfPresent):
+
 2012-10-01  Shinya Kawanaka  <shinyak@chromium.org>
 

trunk/Source/WebCore/bindings/v8/IntrusiveDOMWrapperMap.h

@@ -78 +78 @@
         if (wrapper.IsEmpty())
             return false;
-        ASSERT(wrapper == value);
+        if (wrapper != value)
+            return false;
         node->disposeWrapper();
         m_nodes.remove(node);