Title: [132575] trunk/Source/WebCore
- Revision
- 132575
- Author
- [email protected]
- Date
- 2012-10-26 00:07:15 -0700 (Fri, 26 Oct 2012)
Log Message
[V8] REGRESSION(132540) Assertion failure on V8DOMWrapper::setJSWrapperForDOMNode()
https://bugs.webkit.org/show_bug.cgi?id=100462
Patch by MORITA Hajime <[email protected]> on 2012-10-26
Reviewed by Adam Barth.
Removed the no longer correct assertion statement.
No new tests. Covered by breaking tests.
* bindings/v8/V8DOMWrapper.cpp:
(WebCore::V8DOMWrapper::setJSWrapperForDOMNode):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (132574 => 132575)
--- trunk/Source/WebCore/ChangeLog 2012-10-26 06:39:49 UTC (rev 132574)
+++ trunk/Source/WebCore/ChangeLog 2012-10-26 07:07:15 UTC (rev 132575)
@@ -1,3 +1,17 @@
+2012-10-26 MORITA Hajime <[email protected]>
+
+ [V8] REGRESSION(132540) Assertion failure on V8DOMWrapper::setJSWrapperForDOMNode()
+ https://bugs.webkit.org/show_bug.cgi?id=100462
+
+ Reviewed by Adam Barth.
+
+ Removed the no longer correct assertion statement.
+
+ No new tests. Covered by breaking tests.
+
+ * bindings/v8/V8DOMWrapper.cpp:
+ (WebCore::V8DOMWrapper::setJSWrapperForDOMNode):
+
2012-10-25 Adam Barth <[email protected]>
[V8] WorkerContextExecutionProxy should use ScopedPersistent
Modified: trunk/Source/WebCore/bindings/v8/V8DOMWrapper.cpp (132574 => 132575)
--- trunk/Source/WebCore/bindings/v8/V8DOMWrapper.cpp 2012-10-26 06:39:49 UTC (rev 132574)
+++ trunk/Source/WebCore/bindings/v8/V8DOMWrapper.cpp 2012-10-26 07:07:15 UTC (rev 132575)
@@ -72,7 +72,6 @@
{
v8::Persistent<v8::Object> wrapperHandle = v8::Persistent<v8::Object>::New(wrapper);
ASSERT(maybeDOMWrapper(wrapperHandle));
- ASSERT(!node->isActiveNode());
wrapperHandle.SetWrapperClassId(v8DOMSubtreeClassId);
getDOMNodeMap(isolate).set(node.leakRef(), wrapperHandle);
return wrapperHandle;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
