[chrome.displaySource] Integrate audio encoders into media pipeline.

[chrome.displaySource] Integrate audio encoders into media pipeline.

This part of a Wi-Fi Display audo encoder patch series:
 * https://codereview.chromium.org/1899943005/
   Add a base class for audio encoders.
 * https://codereview.chromium.org/1903823002/
   Implement LPCM audio encoder.
 * https://codereview.chromium.org/1905243003/  <-- This CL
   Integrate audio encoders into media pipeline.

BUG=242107
Committed: https://crrev.com/5e8d2f955b2f5d39f5ed1417361033cfb364969e
Cr-Commit-Position: refs/heads/master@{#390617}

[e_hakkinen, 2016-04-22 09:14:25]

Description was changed from

==========
[chrome.displaySource] Use audio encoders.

This patch integrates new audio encoders into Wi-Fi Display media pipeline.

BUG=242107
==========

to

==========
[chrome.displaySource] Integrate audio encoders into media pipeline.

This part of a Wi-Fi Display audo encoder patch series:
 * https://codereview.chromium.org/1899943005/
   Add a base class for audio encoders.
 * https://codereview.chromium.org/1903823002/
   Implement LPCM audio encoder.
 * https://codereview.chromium.org/1905243003/  <-- This CL
   Integrate audio encoders into media pipeline.

BUG=242107
==========

[e_hakkinen, 2016-04-22 16:31:26]

Patchset #1 (id:1) has been deleted

[e_hakkinen, 2016-04-28 05:54:35]

Patchset #1 (id:20001) has been deleted

[e_hakkinen, 2016-04-28 05:57:05]

eero.hakkinen@intel.com changed reviewers:
+ alexander.shalamov@intel.com, asargent@chromium.org,
mikhail.pozdnyakov@intel.com

[e_hakkinen, 2016-04-28 05:57:06]

PTAL.

[Mikhail, 2016-04-28 12:28:19]

lgtm with a comment

https://codereview.chromium.org/1905243003/diff/40001/extensions/renderer/api...
File
extensions/renderer/api/display_source/wifi_display/wifi_display_media_pipeline.h
(right):

https://codereview.chromium.org/1905243003/diff/40001/extensions/renderer/api...
extensions/renderer/api/display_source/wifi_display/wifi_display_media_pipeline.h:60:
content::MediaStreamAudioSink* audio_sink() { return audio_encoder_.get(); }
better return it as it is: WiFiDisplayAudioEncoder* audio_encoder()

[e_hakkinen, 2016-04-28 12:50:57]

https://codereview.chromium.org/1905243003/diff/40001/extensions/renderer/api...
File
extensions/renderer/api/display_source/wifi_display/wifi_display_media_pipeline.h
(right):

https://codereview.chromium.org/1905243003/diff/40001/extensions/renderer/api...
extensions/renderer/api/display_source/wifi_display/wifi_display_media_pipeline.h:60:
content::MediaStreamAudioSink* audio_sink() { return audio_encoder_.get(); }
On 2016/04/28 12:28:19, Mikhail wrote:
> better return it as it is: WiFiDisplayAudioEncoder* audio_encoder()

Done.

[asargent_no_longer_on_chrome, 2016-04-28 21:15:43]

Doesn't look like there's anything not covered by OWNERS who've already
commented - let me know if you want me to look at anything in particular,
otherwise feel free to go ahead and submit

[e_hakkinen, 2016-04-29 07:41:08]

On 2016/04/28 21:15:43, Antony Sargent wrote:
> Doesn't look like there's anything not covered by OWNERS who've already
> commented - let me know if you want me to look at anything in particular,
> otherwise feel free to go ahead and submit

I will submit once other CLs on which this CL depends have landed.

[e_hakkinen, 2016-04-29 10:50:07]

The CQ bit was checked by eero.hakkinen@intel.com

[e_hakkinen, 2016-04-29 10:50:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
mikhail.pozdnyakov@intel.com
Link to the patchset: https://codereview.chromium.org/1905243003/#ps80001
(title: "Rebase")

[commit-bot: I haz the power, 2016-04-29 10:50:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1905243003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1905243003/80001

[commit-bot: I haz the power, 2016-04-29 11:23:59]

Description was changed from

==========
[chrome.displaySource] Integrate audio encoders into media pipeline.

This part of a Wi-Fi Display audo encoder patch series:
 * https://codereview.chromium.org/1899943005/
   Add a base class for audio encoders.
 * https://codereview.chromium.org/1903823002/
   Implement LPCM audio encoder.
 * https://codereview.chromium.org/1905243003/  <-- This CL
   Integrate audio encoders into media pipeline.

BUG=242107
==========

to

==========
[chrome.displaySource] Integrate audio encoders into media pipeline.

This part of a Wi-Fi Display audo encoder patch series:
 * https://codereview.chromium.org/1899943005/
   Add a base class for audio encoders.
 * https://codereview.chromium.org/1903823002/
   Implement LPCM audio encoder.
 * https://codereview.chromium.org/1905243003/  <-- This CL
   Integrate audio encoders into media pipeline.

BUG=242107
==========

[commit-bot: I haz the power, 2016-04-29 11:24:00]

Committed patchset #3 (id:80001)

[commit-bot: I haz the power, 2016-04-30 17:24:59]

Patchset 3 (id:??) landed as
https://crrev.com/5e8d2f955b2f5d39f5ed1417361033cfb364969e
Cr-Commit-Position: refs/heads/master@{#390617}

[commit-bot: I haz the power, 2016-04-30 18:00:06]

Description was changed from

==========
[chrome.displaySource] Integrate audio encoders into media pipeline.

This part of a Wi-Fi Display audo encoder patch series:
 * https://codereview.chromium.org/1899943005/
   Add a base class for audio encoders.
 * https://codereview.chromium.org/1903823002/
   Implement LPCM audio encoder.
 * https://codereview.chromium.org/1905243003/  <-- This CL
   Integrate audio encoders into media pipeline.

BUG=242107
==========

to

==========
[chrome.displaySource] Integrate audio encoders into media pipeline.

This part of a Wi-Fi Display audo encoder patch series:
 * https://codereview.chromium.org/1899943005/
   Add a base class for audio encoders.
 * https://codereview.chromium.org/1903823002/
   Implement LPCM audio encoder.
 * https://codereview.chromium.org/1905243003/  <-- This CL
   Integrate audio encoders into media pipeline.

BUG=242107

Committed: https://crrev.com/5e8d2f955b2f5d39f5ed1417361033cfb364969e
Cr-Commit-Position: refs/heads/master@{#390617}
==========