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

Side by Side Diff: media/base/android/media_codec_decoder_unittest.cc

Issue 1669193003: Disable flaky media unit tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | media/base/android/media_codec_player_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <utility> 5 #include <utility>
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/thread_task_runner_handle.h" 10 #include "base/thread_task_runner_handle.h"
(...skipping 487 matching lines...) Expand 10 before | Expand all | Expand 10 after
498 base::Unretained(this), true)); 498 base::Unretained(this), true));
499 499
500 EXPECT_TRUE(WaitForCondition(base::Bind(&MediaCodecDecoderTest::is_prefetched, 500 EXPECT_TRUE(WaitForCondition(base::Bind(&MediaCodecDecoderTest::is_prefetched,
501 base::Unretained(this)))); 501 base::Unretained(this))));
502 502
503 // Still, decoder is not configured. 503 // Still, decoder is not configured.
504 EXPECT_FALSE(decoder_->Start(base::TimeDelta::FromMilliseconds(0))); 504 EXPECT_FALSE(decoder_->Start(base::TimeDelta::FromMilliseconds(0)));
505 } 505 }
506 506
507 // http://crbug.com/518900 507 // http://crbug.com/518900
508 TEST_F(MediaCodecDecoderTest, AudioPlayTillCompletion) { 508 TEST_F(MediaCodecDecoderTest, DISABLED_AudioPlayTillCompletion) {
509 SKIP_TEST_IF_MEDIA_CODEC_BRIDGE_IS_NOT_AVAILABLE(); 509 SKIP_TEST_IF_MEDIA_CODEC_BRIDGE_IS_NOT_AVAILABLE();
510 510
511 DVLOG(0) << "AudioPlayTillCompletion started"; 511 DVLOG(0) << "AudioPlayTillCompletion started";
512 512
513 CreateAudioDecoder(); 513 CreateAudioDecoder();
514 514
515 base::TimeDelta duration = base::TimeDelta::FromMilliseconds(500); 515 base::TimeDelta duration = base::TimeDelta::FromMilliseconds(500);
516 base::TimeDelta timeout = base::TimeDelta::FromMilliseconds(1500); 516 base::TimeDelta timeout = base::TimeDelta::FromMilliseconds(1500);
517 517
518 SetDataFactory(scoped_ptr<AudioFactory>(new AudioFactory(duration))); 518 SetDataFactory(scoped_ptr<AudioFactory>(new AudioFactory(duration)));
(...skipping 233 matching lines...) Expand 10 before | Expand all | Expand 10 after
752 752
753 EXPECT_TRUE(decoder_->IsStopped()); 753 EXPECT_TRUE(decoder_->IsStopped());
754 EXPECT_TRUE(decoder_->IsCompleted()); 754 EXPECT_TRUE(decoder_->IsCompleted());
755 EXPECT_EQ(data_factory_->last_pts(), pts_stat_.max()); 755 EXPECT_EQ(data_factory_->last_pts(), pts_stat_.max());
756 756
757 // Check that the reported video size is the one from the in-stream configs. 757 // Check that the reported video size is the one from the in-stream configs.
758 EXPECT_EQ(data_factory_->GetConfigs().video_size, video_size_); 758 EXPECT_EQ(data_factory_->GetConfigs().video_size, video_size_);
759 } 759 }
760 760
761 } // namespace media 761 } // namespace media
OLDNEW
« no previous file with comments | « no previous file | media/base/android/media_codec_player_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698