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

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

Issue 1067803002: Roll WebRTC 8931:8933, Libjingle 8930:8932 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove NativeHandleImpl Created 5 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/rtc_video_decoder.h" 5 #include "content/renderer/media/rtc_video_decoder.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
10 #include "base/message_loop/message_loop_proxy.h" 10 #include "base/message_loop/message_loop_proxy.h"
11 #include "base/metrics/histogram.h" 11 #include "base/metrics/histogram.h"
12 #include "base/numerics/safe_conversions.h" 12 #include "base/numerics/safe_conversions.h"
13 #include "base/stl_util.h" 13 #include "base/stl_util.h"
14 #include "base/synchronization/waitable_event.h" 14 #include "base/synchronization/waitable_event.h"
15 #include "base/task_runner_util.h" 15 #include "base/task_runner_util.h"
16 #include "content/renderer/media/native_handle_impl.h"
17 #include "gpu/command_buffer/common/mailbox_holder.h" 16 #include "gpu/command_buffer/common/mailbox_holder.h"
18 #include "media/base/bind_to_current_loop.h" 17 #include "media/base/bind_to_current_loop.h"
19 #include "media/renderers/gpu_video_accelerator_factories.h" 18 #include "media/renderers/gpu_video_accelerator_factories.h"
20 #include "third_party/skia/include/core/SkBitmap.h" 19 #include "third_party/skia/include/core/SkBitmap.h"
20 #include "third_party/webrtc/base/bind.h"
21 #include "third_party/webrtc/common_video/interface/i420_video_frame.h" 21 #include "third_party/webrtc/common_video/interface/i420_video_frame.h"
22 #include "third_party/webrtc/system_wrappers/interface/ref_count.h" 22 #include "third_party/webrtc/system_wrappers/interface/ref_count.h"
23 23
24 static void ReleaseFrame(scoped_refptr<media::VideoFrame> frame) {
25 }
26
24 namespace content { 27 namespace content {
25 28
26 const int32 RTCVideoDecoder::ID_LAST = 0x3FFFFFFF; 29 const int32 RTCVideoDecoder::ID_LAST = 0x3FFFFFFF;
27 const int32 RTCVideoDecoder::ID_HALF = 0x20000000; 30 const int32 RTCVideoDecoder::ID_HALF = 0x20000000;
28 const int32 RTCVideoDecoder::ID_INVALID = -1; 31 const int32 RTCVideoDecoder::ID_INVALID = -1;
29 32
30 // Maximum number of concurrent VDA::Decode() operations RVD will maintain. 33 // Maximum number of concurrent VDA::Decode() operations RVD will maintain.
31 // Higher values allow better pipelining in the GPU, but also require more 34 // Higher values allow better pipelining in the GPU, but also require more
32 // resources. 35 // resources.
33 static const size_t kMaxInFlightDecodes = 8; 36 static const size_t kMaxInFlightDecodes = 8;
(...skipping 347 matching lines...) Expand 10 before | Expand all | Expand 10 after
381 GetBufferData(picture.bitstream_buffer_id(), &timestamp); 384 GetBufferData(picture.bitstream_buffer_id(), &timestamp);
382 scoped_refptr<media::VideoFrame> frame = 385 scoped_refptr<media::VideoFrame> frame =
383 CreateVideoFrame(picture, pb, timestamp); 386 CreateVideoFrame(picture, pb, timestamp);
384 bool inserted = 387 bool inserted =
385 picture_buffers_at_display_.insert(std::make_pair( 388 picture_buffers_at_display_.insert(std::make_pair(
386 picture.picture_buffer_id(), 389 picture.picture_buffer_id(),
387 pb.texture_id())).second; 390 pb.texture_id())).second;
388 DCHECK(inserted); 391 DCHECK(inserted);
389 392
390 // Create a WebRTC video frame. 393 // Create a WebRTC video frame.
391 webrtc::RefCountImpl<NativeHandleImpl>* handle = 394 webrtc::I420VideoFrame decoded_image(frame.get(),
392 new webrtc::RefCountImpl<NativeHandleImpl>(frame);
393 webrtc::I420VideoFrame decoded_image(handle,
394 picture.visible_rect().width(), 395 picture.visible_rect().width(),
395 picture.visible_rect().height(), 396 picture.visible_rect().height(),
396 timestamp, 397 timestamp,
397 0); 398 0,
399 webrtc::kVideoRotation_0,
400 rtc::Bind(&ReleaseFrame, frame));
398 401
399 // Invoke decode callback. WebRTC expects no callback after Reset or Release. 402 // Invoke decode callback. WebRTC expects no callback after Reset or Release.
400 { 403 {
401 base::AutoLock auto_lock(lock_); 404 base::AutoLock auto_lock(lock_);
402 DCHECK(decode_complete_callback_ != NULL); 405 DCHECK(decode_complete_callback_ != NULL);
403 if (IsBufferAfterReset(picture.bitstream_buffer_id(), 406 if (IsBufferAfterReset(picture.bitstream_buffer_id(),
404 reset_bitstream_buffer_id_)) { 407 reset_bitstream_buffer_id_)) {
405 decode_complete_callback_->Decoded(decoded_image); 408 decode_complete_callback_->Decoded(decoded_image);
406 } 409 }
407 } 410 }
(...skipping 368 matching lines...) Expand 10 before | Expand all | Expand 10 after
776 UMA_HISTOGRAM_BOOLEAN("Media.RTCVideoDecoderInitDecodeSuccess", sample); 779 UMA_HISTOGRAM_BOOLEAN("Media.RTCVideoDecoderInitDecodeSuccess", sample);
777 return status; 780 return status;
778 } 781 }
779 782
780 void RTCVideoDecoder::DCheckGpuVideoAcceleratorFactoriesTaskRunnerIsCurrent() 783 void RTCVideoDecoder::DCheckGpuVideoAcceleratorFactoriesTaskRunnerIsCurrent()
781 const { 784 const {
782 DCHECK(factories_->GetTaskRunner()->BelongsToCurrentThread()); 785 DCHECK(factories_->GetTaskRunner()->BelongsToCurrentThread());
783 } 786 }
784 787
785 } // namespace content 788 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/media/native_handle_impl.cc ('k') | content/renderer/media/webrtc/media_stream_remote_video_source.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698