New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 622901 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocking:
issue 617204



Sign in to add a comment

Add more unittests for mojo AudioDecoder

Project Member Reported by xhw...@chromium.org, Jun 23 2016

Issue description

This is to track adding unit tests for MojoAudioDecoder / MojoAudioDecoderService. The test should cover common scenarios and connection errors.

See https://codereview.chromium.org/2070753002 for an example to test MojoRenderer and MojoRendererService.

See  issue 622897  for the bug to add tests for mojo VideoDecoder.
 

Comment 1 by xhw...@chromium.org, Sep 12 2016

Cc: ti...@chromium.org
Labels: -Pri-3 Pri-2
Owner: xhw...@chromium.org
Tima won't have time to do this. Assign back to myself for triage/impl.
Project Member

Comment 2 by sheriffbot@chromium.org, Sep 13 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 3 by xhw...@chromium.org, Sep 13 2017

Status: Assigned (was: Untriaged)
We already have mojo_audio_decoder_unittests.cc, but we should add more test cases there for better coverage.

https://cs.chromium.org/chromium/src/media/mojo/clients/mojo_audio_decoder_unittest.cc?rcl=736875021f3b839c845c25ceceee3bc9e36c3e65&l=246

Comment 4 by xhw...@chromium.org, Sep 13 2017

Summary: Add more unittests for mojo AudioDecoder (was: Add unittests for mojo AudioDecoder)

Sign in to add a comment