1037331 - PROCESS-CRASH | /tests/content/media/test/test_playback_rate_playpause.html on B2G Desktop Linux Opt

Status: RESOLVED DUPLICATE of bug 1037423

(Core :: Audio/Video, defect)

Description

[JW Wang [:jwwang] [:jw_wang]]

https://tbpl.mozilla.org/php/getParsedLog.php?id=43489185&tree=Try&full=1#error1

19:58:28     INFO -  2014-07-10 02:58:28.982876 UTC - -1356018880[97ea7800]: AudioStream: Shutdown 97b58b20, state 6
19:58:28     INFO -  b2g-bin: /builds/slave/try-linux32_g-0000000000000000/build/media/libcubeb/src/cubeb_alsa.c:870: alsa_stream_destroy: Assertion `stm && (stm->state == INACTIVE || stm->state == ERROR)' failed.

|mState| is 6 (ERRORED) when AudioStream::Shutdown() is called. Therefore Pause() [1] is not called while shutting down and causes assertion in the alsa backend.

Btw, |mState| is set to ERRORED in AudioStream::Cancel().

Hi Matthew,
It looks like AudioStream::Cancel() is a new function to AudioStream. Do you know what it is for?


[1] http://hg.mozilla.org/mozilla-central/file/7f9db2379b3f/content/media/AudioStream.cpp#l834

Comment 1

[Matthew Gregan [:kinetik]]

(In reply to JW Wang [:jwwang] from comment #0)
> It looks like AudioStream::Cancel() is a new function to AudioStream. Do you
> know what it is for?

It's used in AudioSink::PrepareToShutdown to cancel a blocked call to AudioStream::Drain.