|Summary:||MediaStreamTrack can't be FINAL|
|Product:||WebKit||Reporter:||Philippe Normand <pnormand>|
|Component:||WebCore Misc.||Assignee:||Nobody <webkit-unassigned>|
|Version:||528+ (Nightly build)|
|Bug Depends on:|
Description Philippe Normand 2013-10-04 02:26:54 PDT
Because AudioStreamTrack and VideoStreamTrack inherit from it.
Comment 2 Alexey Proskuryakov 2013-10-04 10:20:18 PDT
Comment on attachment 213343 [details] patch Are there any methods in these classes that can be final?
Comment 3 Darin Adler 2013-10-04 17:43:34 PDT
Comment on attachment 213343 [details] patch How did this even compile? Can any of the virtual functions in MediaStreamTrack be FINAL? If the entire class isn’t FINAL, it then becomes helpful to mark the functions themselves FINAL if there is no need to override them. r=me but I suggest adding some FINAL to any functions we don’t need to override.
Comment 4 Eric Carlson 2013-10-06 20:47:15 PDT
I have a fix for this in the massive patch attached to https://bugs.webkit.org/show_bug.cgi?id=121940 if you want to let me take care of this.
Comment 5 Philippe Normand 2013-10-07 00:06:00 PDT
I was thinking of landing this patch with the updates Darin and Alexey suggested. (In reply to comment #3) > (From update of attachment 213343 [details]) > How did this even compile? > It didn't :) Currently no port enables mediastream in their default build. Last week I started working on the GStreamer mediastream backend some other folks started some months ago. One part is in Bug 110150 but I haven't yet uploaded the version that actually builds.