Include builtin_audio_encoder_factory.h in peer_connection_dependency_factory.cc

Include builtin_audio_encoder_factory.h in peer_connection_dependency_factory.cc

Also add builtin_audio_encoder_factory to deps.
Also move the builtin_audio_decoder_factory into the deps that's added only if building with webrtc.

BUG=webrtc:5806
Review-Url: https://codereview.chromium.org/2808743004
Cr-Commit-Position: refs/heads/master@{#471747}
Committed: https://chromium.googlesource.com/chromium/src/+/578c5aa7e965ad70d31a0372fb7532b030781d05

[ossu-chromium, 2017-05-08 13:36:44]

The CQ bit was checked by ossu@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-08 13:36:54]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-08 13:40:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-08 13:40:51]

Dry run: Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)

[ossu-chromium, 2017-05-08 14:27:29]

The CQ bit was checked by ossu@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-08 14:27:53]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ossu-chromium, 2017-05-08 14:49:26]

Description was changed from

==========
Include builtin_audio_encoder_factory.h in peer_connection_dependency_factory.cc

BUG=webrtc:5806
==========

to

==========
Include builtin_audio_encoder_factory.h in peer_connection_dependency_factory.cc

Also add builtin_audio_encoder_factory to deps.
Also move the builtin_audio_decoder_factory into the deps that's added only if
building with webrtc.

BUG=webrtc:5806
==========

[commit-bot: I haz the power, 2017-05-08 16:03:04]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-08 16:03:05]

Dry run: This issue passed the CQ dry run.

[ossu-chromium, 2017-05-09 16:01:42]

ossu@chromium.org changed reviewers:
+ nick@chromium.org, sergeyu@chromium.org

[ossu-chromium, 2017-05-09 16:01:42]

This CL adds a direct dependency and include of
webrtc::BuiltinAudioEncoderFactory to PeerConnectionDependencyFactory. It was
previously reachable through peerconnectioninterface.h, but that include will
soon go away.

+cc kwiberg@

I've moved the builtin_audio_decoder_factory dependency from the main list to
the one included only if building with webrtc support. I expect this is what was
intended originally, but please stop me if that's not the case. :)

[ossu-chromium, 2017-05-11 08:53:45]

Ping!

This should be a real quick change to validate and will let me finish off the
WebRTC injectable audio encoders project.

[ncarter (slow), 2017-05-11 17:11:34]

lgtm

[ossu-chromium, 2017-05-15 08:58:18]

The CQ bit was checked by ossu@chromium.org

[commit-bot: I haz the power, 2017-05-15 08:58:34]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-15 10:48:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-15 10:48:35]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[ossu-chromium, 2017-05-15 11:08:05]

The CQ bit was checked by ossu@chromium.org

[commit-bot: I haz the power, 2017-05-15 11:08:16]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-15 13:44:51]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494846485458560,
"parent_rev": "eec9c47b685afb76aeb26234f0cb9c46ae0aa007", "commit_rev":
"578c5aa7e965ad70d31a0372fb7532b030781d05"}

[commit-bot: I haz the power, 2017-05-15 13:45:06]

Description was changed from

==========
Include builtin_audio_encoder_factory.h in peer_connection_dependency_factory.cc

Also add builtin_audio_encoder_factory to deps.
Also move the builtin_audio_decoder_factory into the deps that's added only if
building with webrtc.

BUG=webrtc:5806
==========

to

==========
Include builtin_audio_encoder_factory.h in peer_connection_dependency_factory.cc

Also add builtin_audio_encoder_factory to deps.
Also move the builtin_audio_decoder_factory into the deps that's added only if
building with webrtc.

BUG=webrtc:5806

Review-Url: https://codereview.chromium.org/2808743004
Cr-Commit-Position: refs/heads/master@{#471747}
Committed:
https://chromium.googlesource.com/chromium/src/+/578c5aa7e965ad70d31a0372fb75...
==========

[commit-bot: I haz the power, 2017-05-15 13:45:07]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/578c5aa7e965ad70d31a0372fb75...