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

Unified Diff: media/test/pipeline_integration_test.cc

Issue 2710133003: Replace FFmpegDemuxer thread per element with base::TaskScheduler. (Closed)
Patch Set: Give WeakPtr to URLProtocol. Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/test/pipeline_integration_test.cc
diff --git a/media/test/pipeline_integration_test.cc b/media/test/pipeline_integration_test.cc
index f303061af91c2e1ed47a2c10078c63fb5fd5c959..3d0a2a6c601cd6c4af415594c45dcbabbb2bd507 100644
--- a/media/test/pipeline_integration_test.cc
+++ b/media/test/pipeline_integration_test.cc
@@ -1768,16 +1768,12 @@ TEST_P(Mp3FastSeekIntegrationTest, FastSeekAccuracy_MP3) {
EXPECT_HASH_EQ(config.hash, GetAudioHash());
}
-// TODO(CHCUNNINGHAM): Re-enable for OSX once 1% flakiness is root caused.
-// See http://crbug.com/571898
-#if !defined(OS_MACOSX)
// CBR seeks should always be fast and accurate.
INSTANTIATE_TEST_CASE_P(
CBRSeek_HasTOC,
Mp3FastSeekIntegrationTest,
::testing::Values(Mp3FastSeekParams("bear-audio-10s-CBR-has-TOC.mp3",
- "-0.71,0.36,2.96,2.68,2.10,-1.08,")));
-#endif
+ "-0.71,0.36,2.96,2.68,2.11,-1.08,")));
INSTANTIATE_TEST_CASE_P(
CBRSeeks_NoTOC,

Powered by Google App Engine
This is Rietveld 408576698