Minor correction of AudioProcessing::set_stream_delay_ms() docstring

Minor correction in the docstring of AudioProcessing::set_stream_delay_ms().

BUG=webrtc:7494
NOTRY=True

Review-Url: https://codereview.webrtc.org/2822253002
Cr-Commit-Position: refs/heads/master@{#17762}
Committed: https://chromium.googlesource.com/external/webrtc/+/13fc18068f3e9013e036d584c9c18acaa6f1430b

[AleBzk, 2017-04-18 17:59:44]

Description was changed from

==========
AudioProcessing::set_stream_delay_ms() comment fixed.

BUG=webrtc:7494
==========

to

==========
AudioProcessing::set_stream_delay_ms() comment fixed.

BUG=webrtc:7494
==========

[AleBzk, 2017-04-18 17:59:44]

alessiob@webrtc.org changed reviewers:
+ aleloi@webrtc.org

[AleBzk, 2017-04-18 18:03:47]

Description was changed from

==========
AudioProcessing::set_stream_delay_ms() comment fixed.

BUG=webrtc:7494
==========

to

==========
Minor correction in the docstring of AudioProcessing::set_stream_delay_ms().

BUG=webrtc:7494
NOTRY=True
==========

[AleBzk, 2017-04-18 18:05:24]

Hi Alex,

I started looking into AGC and I came across a teeny-tiny mistake in a comment.
Just check that the correction is not wrong :)

Alessio

[aleloi, 2017-04-19 08:09:26]

aleloi@webrtc.org changed reviewers:
+ peah@webrtc.org

[aleloi, 2017-04-19 08:09:26]

Super-clear change. Lgtm. I'm not owner, so someone else has to approve. Adding
TBR should be fine for something this simple. I've added peah@.

I don't understand how this method is used and why it has to be public. It's
delay parameter comes from MediaStreamAudioProcessor and from AudioTransport.
But how would these components know the delay; doesn't it depend on the echo
path?

[peah-webrtc, 2017-04-19 12:10:10]

On 2017/04/19 08:09:26, aleloi wrote:
> Super-clear change. Lgtm. I'm not owner, so someone else has to approve.
Adding
> TBR should be fine for something this simple. I've added peah@.
> 
> I don't understand how this method is used and why it has to be public. It's
> delay parameter comes from MediaStreamAudioProcessor and from AudioTransport.
> But how would these components know the delay; doesn't it depend on the echo
> path?

lgtm

Regarding the method, the purpose of that is to allow an external assessment of
the echo path delay to be passed to APM. With the changes in AEC3, that is,
however, no longer used.

[AleBzk, 2017-04-19 12:33:16]

The CQ bit was checked by alessiob@webrtc.org

[commit-bot: I haz the power, 2017-04-19 12:33:21]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-04-19 12:35:51]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492605196793440, "parent_rev":
"9e5b11ea75bae6c410d332d35933103fe454c2e0", "commit_rev":
"13fc18068f3e9013e036d584c9c18acaa6f1430b"}

[commit-bot: I haz the power, 2017-04-19 12:35:55]

Description was changed from

==========
Minor correction in the docstring of AudioProcessing::set_stream_delay_ms().

BUG=webrtc:7494
NOTRY=True
==========

to

==========
Minor correction in the docstring of AudioProcessing::set_stream_delay_ms().

BUG=webrtc:7494
NOTRY=True

Review-Url: https://codereview.webrtc.org/2822253002
Cr-Commit-Position: refs/heads/master@{#17762}
Committed:
https://chromium.googlesource.com/external/webrtc/+/13fc18068f3e9013e036d584c...
==========

[commit-bot: I haz the power, 2017-04-19 12:35:55]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/13fc18068f3e9013e036d584c...