Valgrind reports the following memory leak in webkit_web_view_init. ==9443== 8 bytes in 1 blocks are definitely lost in loss record 1,319 of 31,982 ==9443== at 0x4C2B1C7: operator new(unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) ==9443== by 0x61FB418: webkit_web_view_init(_WebKitWebView*) (webkitwebview.cpp:3558) ==9443== by 0xB19F956: g_type_create_instance (gtype.c:1892) ==9443== by 0xB1840B8: g_object_constructor (gobject.c:1849) ==9443== by 0xB1856E3: g_object_newv (gobject.c:1713) ==9443== by 0xB1861EB: g_object_new (gobject.c:1542) ==9443== by 0x61FB87D: webkit_web_view_new (webkitwebview.cpp:3612) ==9443== by 0x4049B1: createWindow (main.c:306) ==9443== by 0x4053A8: main (main.c:519) Looks like UserMediaClientGtk is not deleted when UserMediaController is destroyed.
This leak can be fixed by deleting UserMediaClientGtk in pageDestroyed() or making the client to be owned by the WebView so that it will be deleted when the view goes away. IMO, the second option is nice because it avoids the pageDestroyed notification function.
Created attachment 145067 [details] Patch Fix for UserMediaClientGtk leak.
Comment on attachment 145067 [details] Patch Clearing flags on attachment: 145067 Committed r119118: <http://trac.webkit.org/changeset/119118>
All reviewed patches have been landed. Closing bug.