Reland of Add mojo interface+impl creation of audio streams. (patchset #1 id:1 of https://codereview.chromium.org/2788173002/ )
Reason for revert:
Fixed issue: https://codereview.chromium.org/2788363002/. Reusing LGTMs since the CL is unchanged.
Original issue's description:
> Revert of Add mojo interface+impl creation of audio streams. (patchset #9 id:360001 of https://codereview.chromium.org/2697663003/ )
>
> Reason for revert:
> Broke sizes test on linux x64: https://build.chromium.org/p/chromium/builders/Linux%20x64/builds/36560
>
> Original issue's description:
> > Add mojo interface+impl creation of audio streams.
> >
> > Summary:
> > * RendererAudioOutputStreamFactory is an interface for creating
> > AudioOutputProviders.
> > * Since a lot of data is common for all factories of a single renderer
> > process, it is held in a context object.
> > (AudioOutputStreamFactoryContext is per-process and
> > RendererAudioOutputStreamFactory is per-frame).
> > * Replaces AudioRendererHost (eventually, after field trials).
> > * Follow-up on https://codereview.chromium.org/2697793002/.
> > * Design doc at https://docs.google.com/document/d/1awQoajq_DLmz2AIU9iweC0zEYlVuHCvEIRQepeYyxh8/edit
> > * audio_output_stream_factory_context_impl_unittest.cc is actually an
> > integration test, but it seems common to suffix test files with
> > "_unittest" anyways, so I did that.
> >
> >
> > BUG=425368
> >
> > Review-Url: https://codereview.chromium.org/2697663003
> > Cr-Commit-Position: refs/heads/master@{#461103}
> > Committed: https://chromium.googlesource.com/chromium/src/+/8757b429c30f14d20697f7c6fe4dfdd510cf0b7f
>
> [email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected]
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=425368
>
> Review-Url: https://codereview.chromium.org/2788173002
> Cr-Commit-Position: refs/heads/master@{#461157}
> Committed: https://chromium.googlesource.com/chromium/src/+/925d324b60de6538885b1f74b214b7ca76caca1a
[email protected],[email protected],[email protected],[email protected],[email protected],[email protected],[email protected]
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=425368
Review-Url: https://codereview.chromium.org/2792293002
Cr-Commit-Position: refs/heads/master@{#461660}
11 files changed
