Chromium Code Reviews
Help | Chromium Project | Sign in
(161)

Issue 11065037: fix the broeken audio for webrtc for precise (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 6 months ago by xians1
Modified:
1 year, 6 months ago
Reviewers:
DaleCurtis, scherkus
CC:
chromium-reviews_chromium.org, feature-media-reviews_chromium.org
Visibility:
Public.

Description

It seems PcmReadi() is often gives out more than 1 packets and this destroy the our scheduling for recording.

BUG=153505
TEST=apprtc.appspot.com?debug=loopback, listen to the audio.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=160661

Patch Set 1 #

Patch Set 2 : fix indentation #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -3 lines) Lint Patch
M media/audio/linux/alsa_input.cc View 1 2 chunks +1 line, -3 lines 0 comments ? errors Download
Trybot results:
Commit:

Messages

Total messages: 6
xians1
Hi Andrew, Could you please take a quick look at this? I need to merge ...
1 year, 6 months ago #1
DaleCurtis
Andrew is out travelling. Won't having PcmReadi provide multiple packets on a regular basis and ...
1 year, 6 months ago #2
xians2
On 2012/10/05 17:25:39, DaleCurtis wrote: > Andrew is out travelling. > > Won't having PcmReadi ...
1 year, 6 months ago #3
DaleCurtis
Thanks for the explanation Xians. LGTM!
1 year, 6 months ago #4
xians1
On 2012/10/05 19:26:56, DaleCurtis wrote: > Thanks for the explanation Xians. LGTM! Thanks Dale. FYI, ...
1 year, 6 months ago #5
I haz the power (commit-bot)
1 year, 6 months ago #6
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1275:d14800f88434