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

Side by Side Diff: content/renderer/media/webmediaplayer_ms_compositor.cc

Issue 2815303006: Convert MediaLog from being ref counted to owned by WebMediaPlayer. (Closed)
Patch Set: Rebase. Created 3 years, 8 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 "content/renderer/media/webmediaplayer_ms_compositor.h" 5 #include "content/renderer/media/webmediaplayer_ms_compositor.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 #include <string> 8 #include <string>
9 9
10 #include "base/command_line.h" 10 #include "base/command_line.h"
11 #include "base/hash.h" 11 #include "base/hash.h"
12 #include "base/single_thread_task_runner.h" 12 #include "base/single_thread_task_runner.h"
13 #include "base/values.h" 13 #include "base/values.h"
14 #include "cc/paint/skia_paint_canvas.h" 14 #include "cc/paint/skia_paint_canvas.h"
15 #include "content/renderer/media/webmediaplayer_ms.h" 15 #include "content/renderer/media/webmediaplayer_ms.h"
16 #include "content/renderer/render_thread_impl.h" 16 #include "content/renderer/render_thread_impl.h"
17 #include "media/base/media_log.h"
18 #include "media/base/media_switches.h" 17 #include "media/base/media_switches.h"
19 #include "media/base/video_frame.h" 18 #include "media/base/video_frame.h"
20 #include "media/base/video_util.h" 19 #include "media/base/video_util.h"
21 #include "media/filters/video_renderer_algorithm.h" 20 #include "media/filters/video_renderer_algorithm.h"
22 #include "media/renderers/skcanvas_video_renderer.h" 21 #include "media/renderers/skcanvas_video_renderer.h"
23 #include "services/ui/public/cpp/gpu/context_provider_command_buffer.h" 22 #include "services/ui/public/cpp/gpu/context_provider_command_buffer.h"
24 #include "skia/ext/platform_canvas.h" 23 #include "skia/ext/platform_canvas.h"
25 #include "third_party/WebKit/public/platform/WebMediaStream.h" 24 #include "third_party/WebKit/public/platform/WebMediaStream.h"
26 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h" 25 #include "third_party/WebKit/public/platform/WebMediaStreamSource.h"
27 #include "third_party/WebKit/public/platform/WebMediaStreamTrack.h" 26 #include "third_party/WebKit/public/platform/WebMediaStreamTrack.h"
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 // Transfer metadata keys. 122 // Transfer metadata keys.
124 new_frame->metadata()->MergeMetadataFrom(frame->metadata()); 123 new_frame->metadata()->MergeMetadataFrom(frame->metadata());
125 return new_frame; 124 return new_frame;
126 } 125 }
127 126
128 } // anonymous namespace 127 } // anonymous namespace
129 128
130 WebMediaPlayerMSCompositor::WebMediaPlayerMSCompositor( 129 WebMediaPlayerMSCompositor::WebMediaPlayerMSCompositor(
131 const scoped_refptr<base::SingleThreadTaskRunner>& compositor_task_runner, 130 const scoped_refptr<base::SingleThreadTaskRunner>& compositor_task_runner,
132 const blink::WebMediaStream& web_stream, 131 const blink::WebMediaStream& web_stream,
133 const base::WeakPtr<WebMediaPlayerMS>& player, 132 const base::WeakPtr<WebMediaPlayerMS>& player)
134 scoped_refptr<media::MediaLog> media_log)
135 : compositor_task_runner_(compositor_task_runner), 133 : compositor_task_runner_(compositor_task_runner),
136 player_(player), 134 player_(player),
137 media_log_(std::move(media_log)),
138 video_frame_provider_client_(nullptr), 135 video_frame_provider_client_(nullptr),
139 current_frame_used_by_compositor_(false), 136 current_frame_used_by_compositor_(false),
140 last_render_length_(base::TimeDelta::FromSecondsD(1.0 / 60.0)), 137 last_render_length_(base::TimeDelta::FromSecondsD(1.0 / 60.0)),
141 total_frame_count_(0), 138 total_frame_count_(0),
142 dropped_frame_count_(0), 139 dropped_frame_count_(0),
143 stopped_(true) { 140 stopped_(true) {
144 main_message_loop_ = base::MessageLoop::current(); 141 main_message_loop_ = base::MessageLoop::current();
145 io_thread_checker_.DetachFromThread(); 142 io_thread_checker_.DetachFromThread();
146 143
147 blink::WebVector<blink::WebMediaStreamTrack> video_tracks; 144 blink::WebVector<blink::WebMediaStreamTrack> video_tracks;
148 if (!web_stream.IsNull()) 145 if (!web_stream.IsNull())
149 web_stream.VideoTracks(video_tracks); 146 web_stream.VideoTracks(video_tracks);
150 147
151 const bool remote_video = 148 const bool remote_video =
152 video_tracks.size() && video_tracks[0].Source().Remote(); 149 video_tracks.size() && video_tracks[0].Source().Remote();
153 150
154 if (remote_video && !base::CommandLine::ForCurrentProcess()->HasSwitch( 151 if (remote_video && !base::CommandLine::ForCurrentProcess()->HasSwitch(
155 switches::kDisableRTCSmoothnessAlgorithm)) { 152 switches::kDisableRTCSmoothnessAlgorithm)) {
156 base::AutoLock auto_lock(current_frame_lock_); 153 base::AutoLock auto_lock(current_frame_lock_);
157 rendering_frame_buffer_.reset(new media::VideoRendererAlgorithm( 154 rendering_frame_buffer_.reset(new media::VideoRendererAlgorithm(
158 base::Bind(&WebMediaPlayerMSCompositor::MapTimestampsToRenderTimeTicks, 155 base::Bind(&WebMediaPlayerMSCompositor::MapTimestampsToRenderTimeTicks,
159 base::Unretained(this)), 156 base::Unretained(this)),
160 media_log_)); 157 &media_log_));
161 } 158 }
162 159
163 // Just for logging purpose. 160 // Just for logging purpose.
164 std::string stream_id = 161 std::string stream_id =
165 web_stream.IsNull() ? std::string() : web_stream.Id().Utf8(); 162 web_stream.IsNull() ? std::string() : web_stream.Id().Utf8();
166 const uint32_t hash_value = base::Hash(stream_id); 163 const uint32_t hash_value = base::Hash(stream_id);
167 serial_ = (hash_value << 1) | (remote_video ? 1 : 0); 164 serial_ = (hash_value << 1) | (remote_video ? 1 : 0);
168 } 165 }
169 166
170 WebMediaPlayerMSCompositor::~WebMediaPlayerMSCompositor() { 167 WebMediaPlayerMSCompositor::~WebMediaPlayerMSCompositor() {
(...skipping 285 matching lines...) Expand 10 before | Expand all | Expand 10 after
456 bool algorithm_enabled) { 453 bool algorithm_enabled) {
457 if (!algorithm_enabled) { 454 if (!algorithm_enabled) {
458 rendering_frame_buffer_.reset(); 455 rendering_frame_buffer_.reset();
459 return; 456 return;
460 } 457 }
461 458
462 if (!rendering_frame_buffer_) { 459 if (!rendering_frame_buffer_) {
463 rendering_frame_buffer_.reset(new media::VideoRendererAlgorithm( 460 rendering_frame_buffer_.reset(new media::VideoRendererAlgorithm(
464 base::Bind(&WebMediaPlayerMSCompositor::MapTimestampsToRenderTimeTicks, 461 base::Bind(&WebMediaPlayerMSCompositor::MapTimestampsToRenderTimeTicks,
465 base::Unretained(this)), 462 base::Unretained(this)),
466 media_log_)); 463 &media_log_));
467 } 464 }
468 } 465 }
469 466
470 } // namespace content 467 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/media/webmediaplayer_ms_compositor.h ('k') | content/renderer/media/webmediaplayer_ms_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698