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

Unified Diff: content/browser/renderer_host/media/video_capture_controller.cc

Issue 1439533004: Remove dead code paths around PIXEL_STORAGE_TEXTURE in capture pipeline. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/media/video_capture_controller.cc
diff --git a/content/browser/renderer_host/media/video_capture_controller.cc b/content/browser/renderer_host/media/video_capture_controller.cc
index 01a77723e4ea50b4a12734baeffe1f3a06f384b7..f29c1abd18fe368b7edad12dc0dab800426150b1 100644
--- a/content/browser/renderer_host/media/video_capture_controller.cc
+++ b/content/browser/renderer_host/media/video_capture_controller.cc
@@ -19,7 +19,6 @@
#include "content/common/gpu/client/gl_helper.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/common/content_switches.h"
-#include "gpu/command_buffer/common/mailbox_holder.h"
#include "media/base/video_frame.h"
#if !defined(OS_ANDROID)
@@ -438,7 +437,8 @@ void VideoCaptureController::DoNewBufferOnIOThread(
DCHECK_EQ(frame->format(), media::PIXEL_FORMAT_I420);
if (frame->storage_type() == media::VideoFrame::STORAGE_GPU_MEMORY_BUFFERS) {
std::vector<gfx::GpuMemoryBufferHandle> handles;
- for (size_t i = 0; i < media::VideoFrame::NumPlanes(frame->format()); ++i) {
+ const size_t num_planes = media::VideoFrame::NumPlanes(frame->format());
+ for (size_t i = 0; i < num_planes; ++i) {
gfx::GpuMemoryBufferHandle remote_handle;
buffer_pool_->ShareToProcess2(buffer_id, i, client->render_process_handle,
&remote_handle);

Powered by Google App Engine
This is Rietveld 408576698