Until now, AppendPipeline emitted duration changes for the MediaSource. This was done with MediaSourcePrivateClient::durationChanged(const MediaTime&), a method which was added to MediaSource in r207889 just to implement this in the GStreamer port. This is not necessary though. AppendPipeline only needs to inform MediaSource of the duration of the initialization segment, and MediaSource will in turn set duration from the multi-platform code. This patch removes MediaSourcePrivateClient::durationChanged(const MediaTime&) from the multi-platform API, along with its usages in the GStreamer port, giving the multi-platform code sole responsibility on duration changes. This is a code cleanup and it's covered by existing tests.
Created attachment 403783 [details] Patch
I agree with the change. This was something suggested by Jer Noble on https://bugs.webkit.org/show_bug.cgi?id=162920#c5, so it's great to have it implemented in this patch.
Committed r264164: <https://trac.webkit.org/changeset/264164> All reviewed patches have been landed. Closing bug and clearing flags on attachment 403783 [details].