Changeset 143105 in webkit

Timestamp:

Feb 16, 2013 10:08:47 AM (11 years ago)

Author:

jochen@chromium.org

Message:

[chromium] destroy the TestPlugin when the destroy() method is invoked.
​https://bugs.webkit.org/show_bug.cgi?id=110012

Reviewed by Adam Barth.

• DumpRenderTree/chromium/TestRunner/src/TestPlugin.cpp:

(WebTestRunner::TestPlugin::destroy):

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/chromium/TestRunner/src/TestPlugin.cpp (modified) (2 diffs)

trunk/Tools/ChangeLog

@@ +1 @@
+2013-02-16  Jochen Eisinger  <jochen@chromium.org>
+
+        [chromium] destroy the TestPlugin when the destroy() method is invoked.
+        https://bugs.webkit.org/show_bug.cgi?id=110012
+
+        Reviewed by Adam Barth.
+
+        * DumpRenderTree/chromium/TestRunner/src/TestPlugin.cpp:
+        (WebTestRunner::TestPlugin::destroy):
+
 2013-02-15  Adenilson Cavalcanti  <cavalcantii@gmail.com>
 

trunk/Tools/DumpRenderTree/chromium/TestRunner/src/TestPlugin.cpp

@@ -140 +140 @@
 }
 
+void deferredDelete(void* context)
+{
+    TestPlugin* plugin = static_cast<TestPlugin*>(context);
+    delete plugin;
+}
+
 }
 
@@ -232 +238 @@
     m_container = 0;
     m_frame = 0;
+
+    Platform::current()->callOnMainThread(deferredDelete, this);
 }