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

Issue 246203006: Re-enabling previously flaky WebRTC audio tests. (Closed)

Created:
6 years, 8 months ago by phoglund_chromium
Modified:
6 years, 7 months ago
CC:
chromium-reviews, glider+watch_chromium.org, timurrrr+watch_chromium.org, bruening+watch_chromium.org
Visibility:
Public.

Description

Re-enabling previously flaky WebRTC audio tests. These should be stable under TSAN now that this patch is landed: https://codereview.chromium.org/240403003. BUG=358390 R=jyasskin@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266909

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -4 lines) Patch
M tools/valgrind/gtest_exclude/content_browsertests.gtest-tsan.txt View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 12 (0 generated)
phoglund_chromium
6 years, 8 months ago (2014-04-23 09:02:50 UTC) #1
Alexander Potapenko
On 2014/04/23 09:02:50, phoglund wrote: This CL affects only TSan v1, but not v2. If ...
6 years, 8 months ago (2014-04-23 09:16:18 UTC) #2
phoglund_chromium
On 2014/04/23 09:16:18, Alexander Potapenko wrote: > On 2014/04/23 09:02:50, phoglund wrote: > > This ...
6 years, 8 months ago (2014-04-23 09:41:15 UTC) #3
phoglund_chromium
> There were no such #ifdefs, so we should be good with this. I think ...
6 years, 8 months ago (2014-04-23 09:42:52 UTC) #4
Alexander Potapenko
On 2014/04/23 09:42:52, phoglund wrote: > > There were no such #ifdefs, so we should ...
6 years, 8 months ago (2014-04-23 10:46:13 UTC) #5
phoglund_chromium
The CQ bit was checked by phoglund@chromium.org
6 years, 7 months ago (2014-04-28 14:45:31 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/phoglund@chromium.org/246203006/1
6 years, 7 months ago (2014-04-28 14:46:09 UTC) #7
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 7 months ago (2014-04-28 15:11:58 UTC) #8
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on android_dbg
6 years, 7 months ago (2014-04-28 15:11:59 UTC) #9
phoglund_chromium
The CQ bit was checked by phoglund@chromium.org
6 years, 7 months ago (2014-04-29 16:14:52 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/phoglund@chromium.org/246203006/1
6 years, 7 months ago (2014-04-29 16:15:32 UTC) #11
commit-bot: I haz the power
6 years, 7 months ago (2014-04-29 17:35:30 UTC) #12
Message was sent while issue was closed.
Change committed as 266909

Powered by Google App Engine
This is Rietveld 408576698