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

Unified Diff: media/test/pipeline_integration_test.cc

Issue 2132593002: Remove remaining calls to deprecated MessageLoop methods on Mac. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove ios call sites Created 4 years, 5 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 e244dde2fbe2fd12471ea7632ce8eb62ca659bb6..ad9ffa9e8e8bef1c4ec8b6b4f5639fbb9f6fb6ce 100644
--- a/media/test/pipeline_integration_test.cc
+++ b/media/test/pipeline_integration_test.cc
@@ -10,12 +10,10 @@
#include "base/bind.h"
#include "base/command_line.h"
-#include "base/location.h"
#include "base/macros.h"
#include "base/memory/ptr_util.h"
#include "base/memory/ref_counted.h"
#include "base/run_loop.h"
-#include "base/single_thread_task_runner.h"
#include "base/strings/string_split.h"
#include "base/strings/string_util.h"
#include "base/threading/thread_task_runner_handle.h"
@@ -951,7 +949,7 @@ TEST_F(PipelineIntegrationTest, PlaybackWithAudioTrackDisabledThenEnabled) {
// Disable audio.
std::vector<MediaTrack::Id> empty;
pipeline_->OnEnabledAudioTracksChanged(empty);
- message_loop_.RunUntilIdle();
+ base::RunLoop().RunUntilIdle();
// Seek to flush the pipeline and ensure there's no prerolled audio data.
ASSERT_TRUE(Seek(base::TimeDelta()));
@@ -968,7 +966,7 @@ TEST_F(PipelineIntegrationTest, PlaybackWithAudioTrackDisabledThenEnabled) {
std::vector<MediaTrack::Id> audioTrackId;
audioTrackId.push_back("2");
pipeline_->OnEnabledAudioTracksChanged(audioTrackId);
- message_loop_.RunUntilIdle();
+ base::RunLoop().RunUntilIdle();
// Restart playback from 500ms position.
ASSERT_TRUE(Seek(k500ms));
@@ -985,7 +983,7 @@ TEST_F(PipelineIntegrationTest, PlaybackWithVideoTrackDisabledThenEnabled) {
// Disable video.
std::vector<MediaTrack::Id> empty;
pipeline_->OnSelectedVideoTrackChanged(empty);
- message_loop_.RunUntilIdle();
+ base::RunLoop().RunUntilIdle();
// Seek to flush the pipeline and ensure there's no prerolled video data.
ASSERT_TRUE(Seek(base::TimeDelta()));
@@ -1006,7 +1004,7 @@ TEST_F(PipelineIntegrationTest, PlaybackWithVideoTrackDisabledThenEnabled) {
std::vector<MediaTrack::Id> videoTrackId;
videoTrackId.push_back("1");
pipeline_->OnSelectedVideoTrackChanged(videoTrackId);
- message_loop_.RunUntilIdle();
+ base::RunLoop().RunUntilIdle();
// Seek to flush video pipeline and reset the video hash again to clear state
// if some prerolled frames got hashed after enabling video.

Powered by Google App Engine
This is Rietveld 408576698