1267078 - "Assertion failure: !mIsDoingXPCOMShutDown" with MediaSource, sourceopen event listener

Status: RESOLVED WORKSFORME

(Core :: Audio/Video: Playback, defect, P3)

Description

[Jesse Ruderman]

Attachment: testcase (triggers assertion on shutdown)

Assertion failure: !mIsDoingXPCOMShutDown, at dom/media/MediaShutdownManager.cpp:52

Comment 1

[Jesse Ruderman]

Attachment: stack

Comment 2

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

function boom()
{
    var source = new window.MediaSource();
    var htmlAudio = document.createElement('audio');
    htmlAudio.src = URL.createObjectURL(source);
    source.addEventListener("sourceopen", function(e) {
        htmlAudio.load();
    });
    document.write("Shut down Firefox to complete the test");
}

This test case is interesting because htmlAudio.load() will call MediaSource::SetReadyState() to fire "sourceopen" recursively. I don't know if this is the intention of this test case.

Comment 3

[Anthony Jones (:ajones, :kentuckyfriedtakahe, :k17e)]

Mass change P2 -> P3

Comment 4

[Ryan VanderMeulen [:RyanVM]]

On both Win10 and Ubuntu, this testcase gets stuck in a very long loop of the following now instead on shutdown:

WARNING: '!ipcActor->SendAccumulateChildHistograms(histogramsToSend)', file z:/build/build/src/toolkit/components/telemetry/ipc/TelemetryIPCAccumulator.cpp, line 296
Error: (msgtype=0x42001D,name=PGPU::Msg_AccumulateChildKeyedHistograms) Closed channel: cannot send/recv

Not the issue this bug was originally filed for, but still pretty sad :)

Comment 5

[BugBot [:suhaib / :marco]]

In the process of migrating remaining bugs to the new severity system, the severity for this bug cannot be automatically determined. Please retriage this bug using the new severity system.

Comment 6

[Paul Adenot (:padenot)]

This shuts down cleanly in a debug Firefox here on Linux. We've changed a lot of things in the mentioned code since then.