Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refine Stub Audio Decoder threading #1941

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

osagie98
Copy link
Contributor

@osagie98 osagie98 commented Nov 8, 2023

Shift consumed and output callbacks to the renderer thread instead of the decoder thread, as calls on the decoder thread caused DCHECK failures.

b/309831511

Shift consumed and output callbacks to the renderer thread instead of
the decoder thread, as calls on the decoder thread caused DCHECK
failures.

b/309831511
Copy link

codecov bot commented Nov 8, 2023

Codecov Report

Merging #1941 (fe1f9c9) into main (0651bec) will decrease coverage by 0.02%.
Report is 2 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1941      +/-   ##
==========================================
- Coverage   57.75%   57.74%   -0.02%     
==========================================
  Files        1916     1916              
  Lines       95115    95116       +1     
==========================================
- Hits        54936    54921      -15     
- Misses      40179    40195      +16     
Files Coverage Δ
...ared/starboard/player/filter/stub_audio_decoder.cc 91.35% <100.00%> (+0.05%) ⬆️

... and 6 files with indirect coverage changes

@osagie98 osagie98 merged commit 31b1c90 into youtube:main Nov 8, 2023
334 of 335 checks passed
@osagie98 osagie98 deleted the move-stub-audio-decoder-consumed branch November 8, 2023 22:29
@osagie98 osagie98 added the cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch label Nov 8, 2023
cobalt-github-releaser-bot pushed a commit that referenced this pull request Nov 8, 2023
Shift consumed and output callbacks to the renderer thread instead of
the decoder thread, as calls on the decoder thread caused DCHECK
failures.

b/309831511

(cherry picked from commit 31b1c90)
osagie98 added a commit that referenced this pull request Nov 8, 2023
Refer to the original PR: #1941

Shift consumed and output callbacks to the renderer thread instead of
the decoder thread, as calls on the decoder thread caused DCHECK
failures.

b/309831511

Co-authored-by: Austin Osagie <[email protected]>
Rongo-JL pushed a commit to Rongo-JL/cobalt that referenced this pull request Dec 19, 2023
Shift consumed and output callbacks to the renderer thread instead of
the decoder thread, as calls on the decoder thread caused DCHECK
failures.

b/309831511
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants