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

Side by Side Diff: media/mojo/services/media_service_unittest.cc

Issue 2383663002: Make mojo renderer capable of supporting multiple streams/tracks (Closed)
Patch Set: nit 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
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 <stdint.h> 5 #include <stdint.h>
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 mojo_video_stream_.reset(new MojoDemuxerStreamImpl( 131 mojo_video_stream_.reset(new MojoDemuxerStreamImpl(
132 &video_stream_, MakeRequest(&video_stream_proxy))); 132 &video_stream_, MakeRequest(&video_stream_proxy)));
133 133
134 mojom::RendererClientAssociatedPtrInfo client_ptr_info; 134 mojom::RendererClientAssociatedPtrInfo client_ptr_info;
135 renderer_client_binding_.Bind(&client_ptr_info, 135 renderer_client_binding_.Bind(&client_ptr_info,
136 renderer_.associated_group()); 136 renderer_.associated_group());
137 137
138 EXPECT_CALL(*this, OnRendererInitialized(expected_result)) 138 EXPECT_CALL(*this, OnRendererInitialized(expected_result))
139 .Times(Exactly(1)) 139 .Times(Exactly(1))
140 .WillOnce(InvokeWithoutArgs(run_loop_.get(), &base::RunLoop::Quit)); 140 .WillOnce(InvokeWithoutArgs(run_loop_.get(), &base::RunLoop::Quit));
141 renderer_->Initialize(std::move(client_ptr_info), nullptr, 141 std::vector<mojom::DemuxerStreamPtr> streams;
142 std::move(video_stream_proxy), base::nullopt, 142 streams.push_back(std::move(video_stream_proxy));
143 base::nullopt, 143 renderer_->Initialize(std::move(client_ptr_info), std::move(streams),
144 base::nullopt, base::nullopt,
144 base::Bind(&MediaServiceTest::OnRendererInitialized, 145 base::Bind(&MediaServiceTest::OnRendererInitialized,
145 base::Unretained(this))); 146 base::Unretained(this)));
146 } 147 }
147 148
148 MOCK_METHOD0(ConnectionClosed, void()); 149 MOCK_METHOD0(ConnectionClosed, void());
149 150
150 protected: 151 protected:
151 std::unique_ptr<service_manager::Connection> connection_; 152 std::unique_ptr<service_manager::Connection> connection_;
152 std::unique_ptr<base::RunLoop> run_loop_; 153 std::unique_ptr<base::RunLoop> run_loop_;
153 154
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
213 // close the connection. 214 // close the connection.
214 EXPECT_CALL(*this, ConnectionClosed()) 215 EXPECT_CALL(*this, ConnectionClosed())
215 .Times(Exactly(1)) 216 .Times(Exactly(1))
216 .WillOnce(Invoke(run_loop_.get(), &base::RunLoop::Quit)); 217 .WillOnce(Invoke(run_loop_.get(), &base::RunLoop::Quit));
217 interface_factory_.reset(); 218 interface_factory_.reset();
218 219
219 run_loop_->Run(); 220 run_loop_->Run();
220 } 221 }
221 222
222 } // namespace media 223 } // namespace media
OLDNEW
« no previous file with comments | « media/mojo/services/demuxer_stream_provider_shim.cc ('k') | media/mojo/services/mojo_renderer_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698