Rename delay_milliseconds to frames_delayed.

Rename audio_delay_milliseconds to frames_delayed.

The current name, delay_milliseconds, is misleading since the value is
actually a count of frames since crrev.com/1687213002

BUG=604313
Committed: https://crrev.com/d49506b3d5dda08982bad081b99ec8a9abe5f331
Cr-Commit-Position: refs/heads/master@{#396535}

[chcunningham, 2016-05-23 18:34:47]

PS1 is a WIP - I'm finding that I have to plumb the sample rate around in the
unit tests because AudioConverter is still expressing its delay as a TimeDelta. 

I'll pause here and do a pre-req CL to move AudioConverter over to reporting
frames delayed.

[chcunningham, 2016-05-26 21:37:19]

Description was changed from

==========
Rename delay_milliseconds to frames_delayed.

The current name, delay_milliseconds, is misleading since the value is
actually a count of frames since crrev.com/1687213002

BUG=604313
==========

to

==========
Rename audio_delay_milliseconds to frames_delayed.

The current name, delay_milliseconds, is misleading since the value is
actually a count of frames since crrev.com/1687213002

BUG=604313
==========

[chcunningham, 2016-05-26 21:39:33]

chcunningham@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[chcunningham, 2016-05-26 21:39:33]

PTAL 

Change to AudioConverter was blocking this (otherwise requiring additional
frames / msec conversions to fix the tests).

[DaleCurtis, 2016-05-27 02:44:35]

lgtm

[chcunningham, 2016-05-27 17:50:25]

The CQ bit was checked by chcunningham@chromium.org

[commit-bot: I haz the power, 2016-05-27 17:51:02]

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

[commit-bot: I haz the power, 2016-05-27 19:42:35]

Description was changed from

==========
Rename audio_delay_milliseconds to frames_delayed.

The current name, delay_milliseconds, is misleading since the value is
actually a count of frames since crrev.com/1687213002

BUG=604313
==========

to

==========
Rename audio_delay_milliseconds to frames_delayed.

The current name, delay_milliseconds, is misleading since the value is
actually a count of frames since crrev.com/1687213002

BUG=604313
==========

[commit-bot: I haz the power, 2016-05-27 19:42:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-27 19:43:40]

Description was changed from

==========
Rename audio_delay_milliseconds to frames_delayed.

The current name, delay_milliseconds, is misleading since the value is
actually a count of frames since crrev.com/1687213002

BUG=604313
==========

to

==========
Rename audio_delay_milliseconds to frames_delayed.

The current name, delay_milliseconds, is misleading since the value is
actually a count of frames since crrev.com/1687213002

BUG=604313

Committed: https://crrev.com/d49506b3d5dda08982bad081b99ec8a9abe5f331
Cr-Commit-Position: refs/heads/master@{#396535}
==========

[commit-bot: I haz the power, 2016-05-27 19:43:41]

Patchset 2 (id:??) landed as
https://crrev.com/d49506b3d5dda08982bad081b99ec8a9abe5f331
Cr-Commit-Position: refs/heads/master@{#396535}

[o1ka, 2016-05-30 08:07:41]

olka@chromium.org changed reviewers:
+ olka@chromium.org

[o1ka, 2016-05-30 08:07:42]

Chris, please take a look at my comment. My impression is that
HtmlAudioElementCapturerSource::OnAudioBus operation is broken.

https://codereview.chromium.org/2004963002/diff/20001/media/blink/webaudiosou...
File media/blink/webaudiosourceprovider_impl.cc (right):

https://codereview.chromium.org/2004963002/diff/20001/media/blink/webaudiosou...
media/blink/webaudiosourceprovider_impl.cc:268:
copy_audio_bus_callback_.Run(std::move(bus_copy), frames_delayed,
HtmlAudioElementCapturerSource::OnAudioBus()
(https://cs-staging.chromium.org/chromium/src/content/renderer/media/html_audi...)
which ends up as copy_audio_bus_callback_ here
(https://cs-staging.chromium.org/chromium/src/content/renderer/media/html_audi...),
still interprets it as ms, isn't it?