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

Issue 784008 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression

Blocking:
issue 777555



Sign in to add a comment

Audio file is playing on M63 but not working on M64

Project Member Reported by avkodipelli@chromium.org, Nov 11 2017

Issue description

Chrome Version: 64.0.3264.0
Chrome OS Version: 10115.0.0
Chrome OS Platform: All(pi, big, eve, few new device)
Network info: Wifi

Please specify Cr-* of the system to which this bug/feature applies (add
the label below).

Steps To Reproduce:
(1) start playing below added audio file either in default player or browser
(2) Observe playback
(3)

Expected Result:
audio will be played

Actual Result:
Playback not starting 

How frequently does this problem reproduce? (Always, sometimes, hard to
reproduce?)
Always. It is working on 10032.32.0/63.0.3239.42
What is the impact to the user, and is there a workaround? If so, what is
it?
Audio file added in next comment.
Feedback report: https://listnr.corp.google.com/report/84637016025
 
Download audio from here: https://pantheon.corp.google.com/storage/browser/chromiumos-test-logs/bugfiles/cr/784008

Not able to reproduce on other audio files.

Comment 2 by dgreid@chromium.org, Nov 11 2017

Owner: wuchengli@chromium.org
please reassign as you see fit.
Cc: chinyue@chromium.org jen...@chromium.org cychiang@chromium.org
Owner: maxmorin@chromium.org
Status: Assigned (was: Untriaged)
The log below should be related. Max. Do you know who can help triage?

[1:15:1110/155754.594403:ERROR:render_media_log.cc(30)] MediaEvent: MEDIA_ERROR_LOG_ENTRY {"error":"Unsupported midstream configuration change! Sample Rate: 48000 vs 48000 ChannelLayout: 10 vs 12 \u003C\u003C Channels: 6 vs 6, Sample Format: 2 vs 2"}
[1:15:1110/155754.595428:ERROR:render_media_log.cc(30)] MediaEvent: MEDIA_ERROR_LOG_ENTRY {"error":"audio decoder reinitialization failed"}
[1:15:1110/155754.597050:ERROR:render_media_log.cc(30)] MediaEvent: MEDIA_ERROR_LOG_ENTRY {"error":"audio error during playing, status: PIPELINE_ERROR_DECODE"}

Cc: maxmorin@chromium.org tguilbert@chromium.org
Owner: dalecur...@chromium.org
The log comes from https://cs.chromium.org/chromium/src/media/filters/ffmpeg_audio_decoder.cc?l=235. For the record, layout 10 is CHANNEL_LAYOUT_5_1 and layout 12 is CHANNEL_LAYOUT_5_1_BACK.  Bug 600538  looks related, and there's also a TODO for Dale and tguilbert regarding channel layout changes, so I'm giving this to them.
Blocking: 777555
Probably okay to allow this conversion. I'll take a look.
Project Member

Comment 7 by bugdroid1@chromium.org, Nov 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0a540dece3c01a090be7c428949e0abb964a4a2c

commit 0a540dece3c01a090be7c428949e0abb964a4a2c
Author: Dale Curtis <dalecurtis@chromium.org>
Date: Fri Nov 17 23:32:32 2017

Allow midstream configuration changes for all codecs.

These seem to happen in a few different circumstances and there's
no real point to restricting to just AAC; the code is simpler and
we already have to handle this regardless downstream in rendering.

BUG= 784008 
TEST=new unittest

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I4e3d1dd9fee0ea1e0817046b13e085d683f5ad84
Reviewed-on: https://chromium-review.googlesource.com/777540
Reviewed-by: Thomas Guilbert <tguilbert@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517601}
[modify] https://crrev.com/0a540dece3c01a090be7c428949e0abb964a4a2c/media/filters/ffmpeg_audio_decoder.cc
[add] https://crrev.com/0a540dece3c01a090be7c428949e0abb964a4a2c/media/test/data/layout_change.wav
[modify] https://crrev.com/0a540dece3c01a090be7c428949e0abb964a4a2c/media/test/pipeline_integration_test.cc

Status: Fixed (was: Assigned)
Cc: avkodipelli@chromium.org
Status: Verified (was: Fixed)
Verified on 10165.0.0, 64.0.3274.0

Sign in to add a comment