On Thu, Jul 16, 2015 at 5:09 PM, Jonas Sicking <jo...@sicking.cc> wrote:

> On Wed, Jul 15, 2015 at 12:42 PM, Jan-Ivar Bruaroey <j...@mozilla.com>
> wrote:
> > This means it will throw TypeError on set of: MediaSource objects, Blob
> > objects, and File objects, for now.
>
> For what it's worth, I think implementing Blob/File support would be
> quite trivial. Just make
>
> elem.srcObject = blob;
>
> internally call
>
> URL.revokeObjectURL(this.mOldObjectURL);
> this.mOldObjectURL = URL.createObjectURL(blob);
> LoadSrc(this.mOldObjectURL);
>

With this implementation, if you set elem.srcObject = blob and then release
all references to the media element, the media element is GCed and the Blob
is leaked. Avoiding that is exactly why the srcObject API is preferred over
createObjectURL.

Rob
-- 
lbir ye,ea yer.tnietoehr  rdn rdsme,anea lurpr  edna e hnysnenh hhe uresyf
toD
selthor  stor  edna  siewaoeodm  or v sstvr  esBa  kbvted,t
rdsme,aoreseoouoto
o l euetiuruewFa  kbn e hnystoivateweh uresyf tulsa rehr  rdm  or rnea
lurpr
.a war hsrer holsa rodvted,t  nenh hneireseoouot.tniesiewaoeivatewt sstvr
esn
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to