Chromium Code Reviews
DescriptionReland of [Chromecast] Process streams with different post-processing. (patchset #1 id:1 of https://codereview.chromium.org/2722673004/ )
Reason for revert:
Fix breakage
Original issue's description:
> Revert of [Chromecast] Process streams with different post-processing. (patchset #14 id:260001 of https://codereview.chromium.org/2701613006/ )
>
> Reason for revert:
> Breaks some chromecast builds due to missing include for <string> in media_pipeline_device_params.h
>
> Original issue's description:
> > [Chromecast] Process streams with different post-processing.
> >
> > Adds |name| to stream_mixer_alsa_input_impl.
> > stream_mixer_alsa can then mix streams of the same time separately,
> > and apply different post-processing to each.
> > Also reduce buffering of system streams from 300ms to 60ms.
> >
> > BUG=internal b/34817384
> > TEST=manual
> >
> > Change-Id: Ieb91e419d7d34c374d83941c02f2546696b4f489
> > Review-Url: https://codereview.chromium.org/2701613006
> > Cr-Commit-Position: refs/heads/master@{#453522}
> > Committed: https://chromium.googlesource.com/chromium/src/+/e975664dab5f430b66817238898839b333822ac3
>
> TBR=kmackay@chromium.org,tianyuwang@google.com,gfhuang@chromium.org,wzhong@chromium.org,jyw@chromium.org,alokp@chromium.org,halliwell@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=internal b/34817384
>
> Review-Url: https://codereview.chromium.org/2722673004
> Cr-Commit-Position: refs/heads/master@{#453613}
> Committed: https://chromium.googlesource.com/chromium/src/+/65dc9e8742f0e87c511806dcc3823cc4c4be8ad4
TBR=kmackay@chromium.org,tianyuwang@google.com,gfhuang@chromium.org,wzhong@chromium.org,jyw@chromium.org,alokp@chromium.org,halliwell@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=internal b/34817384
Review-Url: https://codereview.chromium.org/2722833003
Cr-Commit-Position: refs/heads/master@{#453769}
Committed: https://chromium.googlesource.com/chromium/src/+/d6a8436e67d97350649ca029d212699a93af5e67
Patch Set 1 #Patch Set 2 : Fix include error #Patch Set 3 : Handle empty device_id as DefaultDevice #Messages
Total messages: 12 (4 generated)
|