Matthew Gregan [:kinetik] Assignee
	Description • 10 years ago

Rather than have an ugly subclass of MediaDecoderStateMachine just to get a
reference to the reader and task queue, give MediaSourceReader its own task
queue (from the same pool as the queue we were using).

	Matthew Gregan [:kinetik] Assignee
	Comment 2 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/012b4107cfa0

	Nigel Babu [:nigelb]
	Comment 4 • 10 years ago

Backed out for mochitest-1 and crashtest bustage

https://hg.mozilla.org/integration/mozilla-inbound/rev/a7d98b35305f

	Chris Pearce [:cpearce (Not reading bugmail)]
	Comment 5 • 10 years ago

Comment on attachment 8473524 [details] [diff] [review]
Shut down MSR task queue.

Review of attachment 8473524 [details] [diff] [review]:
-----------------------------------------------------------------

::: content/media/mediasource/MediaSourceReader.cpp
@@ +46,5 @@
>  }
>  
> +MediaSourceReader::~MediaSourceReader()
> +{
> +  mTaskQueue->Shutdown();

Do this at the end of your MediaDecoderReader::Shutdown() override.

Funny, I had wondered if this would be a problem with your previous patch, but I figured since your Shutdown() override already called Shutdown() on the subreaders, it would be ok. Guess not.. :P

	Matthew Gregan [:kinetik] Assignee
	Comment 6 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/0f59a9eb1658

	Matthew Gregan [:kinetik] Assignee
	Comment 7 • 10 years ago

And backed out again:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9bde32496fa3

Due to bustage:
https://tbpl.mozilla.org/php/getParsedLog.php?id=46144803&tree=Mozilla-Inbound#error0

	Matthew Gregan [:kinetik] Assignee
	Comment 9 • 10 years ago

Simpler approach: use the existing task queue directly, rather than going via MDSM.

	Matthew Gregan [:kinetik] Assignee
	Comment 10 • 10 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/65c3554ccc07

	Ed Morley [:emorley]
	Comment 11 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/65c3554ccc07