New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 615783 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

HtmlAudioElementCapturerSource::OnAudioBus receives delay in frames but interprets it as ms.

Project Member Reported by olka@chromium.org, May 30 2016

Issue description

Comment 1 by olka@chromium.org, May 30 2016

Summary: HtmlAudioElementCapturerSource::OnAudioBus receives delay in frames but interprets it as ms. (was: HtmlAudioElementCapturerSource::OnAudioBus receives delay in frames bu interprets it as ms.)

Comment 2 by olka@chromium.org, May 30 2016

Status: Started (was: Assigned)

Comment 4 by olka@chromium.org, May 31 2016

Status: Fixed (was: Started)

Comment 5 by olka@chromium.org, Jun 2 2016

Cc: olka@chromium.org
Owner: mcasas@chromium.org
I'm not sure if the original code ended up in M52 (looks like it was re-laned later).
mcasas@, could you take care of the merge to M52 if it's the case?
Components: Blink>MediaStream>CaptureFromElement
Labels: Merge-Request-52 OS-All
Original CL landed as https://crrev.com/2010173002/ --> 396564
and 52 Dev is in 394939, so we should merge it there. Will do
the merge.

Comment 7 by tin...@google.com, Jun 2 2016

Labels: -Merge-Request-52 Merge-Approved-52 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M52 (branch: 2743)
Labels: -Hotlist-Merge-Approved -Merge-Approved-52
Sorry, it must (have) be(en) too early and couldn't read the numbers I wrote in #6 :P

It landed as 396564, and Dev is 394939, so it's not necessary
(nor possible) to merge back to M52 (and I just found out
while trying to merge back and finding no html_audio... files).

Apologies tinazh@

Sign in to add a comment