We should make those static in the cpp file: mediaPlayerPrivateMessageCallback mediaPlayerPrivateRepaintCallback mediaPlayerPrivateSourceChangedCallback
Created attachment 89182 [details] proposed patch
Comment on attachment 89182 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=89182&action=review Otherwise the patch looks good for me. It makes the code easier to read. > Source/WebCore/ChangeLog:5 > + [GStreamer] some public callbacks remain Shouldn't you put a uper case letter after [GStreamer]
Comment on attachment 89182 [details] proposed patch Nice cleanup.
Committed r83598: <http://trac.webkit.org/changeset/83598>