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

Side by Side Diff: media/filters/pipeline_controller_unittest.cc

Issue 2688383006: [Media, Video] Update WMPI when the pipeline gets a new average keyframe distance. (Closed)
Patch Set: 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 unified diff | Download patch
« no previous file with comments | « media/blink/webmediaplayer_impl.cc ('k') | media/test/pipeline_integration_test_base.h » ('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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "media/filters/pipeline_controller.h" 5 #include "media/filters/pipeline_controller.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 void OnError(PipelineStatus status) override { NOTREACHED(); } 129 void OnError(PipelineStatus status) override { NOTREACHED(); }
130 void OnEnded() override {} 130 void OnEnded() override {}
131 void OnMetadata(PipelineMetadata metadata) override {} 131 void OnMetadata(PipelineMetadata metadata) override {}
132 void OnBufferingStateChange(BufferingState state) override {} 132 void OnBufferingStateChange(BufferingState state) override {}
133 void OnDurationChange() override {} 133 void OnDurationChange() override {}
134 void OnAddTextTrack(const TextTrackConfig& config, 134 void OnAddTextTrack(const TextTrackConfig& config,
135 const AddTextTrackDoneCB& done_cb) override {} 135 const AddTextTrackDoneCB& done_cb) override {}
136 void OnWaitingForDecryptionKey() override {} 136 void OnWaitingForDecryptionKey() override {}
137 void OnVideoNaturalSizeChange(const gfx::Size& size) override {} 137 void OnVideoNaturalSizeChange(const gfx::Size& size) override {}
138 void OnVideoOpacityChange(bool opaque) override {} 138 void OnVideoOpacityChange(bool opaque) override {}
139 void OnVideoAverageKeyframeDistanceUpdate() override {}
139 140
140 base::MessageLoop message_loop_; 141 base::MessageLoop message_loop_;
141 142
142 NiceMock<MockDemuxer> demuxer_; 143 NiceMock<MockDemuxer> demuxer_;
143 StrictMock<MockPipeline> pipeline_; 144 StrictMock<MockPipeline> pipeline_;
144 PipelineController pipeline_controller_; 145 PipelineController pipeline_controller_;
145 146
146 bool was_seeked_ = false; 147 bool was_seeked_ = false;
147 bool last_seeked_time_updated_ = false; 148 bool last_seeked_time_updated_ = false;
148 bool was_suspended_ = false; 149 bool was_suspended_ = false;
(...skipping 183 matching lines...) Expand 10 before | Expand all | Expand 10 after
332 // Request a seek to the same time again. 333 // Request a seek to the same time again.
333 pipeline_controller_.Seek(seek_time, true); 334 pipeline_controller_.Seek(seek_time, true);
334 base::RunLoop().RunUntilIdle(); 335 base::RunLoop().RunUntilIdle();
335 336
336 // Expect the second seek to trigger when the first seek completes. 337 // Expect the second seek to trigger when the first seek completes.
337 EXPECT_CALL(pipeline_, Seek(seek_time, _)); 338 EXPECT_CALL(pipeline_, Seek(seek_time, _));
338 Complete(seek_cb_1); 339 Complete(seek_cb_1);
339 } 340 }
340 341
341 } // namespace media 342 } // namespace media
OLDNEW
« no previous file with comments | « media/blink/webmediaplayer_impl.cc ('k') | media/test/pipeline_integration_test_base.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698