summaryrefslogtreecommitdiff
path: root/video_engine/vie_capturer.cc
diff options
context:
space:
mode:
authorwuchengli@chromium.org <wuchengli@chromium.org@4adac7df-926f-26a2-2b94-8c16560cd09d>2014-05-27 14:12:58 +0000
committerwuchengli@chromium.org <wuchengli@chromium.org@4adac7df-926f-26a2-2b94-8c16560cd09d>2014-05-27 14:12:58 +0000
commit5424828677e7e04c9eb433a76788e3b0f7378286 (patch)
treee0f4484c2a4db7d36575d030d50657857452de93 /video_engine/vie_capturer.cc
parenta3b8c852fe775a455c29d436c73163db662b7a6d (diff)
downloadwebrtc-5424828677e7e04c9eb433a76788e3b0f7378286.tar.gz
Revert "Add support of texture frames for video capturer."
This reverts commit 83c89cd003be75d7d06ef9a2b139588f08d280ca. Reason: The Buildbot has detected a new failure on builder Android Chromium-APK Tests. BUG=chromium:362437 TBR=fischman@webrtc.org, perkj@webrtc.org, stefan@webrtc.org, wu@webrtc.org Review URL: https://webrtc-codereview.appspot.com/12599004 git-svn-id: http://webrtc.googlecode.com/svn/trunk/webrtc@6253 4adac7df-926f-26a2-2b94-8c16560cd09d
Diffstat (limited to 'video_engine/vie_capturer.cc')
-rw-r--r--video_engine/vie_capturer.cc39
1 files changed, 8 insertions, 31 deletions
diff --git a/video_engine/vie_capturer.cc b/video_engine/vie_capturer.cc
index 30d66335..867de9b1 100644
--- a/video_engine/vie_capturer.cc
+++ b/video_engine/vie_capturer.cc
@@ -10,7 +10,6 @@
#include "webrtc/video_engine/vie_capturer.h"
-#include "webrtc/common_video/interface/texture_video_frame.h"
#include "webrtc/common_video/libyuv/include/webrtc_libyuv.h"
#include "webrtc/modules/interface/module_common_types.h"
#include "webrtc/modules/utility/interface/process_thread.h"
@@ -347,16 +346,11 @@ void ViECapturer::OnIncomingCapturedFrame(const int32_t capture_id,
TRACE_EVENT_ASYNC_BEGIN1("webrtc", "Video", video_frame.render_time_ms(),
"render_time", video_frame.render_time_ms());
- if (video_frame.native_handle() != NULL) {
- captured_frame_.reset(video_frame.CloneFrame());
- } else {
- if (captured_frame_ == NULL || captured_frame_->native_handle() != NULL)
- captured_frame_.reset(new I420VideoFrame());
- captured_frame_->SwapFrame(&video_frame);
- }
+ captured_frame_.SwapFrame(&video_frame);
capture_event_.Set();
- overuse_detector_->FrameCaptured(captured_frame_->width(),
- captured_frame_->height());
+ overuse_detector_->FrameCaptured(captured_frame_.width(),
+ captured_frame_.height());
+ return;
}
void ViECapturer::OnCaptureDelayChanged(const int32_t id,
@@ -479,9 +473,7 @@ bool ViECapturer::ViECaptureProcess() {
deliver_cs_->Enter();
if (SwapCapturedAndDeliverFrameIfAvailable()) {
encode_start_time = Clock::GetRealTimeClock()->TimeInMilliseconds();
- DeliverI420Frame(deliver_frame_.get());
- if (deliver_frame_->native_handle() != NULL)
- deliver_frame_.reset(); // Release the texture so it can be reused.
+ DeliverI420Frame(&deliver_frame_);
}
deliver_cs_->Leave();
if (current_brightness_level_ != reported_brightness_level_) {
@@ -502,11 +494,6 @@ bool ViECapturer::ViECaptureProcess() {
}
void ViECapturer::DeliverI420Frame(I420VideoFrame* video_frame) {
- if (video_frame->native_handle() != NULL) {
- ViEFrameProviderBase::DeliverFrame(video_frame);
- return;
- }
-
// Apply image enhancement and effect filter.
if (deflicker_frame_stats_) {
if (image_proc_module_->GetFrameStats(deflicker_frame_stats_,
@@ -621,21 +608,11 @@ void ViECapturer::OnNoPictureAlarm(const int32_t id,
bool ViECapturer::SwapCapturedAndDeliverFrameIfAvailable() {
CriticalSectionScoped cs(capture_cs_.get());
- if (captured_frame_ == NULL)
- return false;
-
- if (captured_frame_->native_handle() != NULL) {
- deliver_frame_.reset(captured_frame_.release());
- return true;
- }
-
- if (captured_frame_->IsZeroSize())
+ if (captured_frame_.IsZeroSize())
return false;
- if (deliver_frame_ == NULL)
- deliver_frame_.reset(new I420VideoFrame());
- deliver_frame_->SwapFrame(captured_frame_.get());
- captured_frame_->ResetSize();
+ deliver_frame_.SwapFrame(&captured_frame_);
+ captured_frame_.ResetSize();
return true;
}