Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(244)

Issue 271081: Pad time to next audio data write to compensate for nanosleep inaccuracy on linux. (Closed)

Created:
11 years, 2 months ago by awong
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com, Markus (顧孟勤)
Visibility:
Public.

Description

Pad time to next audio data write to compensate for nanosleep inaccuracy on linux. Nanosleep on linux can miss its deadline by up to 10ms (according to the manpage). Pad 20ms (2x that amount) to give room for missed deadlines. BUG=23974 TEST=listened to previously stuttering audio clip on low-end hardware.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -0 lines) Patch
M media/audio/linux/alsa_output.cc View 2 chunks +10 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
awong
Attempt to fix the clicks on low-end hardware.
11 years, 2 months ago (2009-10-13 22:55:54 UTC) #1
awong
ping
11 years, 2 months ago (2009-10-14 22:29:20 UTC) #2
awong
Alpha, if you get a chance, can you review this change?
11 years, 2 months ago (2009-10-14 23:42:43 UTC) #3
Alpha Left Google
LGTM
11 years, 2 months ago (2009-10-14 23:46:21 UTC) #4
scherkus (not reviewing)
11 years, 2 months ago (2009-10-15 00:37:46 UTC) #5
LGTM -- sorry for delay

Powered by Google App Engine
This is Rietveld 408576698