Minor cleanup of the MediaPlayerPrivateAVFoundation and InbandTextTrackPrivateAVF classes
<rdar://problem/13281692>
Created attachment 189987 [details] Proposed patch
Comment on attachment 189987 [details] Proposed patch Attachment 189987 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://queues.webkit.org/results/16730681
Comment on attachment 189987 [details] Proposed patch Attachment 189987 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/16735666
Created attachment 189988 [details] Updated patch.
Comment on attachment 189988 [details] Updated patch. View in context: https://bugs.webkit.org/attachment.cgi?id=189988&action=review > Source/WebCore/platform/graphics/avfoundation/InbandTextTrackPrivateAVF.h:40 > + virtual ~AVFInbandTrackParent() { } Since there is already a matching .cpp file, it's better to to have the implementation there. Inlining destructors and constructors of polymorphic classes significantly and unnecessarily increases code bloat.
Committed http://trac.webkit.org/changeset/143884.