Changeset 108760 in webkit
- Timestamp:
- Feb 24, 2012 2:19:42 AM (12 years ago)
- Location:
- trunk/Source/WebKit2
- Files:
-
- 2 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/Source/WebKit2/ChangeLog
r108730 r108760 1 2012-02-24 Sergio Villar Senin <svillar@igalia.com> 2 3 [WK2] [GTK] Destructor not invoked in EditorClientFrameDestructionObserver 4 https://bugs.webkit.org/show_bug.cgi?id=79466 5 6 Reviewed by Philippe Normand. 7 8 Explicitly cast the observer before deleting it instead of just 9 deleting a generic pointer. We must to that in order to get the 10 destructor of the object properly called. 11 12 * WebProcess/WebCoreSupport/gtk/WebEditorClientGtk.cpp: 13 (WebKit::EditorClientFrameDestructionObserver::destroyOnClosureFinalization): 14 1 15 2012-02-23 Andy Estes <aestes@apple.com> 2 16 -
trunk/Source/WebKit2/WebProcess/WebCoreSupport/gtk/WebEditorClientGtk.cpp
r108631 r108760 149 149 GClosure* m_closure; 150 150 151 static void destroyOnClosureFinalization(gpointer data, GClosure* closure) { delete data; } 151 static void destroyOnClosureFinalization(gpointer data, GClosure* closure) 152 { 153 // Calling delete void* will free the memory but won't invoke 154 // the destructor, something that is a must for us. 155 EditorClientFrameDestructionObserver* observer = static_cast<EditorClientFrameDestructionObserver*>(data); 156 delete observer; 157 } 152 158 }; 153 159
Note: See TracChangeset
for help on using the changeset viewer.