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

Side by Side Diff: media/mojo/services/mojo_renderer_service.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
« no previous file with comments | « media/mojo/services/mojo_renderer_service.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/mojo/services/mojo_renderer_service.h" 5 #include "media/mojo/services/mojo_renderer_service.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 DVLOG(1) << __func__; 70 DVLOG(1) << __func__;
71 DCHECK(renderer_); 71 DCHECK(renderer_);
72 72
73 weak_this_ = weak_factory_.GetWeakPtr(); 73 weak_this_ = weak_factory_.GetWeakPtr();
74 } 74 }
75 75
76 MojoRendererService::~MojoRendererService() {} 76 MojoRendererService::~MojoRendererService() {}
77 77
78 void MojoRendererService::Initialize( 78 void MojoRendererService::Initialize(
79 mojom::RendererClientAssociatedPtrInfo client, 79 mojom::RendererClientAssociatedPtrInfo client,
80 mojom::DemuxerStreamPtr audio, 80 base::Optional<std::vector<mojom::DemuxerStreamPtr>> streams,
81 mojom::DemuxerStreamPtr video,
82 const base::Optional<GURL>& media_url, 81 const base::Optional<GURL>& media_url,
83 const base::Optional<GURL>& first_party_for_cookies, 82 const base::Optional<GURL>& first_party_for_cookies,
84 const InitializeCallback& callback) { 83 const InitializeCallback& callback) {
85 DVLOG(1) << __func__; 84 DVLOG(1) << __func__;
86 DCHECK_EQ(state_, STATE_UNINITIALIZED); 85 DCHECK_EQ(state_, STATE_UNINITIALIZED);
87 86
88 client_.Bind(std::move(client)); 87 client_.Bind(std::move(client));
89 state_ = STATE_INITIALIZING; 88 state_ = STATE_INITIALIZING;
90 89
91 if (media_url == base::nullopt) { 90 if (media_url == base::nullopt) {
91 DCHECK(streams.has_value());
92 stream_provider_.reset(new DemuxerStreamProviderShim( 92 stream_provider_.reset(new DemuxerStreamProviderShim(
93 std::move(audio), std::move(video), 93 std::move(*streams),
94 base::Bind(&MojoRendererService::OnStreamReady, weak_this_, callback))); 94 base::Bind(&MojoRendererService::OnStreamReady, weak_this_, callback)));
95 return; 95 return;
96 } 96 }
97 97
98 DCHECK(!audio);
99 DCHECK(!video);
100 DCHECK(!media_url.value().is_empty()); 98 DCHECK(!media_url.value().is_empty());
101 DCHECK(first_party_for_cookies); 99 DCHECK(first_party_for_cookies);
102 stream_provider_.reset(new MediaUrlDemuxer(nullptr, media_url.value(), 100 stream_provider_.reset(new MediaUrlDemuxer(nullptr, media_url.value(),
103 first_party_for_cookies.value())); 101 first_party_for_cookies.value()));
104 renderer_->Initialize( 102 renderer_->Initialize(
105 stream_provider_.get(), this, 103 stream_provider_.get(), this,
106 base::Bind(&MojoRendererService::OnRendererInitializeDone, weak_this_, 104 base::Bind(&MojoRendererService::OnRendererInitializeDone, weak_this_,
107 callback)); 105 callback));
108 } 106 }
109 107
(...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
288 DCHECK(!bad_message_cb_.is_null()); 286 DCHECK(!bad_message_cb_.is_null());
289 bad_message_cb_.Run(); 287 bad_message_cb_.Run();
290 288
291 return; 289 return;
292 } 290 }
293 291
294 callback.Run(initiate_surface_request_cb_.Run()); 292 callback.Run(initiate_surface_request_cb_.Run());
295 } 293 }
296 294
297 } // namespace media 295 } // namespace media
OLDNEW
« no previous file with comments | « media/mojo/services/mojo_renderer_service.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698