admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Reland "Remove usage of webrtc::NativeHandle since is just adds an extra level of indirection.""

Browse Source 
This reverts commit e41d774c4d0a60066866fc2d0ae48dd0e839ff23.

Original code review: https://webrtc-codereview.appspot.com/43999004/
Reason for reland: There was nothing wrong with this cl as is, but it breaks chrome compatibility. We will now reland this and fix Chrome during roll.

Patset 1: Original cl.
Patchset 2: Removed more code that is no longer needed.

R=magjed@webrtc.org, pbos@webrtc.org
TBR=mflodman@webrtc.org

BUG=1128

Review URL: https://webrtc-codereview.appspot.com/45049004

Cr-Commit-Position: refs/heads/master@{#8956}
This commit is contained in:
Per 2015-04-09 13:44:16 +02:00
parent 2c37078e40
commit 9b3f56ea05
17 changed files with 110 additions and 181 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
talk/app/webrtc/java/jni/androidmediadecoder_jni.cc
View File

@ -654,8 +654,9 @@ bool MediaCodecVideoDecoder::DeliverPendingOutputs(
int32_t callback_status = WEBRTC_VIDEO_CODEC_OK;
if (use_surface_) {
native_handle_.SetTextureObject(surface_texture_, texture_id);
I420VideoFrame texture_image(
&native_handle_, width, height, output_timestamp_, 0);
I420VideoFrame texture_image(&native_handle_, width, height,
output_timestamp_, 0, webrtc::kVideoRotation_0,
rtc::Callback0<void>());
texture_image.set_ntp_time_ms(output_ntp_time_ms_);
callback_status = callback_->Decoded(texture_image);
} else {

23
talk/app/webrtc/java/jni/native_handle_impl.h
View File

@ -29,23 +29,14 @@
#ifndef TALK_APP_WEBRTC_JAVA_JNI_NATIVE_HANDLE_IMPL_H_
#define TALK_APP_WEBRTC_JAVA_JNI_NATIVE_HANDLE_IMPL_H_
#include "webrtc/common_video/interface/native_handle.h"
namespace webrtc_jni {
// Wrapper for texture object in TextureBuffer.
class NativeHandleImpl : public webrtc::NativeHandle {
// Wrapper for texture object.
class NativeHandleImpl {
public:
NativeHandleImpl() :
ref_count_(0), texture_object_(NULL), texture_id_(-1) {}
virtual ~NativeHandleImpl() {}
virtual int32_t AddRef() {
return ++ref_count_;
}
virtual int32_t Release() {
return --ref_count_;
}
virtual void* GetHandle() {
NativeHandleImpl() : texture_object_(NULL), texture_id_(-1) {}
void* GetHandle() {
return texture_object_;
}
int GetTextureId() {
@ -55,12 +46,8 @@ class NativeHandleImpl : public webrtc::NativeHandle {
texture_object_ = reinterpret_cast<jobject>(texture_object);
texture_id_ = texture_id;
}
int32_t ref_count() {
return ref_count_;
}
private:
int32_t ref_count_;
jobject texture_object_;
int32_t texture_id_;
};

1
talk/app/webrtc/java/jni/peerconnection_jni.cc
View File

@ -105,7 +105,6 @@ using webrtc::DataChannelInit;
using webrtc::DataChannelInterface;
using webrtc::DataChannelObserver;
using webrtc::IceCandidateInterface;
using webrtc::NativeHandle;
using webrtc::MediaConstraintsInterface;
using webrtc::MediaSourceInterface;
using webrtc::MediaStreamInterface;

17
talk/media/webrtc/webrtcvideoframe.cc
View File

@ -71,23 +71,6 @@ WebRtcVideoFrame::WebRtcVideoFrame(
rotation_(webrtc::kVideoRotation_0) {
}
WebRtcVideoFrame::WebRtcVideoFrame(webrtc::NativeHandle* handle,
int width,
int height,
int64_t elapsed_time_ns,
int64_t time_stamp_ns,
webrtc::VideoRotation rotation)
: video_frame_buffer_(
new rtc::RefCountedObject<webrtc::TextureBuffer>(handle,
width,
height)),
pixel_width_(1),
pixel_height_(1),
elapsed_time_ns_(elapsed_time_ns),
time_stamp_ns_(time_stamp_ns),
rotation_(rotation) {
}
WebRtcVideoFrame::~WebRtcVideoFrame() {}
bool WebRtcVideoFrame::Init(uint32 format,

6
talk/media/webrtc/webrtcvideoframe.h
View File

@ -52,12 +52,6 @@ class WebRtcVideoFrame : public VideoFrame {
int64_t elapsed_time_ns,
int64_t time_stamp_ns);
WebRtcVideoFrame(webrtc::NativeHandle* handle,
int width,
int height,
int64_t elapsed_time_ns,
int64_t time_stamp_ns,
webrtc::VideoRotation rotation);
~WebRtcVideoFrame();
// Creates a frame from a raw sample with FourCC "format" and size "w" x "h".

31
talk/media/webrtc/webrtcvideoframe_unittest.cc
View File

@ -30,19 +30,6 @@
#include "talk/media/base/videoframe_unittest.h"
#include "talk/media/webrtc/webrtcvideoframe.h"
class NativeHandleImpl : public webrtc::NativeHandle {
public:
NativeHandleImpl() : ref_count_(0) {}
virtual ~NativeHandleImpl() {}
virtual int32_t AddRef() { return ++ref_count_; }
virtual int32_t Release() { return --ref_count_; }
virtual void* GetHandle() { return NULL; }
int32_t ref_count() { return ref_count_; }
private:
int32_t ref_count_;
};
namespace {
class WebRtcVideoTestFrame : public cricket::WebRtcVideoFrame {
@ -339,10 +326,12 @@ TEST_F(WebRtcVideoFrameTest, InitRotated90DontApplyRotation) {
}
TEST_F(WebRtcVideoFrameTest, TextureInitialValues) {
NativeHandleImpl handle;
cricket::WebRtcVideoFrame frame(&handle, 640, 480, 100, 200,
webrtc::kVideoRotation_0);
EXPECT_EQ(&handle, frame.GetNativeHandle());
void* dummy_handle = reinterpret_cast<void*>(0x1);
webrtc::TextureBuffer* buffer =
new rtc::RefCountedObject<webrtc::TextureBuffer>(dummy_handle, 640, 480,
rtc::Callback0<void>());
cricket::WebRtcVideoFrame frame(buffer, 100, 200, webrtc::kVideoRotation_0);
EXPECT_EQ(dummy_handle, frame.GetNativeHandle());
EXPECT_EQ(640u, frame.GetWidth());
EXPECT_EQ(480u, frame.GetHeight());
EXPECT_EQ(100, frame.GetElapsedTime());
@ -354,9 +343,11 @@ TEST_F(WebRtcVideoFrameTest, TextureInitialValues) {
}
TEST_F(WebRtcVideoFrameTest, CopyTextureFrame) {
NativeHandleImpl handle;
cricket::WebRtcVideoFrame frame1(&handle, 640, 480, 100, 200,
webrtc::kVideoRotation_0);
void* dummy_handle = reinterpret_cast<void*>(0x1);
webrtc::TextureBuffer* buffer =
new rtc::RefCountedObject<webrtc::TextureBuffer>(dummy_handle, 640, 480,
rtc::Callback0<void>());
cricket::WebRtcVideoFrame frame1(buffer, 100, 200, webrtc::kVideoRotation_0);
cricket::VideoFrame* frame2 = frame1.Copy();
EXPECT_EQ(frame1.GetNativeHandle(), frame2->GetNativeHandle());
EXPECT_EQ(frame1.GetWidth(), frame2->GetWidth());

1
webrtc/common_video/BUILD.gn
View File

@ -21,7 +21,6 @@ source_set("common_video") {
"i420_video_frame.cc",
"interface/i420_video_frame.h",
"interface/i420_buffer_pool.h",
"interface/native_handle.h",
"interface/video_frame_buffer.h",
"libyuv/include/scaler.h",
"libyuv/include/webrtc_libyuv.h",

1
webrtc/common_video/common_video.gyp
View File

@ -41,7 +41,6 @@
'sources': [
'interface/i420_buffer_pool.h',
'interface/i420_video_frame.h',
'interface/native_handle.h',
'interface/video_frame_buffer.h',
'i420_buffer_pool.cc',
'i420_video_frame.cc',

4
webrtc/common_video/i420_buffer_pool.cc
View File

@ -38,9 +38,7 @@ class PooledI420Buffer : public webrtc::VideoFrameBuffer {
int stride(webrtc::PlaneType type) const override {
return buffer_->stride(type);
}
rtc::scoped_refptr<webrtc::NativeHandle> native_handle() const override {
return nullptr;
}
void* native_handle() const override { return nullptr; }
friend class rtc::RefCountedObject<PooledI420Buffer>;
rtc::scoped_refptr<webrtc::I420Buffer> buffer_;

23
webrtc/common_video/i420_video_frame.cc
View File

@ -14,6 +14,7 @@
#include <algorithm> // swap
#include "webrtc/base/bind.h"
#include "webrtc/base/checks.h"
namespace webrtc {
@ -36,20 +37,20 @@ I420VideoFrame::I420VideoFrame(
rotation_(rotation) {
}
I420VideoFrame::I420VideoFrame(NativeHandle* handle,
I420VideoFrame::I420VideoFrame(void* native_handle,
int width,
int height,
uint32_t timestamp,
int64_t render_time_ms)
: video_frame_buffer_(
new rtc::RefCountedObject<TextureBuffer>(handle, width, height)),
timestamp_(timestamp),
ntp_time_ms_(0),
render_time_ms_(render_time_ms),
rotation_(kVideoRotation_0) {
DCHECK(handle != nullptr);
DCHECK_GT(width, 0);
DCHECK_GT(height, 0);
int64_t render_time_ms,
VideoRotation rotation,
const rtc::Callback0<void>& no_longer_used)
: I420VideoFrame(new rtc::RefCountedObject<TextureBuffer>(native_handle,
width,
height,
no_longer_used),
timestamp,
render_time_ms,
rotation) {
}
int I420VideoFrame::CreateEmptyFrame(int width, int height,

28
webrtc/common_video/i420_video_frame_unittest.cc
View File

@ -12,22 +12,21 @@
#include <string.h>
#include "testing/gtest/include/gtest/gtest.h"
#include "webrtc/base/bind.h"
#include "webrtc/base/scoped_ptr.h"
#include "webrtc/common_video/interface/i420_video_frame.h"
namespace webrtc {
class NativeHandleImpl : public NativeHandle {
class NativeHandleImpl {
public:
NativeHandleImpl() : ref_count_(0) {}
NativeHandleImpl() : no_longer_needed_(false) {}
virtual ~NativeHandleImpl() {}
virtual int32_t AddRef() { return ++ref_count_; }
virtual int32_t Release() { return --ref_count_; }
virtual void* GetHandle() { return NULL; }
bool no_longer_needed() const { return no_longer_needed_; }
void SetNoLongerNeeded() { no_longer_needed_ = true; }
int32_t ref_count() { return ref_count_; }
private:
int32_t ref_count_;
bool no_longer_needed_;
};
bool EqualPlane(const uint8_t* data1,
@ -256,7 +255,8 @@ TEST(TestI420VideoFrame, FailToReuseAllocation) {
TEST(TestI420VideoFrame, TextureInitialValues) {
NativeHandleImpl handle;
I420VideoFrame frame(&handle, 640, 480, 100, 10);
I420VideoFrame frame(&handle, 640, 480, 100, 10, webrtc::kVideoRotation_0,
rtc::Callback0<void>());
EXPECT_EQ(640, frame.width());
EXPECT_EQ(480, frame.height());
EXPECT_EQ(100u, frame.timestamp());
@ -269,13 +269,15 @@ TEST(TestI420VideoFrame, TextureInitialValues) {
EXPECT_EQ(20, frame.render_time_ms());
}
TEST(TestI420VideoFrame, RefCount) {
TEST(TestI420VideoFrame, NoLongerNeeded) {
NativeHandleImpl handle;
EXPECT_EQ(0, handle.ref_count());
I420VideoFrame *frame = new I420VideoFrame(&handle, 640, 480, 100, 200);
EXPECT_EQ(1, handle.ref_count());
ASSERT_FALSE(handle.no_longer_needed());
I420VideoFrame* frame = new I420VideoFrame(
&handle, 640, 480, 100, 200, webrtc::kVideoRotation_0,
rtc::Bind(&NativeHandleImpl::SetNoLongerNeeded, &handle));
EXPECT_FALSE(handle.no_longer_needed());
delete frame;
EXPECT_EQ(0, handle.ref_count());
EXPECT_TRUE(handle.no_longer_needed());
}
bool EqualPlane(const uint8_t* data1,

36
webrtc/common_video/interface/native_handle.h
View File

@ -1,36 +0,0 @@
/*
* Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
*
* Use of this source code is governed by a BSD-style license
* that can be found in the LICENSE file in the root of the source
* tree. An additional intellectual property rights grant can be found
* in the file PATENTS. All contributing project authors may
* be found in the AUTHORS file in the root of the source tree.
*/
#ifndef COMMON_VIDEO_INTERFACE_NATIVEHANDLE_H_
#define COMMON_VIDEO_INTERFACE_NATIVEHANDLE_H_
#include "webrtc/typedefs.h"
namespace webrtc {
// A class to store an opaque handle of the underlying video frame. This is used
// when the frame is backed by a texture. WebRTC carries the handle in
// TextureBuffer. This object keeps a reference to the handle. The reference
// is cleared when the object is destroyed. It is important to destroy the
// object as soon as possible so the texture can be recycled.
class NativeHandle {
public:
virtual ~NativeHandle() {}
// For scoped_refptr
virtual int32_t AddRef() = 0;
virtual int32_t Release() = 0;
// Gets the handle.
virtual void* GetHandle() = 0;
};
} // namespace webrtc
#endif // COMMON_VIDEO_INTERFACE_NATIVEHANDLE_H_

23
webrtc/common_video/interface/video_frame_buffer.h
View File

@ -15,7 +15,6 @@
#include "webrtc/base/refcount.h"
#include "webrtc/base/scoped_ptr.h"
#include "webrtc/base/scoped_ref_ptr.h"
#include "webrtc/common_video/interface/native_handle.h"
#include "webrtc/system_wrappers/interface/aligned_malloc.h"
namespace webrtc {
@ -51,7 +50,7 @@ class VideoFrameBuffer : public rtc::RefCountInterface {
// Return the handle of the underlying video frame. This is used when the
// frame is backed by a texture.
virtual rtc::scoped_refptr<NativeHandle> native_handle() const = 0;
virtual void* native_handle() const = 0;
protected:
virtual ~VideoFrameBuffer();
@ -68,7 +67,7 @@ class I420Buffer : public VideoFrameBuffer {
const uint8_t* data(PlaneType type) const override;
uint8_t* data(PlaneType type) override;
int stride(PlaneType type) const override;
rtc::scoped_refptr<NativeHandle> native_handle() const override;
void* native_handle() const override;
protected:
~I420Buffer() override;
@ -82,27 +81,31 @@ class I420Buffer : public VideoFrameBuffer {
const rtc::scoped_ptr<uint8_t, AlignedFreeDeleter> data_;
};
// Texture buffer around a NativeHandle.
// Texture buffer is a VideoFrameBuffer wrapper around a |native_handle|.
// |native_handle| must be valid for the lifetime of an instance of this object.
// |no_longer_used| can be used to manage the lifetime of |native_handle|.
class TextureBuffer : public VideoFrameBuffer {
public:
TextureBuffer(const rtc::scoped_refptr<NativeHandle>& native_handle,
TextureBuffer(void* native_handle,
int width,
int height);
int height,
const rtc::Callback0<void>& no_longer_used);
int width() const override;
int height() const override;
const uint8_t* data(PlaneType type) const override;
uint8_t* data(PlaneType type) override;
int stride(PlaneType type) const override;
rtc::scoped_refptr<NativeHandle> native_handle() const override;
void* native_handle() const override;
private:
friend class rtc::RefCountedObject<TextureBuffer>;
~TextureBuffer() override;
const rtc::scoped_refptr<NativeHandle> native_handle_;
// |native_handle_| is a raw pointer and not owned by TextureBuffer.
void* native_handle_;
const int width_;
const int height_;
rtc::Callback0<void> no_longer_used_cb_;
};
class WrappedI420Buffer : public webrtc::VideoFrameBuffer {
@ -125,7 +128,7 @@ class WrappedI420Buffer : public webrtc::VideoFrameBuffer {
uint8_t* data(PlaneType type) override;
int stride(PlaneType type) const override;
rtc::scoped_refptr<NativeHandle> native_handle() const override;
void* native_handle() const override;
private:
friend class rtc::RefCountedObject<WrappedI420Buffer>;

23
webrtc/common_video/video_frame_buffer.cc
View File

@ -10,6 +10,7 @@
#include "webrtc/common_video/interface/video_frame_buffer.h"
#include "webrtc/base/bind.h"
#include "webrtc/base/checks.h"
// Aligning pointer to 64 bytes for improved performance, e.g. use SIMD.
@ -89,21 +90,25 @@ int I420Buffer::stride(PlaneType type) const {
}
}
rtc::scoped_refptr<NativeHandle> I420Buffer::native_handle() const {
void* I420Buffer::native_handle() const {
return nullptr;
}
TextureBuffer::TextureBuffer(
const rtc::scoped_refptr<NativeHandle>& native_handle,
int width,
int height)
: native_handle_(native_handle), width_(width), height_(height) {
DCHECK(native_handle.get());
TextureBuffer::TextureBuffer(void* native_handle,
int width,
int height,
const rtc::Callback0<void>& no_longer_used)
: native_handle_(native_handle),
width_(width),
height_(height),
no_longer_used_cb_(no_longer_used) {
DCHECK(native_handle != nullptr);
DCHECK_GT(width, 0);
DCHECK_GT(height, 0);
}
TextureBuffer::~TextureBuffer() {
no_longer_used_cb_();
}
int TextureBuffer::width() const {
@ -129,7 +134,7 @@ int TextureBuffer::stride(PlaneType type) const {
return 0;
}
rtc::scoped_refptr<NativeHandle> TextureBuffer::native_handle() const {
void* TextureBuffer::native_handle() const {
return native_handle_;
}
@ -211,7 +216,7 @@ int WrappedI420Buffer::stride(PlaneType type) const {
}
}
rtc::scoped_refptr<NativeHandle> WrappedI420Buffer::native_handle() const {
void* WrappedI420Buffer::native_handle() const {
return nullptr;
}

32
webrtc/video/video_send_stream_tests.cc
View File

@ -12,11 +12,11 @@
#include "testing/gtest/include/gtest/gtest.h"
#include "webrtc/base/bind.h"
#include "webrtc/base/checks.h"
#include "webrtc/base/scoped_ptr.h"
#include "webrtc/call.h"
#include "webrtc/common_video/interface/i420_video_frame.h"
#include "webrtc/common_video/interface/native_handle.h"
#include "webrtc/frame_callback.h"
#include "webrtc/modules/rtp_rtcp/interface/rtp_header_parser.h"
#include "webrtc/modules/rtp_rtcp/interface/rtp_rtcp.h"
@ -50,13 +50,16 @@ void ExpectEqualFramesVector(const std::vector<I420VideoFrame>& frames1,
const std::vector<I420VideoFrame>& frames2);
I420VideoFrame CreateI420VideoFrame(int width, int height, uint8_t data);
class FakeNativeHandle : public NativeHandle {
class FakeNativeHandle {
public:
FakeNativeHandle() {}
virtual ~FakeNativeHandle() {}
virtual void* GetHandle() { return nullptr; }
~FakeNativeHandle() {}
};
void DeleteNativeHandle(FakeNativeHandle* handle) {
delete handle;
}
class VideoSendStreamTest : public test::CallTest {
protected:
void TestNackRetransmission(uint32_t retransmit_ssrc,
@ -1074,17 +1077,20 @@ TEST_F(VideoSendStreamTest, CapturesTextureAndI420VideoFrames) {
std::vector<I420VideoFrame> input_frames;
int width = static_cast<int>(encoder_config_.streams[0].width);
int height = static_cast<int>(encoder_config_.streams[0].height);
webrtc::RefCountImpl<FakeNativeHandle>* handle1 =
new webrtc::RefCountImpl<FakeNativeHandle>();
webrtc::RefCountImpl<FakeNativeHandle>* handle2 =
new webrtc::RefCountImpl<FakeNativeHandle>();
webrtc::RefCountImpl<FakeNativeHandle>* handle3 =
new webrtc::RefCountImpl<FakeNativeHandle>();
input_frames.push_back(I420VideoFrame(handle1, width, height, 1, 1));
input_frames.push_back(I420VideoFrame(handle2, width, height, 2, 2));
FakeNativeHandle* handle1 = new FakeNativeHandle();
FakeNativeHandle* handle2 = new FakeNativeHandle();
FakeNativeHandle* handle3 = new FakeNativeHandle();
input_frames.push_back(
I420VideoFrame(handle1, width, height, 1, 1, kVideoRotation_0,
rtc::Bind(&DeleteNativeHandle, handle1)));
input_frames.push_back(
I420VideoFrame(handle2, width, height, 2, 2, kVideoRotation_0,
rtc::Bind(&DeleteNativeHandle, handle2)));
input_frames.push_back(CreateI420VideoFrame(width, height, 3));
input_frames.push_back(CreateI420VideoFrame(width, height, 4));
input_frames.push_back(I420VideoFrame(handle3, width, height, 5, 5));
input_frames.push_back(
I420VideoFrame(handle3, width, height, 5, 5, kVideoRotation_0,
rtc::Bind(&DeleteNativeHandle, handle3)));
send_stream_->Start();
for (size_t i = 0; i < input_frames.size(); i++) {

30
webrtc/video_engine/vie_capturer_unittest.cc
View File

@ -18,7 +18,6 @@
#include "testing/gtest/include/gtest/gtest.h"
#include "webrtc/base/scoped_ptr.h"
#include "webrtc/common.h"
#include "webrtc/common_video/interface/native_handle.h"
#include "webrtc/modules/utility/interface/mock/mock_process_thread.h"
#include "webrtc/modules/video_capture/include/mock/mock_video_capture.h"
#include "webrtc/system_wrappers/interface/critical_section_wrapper.h"
@ -48,13 +47,6 @@ bool EqualFramesVector(const ScopedVector<I420VideoFrame>& frames1,
const ScopedVector<I420VideoFrame>& frames2);
I420VideoFrame* CreateI420VideoFrame(uint8_t length);
class FakeNativeHandle : public NativeHandle {
public:
FakeNativeHandle() {}
virtual ~FakeNativeHandle() {}
virtual void* GetHandle() { return NULL; }
};
class ViECapturerTest : public ::testing::Test {
protected:
ViECapturerTest()
@ -181,13 +173,14 @@ TEST_F(ViECapturerTest, TestRtpTimeStampSet) {
TEST_F(ViECapturerTest, TestTextureFrames) {
const int kNumFrame = 3;
for (int i = 0 ; i < kNumFrame; ++i) {
webrtc::RefCountImpl<FakeNativeHandle>* handle =
new webrtc::RefCountImpl<FakeNativeHandle>();
void* dummy_handle = reinterpret_cast<void*>(i+1);
// Add one to |i| so that width/height > 0.
input_frames_.push_back(
new I420VideoFrame(handle, i + 1, i + 1, i + 1, i + 1));
new I420VideoFrame(dummy_handle, i + 1, i + 1, i + 1, i + 1,
webrtc::kVideoRotation_0, rtc::Callback0<void>()));
AddInputFrame(input_frames_[i]);
WaitOutputFrame();
EXPECT_EQ(dummy_handle, output_frames_[i]->native_handle());
}
EXPECT_TRUE(EqualFramesVector(input_frames_, output_frames_));
@ -211,11 +204,13 @@ TEST_F(ViECapturerTest, TestI420Frames) {
}
TEST_F(ViECapturerTest, TestI420FrameAfterTextureFrame) {
webrtc::RefCountImpl<FakeNativeHandle>* handle =
new webrtc::RefCountImpl<FakeNativeHandle>();
input_frames_.push_back(new I420VideoFrame(handle, 1, 1, 1, 1));
void* dummy_handle = &input_frames_;
input_frames_.push_back(new I420VideoFrame(dummy_handle, 1, 1, 1, 1,
webrtc::kVideoRotation_0,
rtc::Callback0<void>()));
AddInputFrame(input_frames_[0]);
WaitOutputFrame();
EXPECT_EQ(dummy_handle, output_frames_[0]->native_handle());
input_frames_.push_back(CreateI420VideoFrame(2));
AddInputFrame(input_frames_[1]);
@ -229,9 +224,10 @@ TEST_F(ViECapturerTest, TestTextureFrameAfterI420Frame) {
AddInputFrame(input_frames_[0]);
WaitOutputFrame();
webrtc::RefCountImpl<FakeNativeHandle>* handle =
new webrtc::RefCountImpl<FakeNativeHandle>();
input_frames_.push_back(new I420VideoFrame(handle, 1, 1, 2, 2));
void* dummy_handle = &input_frames_;
input_frames_.push_back(new I420VideoFrame(dummy_handle, 1, 1, 2, 2,
webrtc::kVideoRotation_0,
rtc::Callback0<void>()));
AddInputFrame(input_frames_[1]);
WaitOutputFrame();

7
webrtc/video_frame.h
View File

@ -12,7 +12,6 @@
#define WEBRTC_VIDEO_FRAME_H_
#include "webrtc/base/scoped_ref_ptr.h"
#include "webrtc/common_video/interface/native_handle.h"
#include "webrtc/common_video/interface/video_frame_buffer.h"
#include "webrtc/common_video/rotation.h"
#include "webrtc/typedefs.h"
@ -26,11 +25,13 @@ class I420VideoFrame {
uint32_t timestamp,
int64_t render_time_ms,
VideoRotation rotation);
I420VideoFrame(NativeHandle* handle,
I420VideoFrame(void* native_handle,
int width,
int height,
uint32_t timestamp,
int64_t render_time_ms);
int64_t render_time_ms,
VideoRotation rotation,
const rtc::Callback0<void>& no_longer_used);
// TODO(pbos): Make all create/copy functions void, they should not be able to
// fail (which should be DCHECK/CHECKed instead).