Changeset 49429 in webkit

Timestamp:

Oct 11, 2009 3:48:54 PM (15 years ago)

Author:

eric@webkit.org

Message:

2009-10-11 Søren Gjesse <​sgjesse@chromium.org>

Reviewed by Adam Barth.

[V8] Add a context scope in JS listener destructor. The destructor code uses the cotext. See ​http://crbug.com/24200.
​https://bugs.webkit.org/show_bug.cgi?id=30250

• bindings/v8/V8AbstractEventListener.cpp: (WebCore::V8AbstractEventListener::~V8AbstractEventListener):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• bindings/v8/V8AbstractEventListener.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-10-11  Søren Gjesse  <sgjesse@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        [V8] Add a context scope in JS listener destructor. The destructor code uses the cotext. See http://crbug.com/24200.
+        https://bugs.webkit.org/show_bug.cgi?id=30250
+
+        * bindings/v8/V8AbstractEventListener.cpp:
+        (WebCore::V8AbstractEventListener::~V8AbstractEventListener):
+
 2009-10-11  Lyon Chen  <lyon.chen@torchmobile.com>
 

trunk/WebCore/bindings/v8/V8AbstractEventListener.cpp

@@ -77 +77 @@
 {
     if (!m_listener.IsEmpty()) {
-        v8::HandleScope scope;
+        v8::HandleScope handleScope;
+        v8::Context::Scope scope(m_context->get());
         v8::Local<v8::Object> listener = v8::Local<v8::Object>::New(m_listener);
         V8EventListenerList::clearWrapper(listener, m_isAttribute);