Skip to content

Commit

Permalink
Unreviewed, reverting 272410@main.
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=266899

triggered ASSERTs on Debug bots

Reverted changeset:

"[GStreamer] Fix delayed dispatch of async task post MediaPlayer destruction"
https://bugs.webkit.org/show_bug.cgi?id=266715
https://commits.webkit.org/272410@main

Canonical link: https://commits.webkit.org/272502@main
  • Loading branch information
webkit-commit-queue authored and philn committed Dec 26, 2023
1 parent 359a95f commit 6248d02
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
1 change: 0 additions & 1 deletion Source/WebCore/platform/AbortableTaskQueue.h
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ class AbortableTaskQueue final {
ASSERT(isMainThread());
ASSERT(!m_lock.isHeld());
ASSERT(m_channel.isEmpty());
startAborting();
}

// ===========================
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -170,10 +170,8 @@ GstObject* TrackPrivateBaseGStreamer::objectForLogging() const

void TrackPrivateBaseGStreamer::disconnect()
{
if (m_stream) {
if (m_stream)
g_signal_handlers_disconnect_matched(m_stream.get(), G_SIGNAL_MATCH_DATA, 0, 0, nullptr, nullptr, this);
ASSERT(!g_signal_handler_find(m_stream.get(), G_SIGNAL_MATCH_UNBLOCKED, 0, 0, nullptr, nullptr, nullptr));
}

m_tags.clear();

Expand All @@ -185,11 +183,8 @@ void TrackPrivateBaseGStreamer::disconnect()
m_bestUpstreamPad.clear();
}

if (m_pad) {
g_signal_handlers_disconnect_matched(m_pad.get(), G_SIGNAL_MATCH_DATA, 0, 0, nullptr, nullptr, this);
ASSERT(!g_signal_handler_find(m_pad.get(), G_SIGNAL_MATCH_UNBLOCKED, 0, 0, nullptr, nullptr, nullptr));
if (m_pad)
m_pad.clear();
}
}

void TrackPrivateBaseGStreamer::tagsChanged()
Expand Down

0 comments on commit 6248d02

Please sign in to comment.