Move NV21 colorspace treatment from VideoCapture java to C++

content/browser/renderer_host/media/video_capture_controller.h

Index: content/browser/renderer_host/media/video_capture_controller.h
diff --git a/content/browser/renderer_host/media/video_capture_controller.h b/content/browser/renderer_host/media/video_capture_controller.h
index 5d33d01163c62c5f661ec523ac552b81a379a65b..a94178ceb7c7fe0b05a53640e57496f60b63e699 100644
--- a/content/browser/renderer_host/media/video_capture_controller.h
+++ b/content/browser/renderer_host/media/video_capture_controller.h
@@ -156,6 +156,10 @@ class CONTENT_EXPORT VideoCaptureController
   bool device_in_use_;
   VideoCaptureState state_;
 
+  // For NV21 we have to do color conversion into the intermediate buffer and
+  // from there the rotations.
+  scoped_ptr<uint8[]> i420_intermediate_buffer_;

[bulach, 2013/09/11 14:59:07]

nit: shouldn't this be a scoped_array instead?
or even better, how about just a vector<uint8> ?

[ncarter (slow), 2013/09/11 20:07:23]

FWIW This looks fine to me. scoped_array doesn't even exist anymore,  scoped_ptr
of an array type is the preferred idiom nowadays. (
https://codereview.chromium.org/14081006 was where scoped_array was removed from
chromium)

+
   DISALLOW_IMPLICIT_CONSTRUCTORS(VideoCaptureController);
 };