Fixing HtmlAudioElementCapturerSource::OnAudioBus() to accept delay frames intead of milliseconds.

Fixing HtmlAudioElementCapturerSource::OnAudioBus() to accept delay frames intead of milliseconds.
(frames are passed here https://cs-staging.chromium.org/chromium/src/media/blink/webaudiosourceprovider_impl.cc?sq=package:chromium&dr=C&rcl=1464592669&l=276)

BUG=615783
Committed: https://crrev.com/09f80bc860fedce41702537cdd742ae03370bfdf
Cr-Commit-Position: refs/heads/master@{#396744}

[o1ka, 2016-05-30 12:06:50]

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

[o1ka, 2016-05-30 12:06:51]

tommi@, could you PTAL?

[tommi (sloooow) - chröme, 2016-05-30 12:09:26]

lgtm! nice find - I guess we'll need to merge this to 52.

[o1ka, 2016-05-30 12:10:17]

The CQ bit was checked by olka@chromium.org

[commit-bot: I haz the power, 2016-05-30 12:10:26]

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

[commit-bot: I haz the power, 2016-05-30 13:48:35]

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

[commit-bot: I haz the power, 2016-05-30 13:48:36]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[o1ka, 2016-05-30 15:56:44]

The CQ bit was checked by olka@chromium.org

[commit-bot: I haz the power, 2016-05-30 15:56:57]

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

[commit-bot: I haz the power, 2016-05-30 17:41:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-30 17:42:39]

Description was changed from

==========
Fixing HtmlAudioElementCapturerSource::OnAudioBus() to accept delay frames
intead of milliseconds.
(frames are passed here
https://cs-staging.chromium.org/chromium/src/media/blink/webaudiosourceprovid...)

BUG=615783
==========

to

==========
Fixing HtmlAudioElementCapturerSource::OnAudioBus() to accept delay frames
intead of milliseconds.
(frames are passed here
https://cs-staging.chromium.org/chromium/src/media/blink/webaudiosourceprovid...)

BUG=615783

Committed: https://crrev.com/09f80bc860fedce41702537cdd742ae03370bfdf
Cr-Commit-Position: refs/heads/master@{#396744}
==========

[commit-bot: I haz the power, 2016-05-30 17:42:40]

Patchset 1 (id:??) landed as
https://crrev.com/09f80bc860fedce41702537cdd742ae03370bfdf
Cr-Commit-Position: refs/heads/master@{#396744}

[chcunningham, 2016-05-31 16:21:49]

Sorry I missed this one! Thanks a ton

[miu, 2016-05-31 19:59:27]

On 2016/05/31 16:21:49, chcunningham wrote:
> Sorry I missed this one! Thanks a ton

I think this was new code being committed at the same time and the frame_delay
change.

[o1ka, 2016-06-01 11:31:37]

On 2016/05/31 19:59:27, miu wrote:
> On 2016/05/31 16:21:49, chcunningham wrote:
> > Sorry I missed this one! Thanks a ton
> 
> I think this was new code being committed at the same time and the frame_delay
> change.

Yes, combined with misleading naming the new code relied on
(https://bugs.chromium.org/p/chromium/issues/detail?id=604313).
But we caught it early this time :)

[o1ka, 2016-06-15 16:44:37]

Description was changed from

==========
Fixing HtmlAudioElementCapturerSource::OnAudioBus() to accept delay frames
intead of milliseconds.
(frames are passed here
https://cs-staging.chromium.org/chromium/src/media/blink/webaudiosourceprovid...)

BUG=615783

Committed: https://crrev.com/09f80bc860fedce41702537cdd742ae03370bfdf
Cr-Commit-Position: refs/heads/master@{#396744}
==========

to

==========
Fixing HtmlAudioElementCapturerSource::OnAudioBus() to accept delay frames
intead of milliseconds.
(frames are passed here
https://cs-staging.chromium.org/chromium/src/media/blink/webaudiosourceprovid...)

BUG=615783

Committed: https://crrev.com/09f80bc860fedce41702537cdd742ae03370bfdf
Cr-Commit-Position: refs/heads/master@{#396744}
==========