957832 - "ASSERTION: Shouldn't have already notified of finish *and* have output!"

Status: RESOLVED DUPLICATE of bug 962719

(Core :: WebRTC: Audio/Video, defect)

Description

[Jesse Ruderman]

Attachment: testcase

With:
  user_pref("media.navigator.permission.disabled", true);

###!!! ASSERTION: Shouldn't have already notified of finish *and* have output!: '!streamHasOutput[i] || !stream->mNotifiedFinished', file content/media/MediaStreamGraph.cpp, line 398

This assertion was added in bug 943461:

changeset:   http://hg.mozilla.org/mozilla-central/rev/fcc3f7ba913e
user:        Robert O'Callahan
date:        Wed Dec 04 18:59:14 2013 +1300
summary:     Bug 943461. Part 2: Add assertion to catch cases where we NotifyOuput after NotifyFinished. r=padenot

Comment 1

[Jesse Ruderman]

Attachment: stack

Comment 2

[Ryan VanderMeulen [:RyanVM]]

roc, we're hitting this assert pretty often in automation. Can you please take a look? :)

Comment 3

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Does not reproduce for me on Linux unfortunately.

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I will try on the office Mac but it's late Friday afternoon and I won't be able to work on this during the weekend.

Comment 5

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Tried this on Mac debug and opt. Did not fail.

Comment 6

[Karl Tomlinson (:karlt)]

Bug 962719 is reproducible thanks and the explanation in bug 962719 comment 10 means that the assertion is expected to fail.  I'll try to make the notifications more consistent in that bug.