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

Issue 8957006: Remove the RecordWithSlowSink test. The test has problems running on the build bots and after di... (Closed)

Created:
9 years ago by tommi (sloooow) - chröme
Modified:
9 years ago
CC:
chromium-reviews, hclam+watch_chromium.org, ddorwin+watch_chromium.org, fischman+watch_chromium.org, acolwell+watch_chromium.org, annacc+watch_chromium.org, Paweł Hajdan Jr., vrk (LEFT CHROMIUM), scherkus (not reviewing), ihf+watch_chromium.org
Visibility:
Public.

Description

Remove the RecordWithSlowSink test. The test has problems running on the build bots and after discussing its benefits, we decided that we don't need to test this level of tolerance and that the Record test is enough. BUG=107546 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114628

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -31 lines) Patch
M media/audio/audio_input_unittest.cc View 1 chunk +0 lines, -31 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
tommi (sloooow) - chröme
9 years ago (2011-12-15 12:19:10 UTC) #1
no longer working on chromium
9 years ago (2011-12-15 12:27:43 UTC) #2
On 2011/12/15 12:19:10, tommi wrote:

lgtm

Powered by Google App Engine
This is Rietveld 408576698