16 bit capture and GPU&CPU memory buffer support.

16 bit capture and GPU&CPU memory buffer support.

Windows, Linux and ChromeOS support.

BUG=624436
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

[aleksandar.stojiljkovic, 2016-10-05 04:04:53]

Description was changed from

==========
16 bit capture and GPU&CPU memory buffer support.

Windows, Linux and ChromeOS support.

BUG=624436
==========

to

==========
16 bit capture and GPU&CPU memory buffer support.

Windows, Linux and ChromeOS support.

BUG=624436
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[aleksandar.stojiljkovic, 2016-10-05 04:10:09]

The CQ bit was checked by aleksandar.stojiljkovic@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-05 04:10:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-05 04:25:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-05 04:25:35]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[aleksandar.stojiljkovic, 2016-10-05 04:26:52]

aleksandar.stojiljkovic@intel.com changed reviewers:
+ mcasas@chromium.org

[aleksandar.stojiljkovic, 2016-10-05 04:26:53]

media and content code split from 2121043002: (8 and 16 bpp video stream capture
and render - Realsense R200 & SR300 support).
Patch here covers 16 bit capture pipeline all the way through to cc
video_resource_updater (where buffers are just ignored).
mcasas@, could you please review this?
Thanks.

[aleksandar.stojiljkovic, 2016-10-05 18:09:44]

aleksandar.stojiljkovic@intel.com changed reviewers:
+ hubbe@chromium.org

[aleksandar.stojiljkovic, 2016-10-05 18:09:45]

+hubbe@
for Owner review of media/video/gpu_memory_buffer_video_frame_pool.cc.
Thanks

[hubbe, 2016-10-05 18:23:22]

On 2016/10/05 18:09:45, aleksandar.stojiljkovic wrote:
> +hubbe@
> for Owner review of media/video/gpu_memory_buffer_video_frame_pool.cc.
> Thanks

I think you'd be better off sending this to dcastagna@chromium.org, he's
currently reviewing my CL that touch this file... :)

[aleksandar.stojiljkovic, 2016-10-05 18:47:33]

aleksandar.stojiljkovic@intel.com changed reviewers:
+ dcastagna@chromium.org
- hubbe@chromium.org

[aleksandar.stojiljkovic, 2016-10-05 18:47:34]

On 2016/10/05 18:23:22, hubbe wrote:
> On 2016/10/05 18:09:45, aleksandar.stojiljkovic wrote:
> > +hubbe@
> > for Owner review of media/video/gpu_memory_buffer_video_frame_pool.cc.
> > Thanks
> 
> I think you'd be better off sending this to mailto:dcastagna@chromium.org,
he's
> currently reviewing my CL that touch this file... :)
Thanks.
+dcastagna@
for Owner review of media/video/gpu_memory_buffer_video_frame_pool.cc.

[aleksandar.stojiljkovic, 2016-10-07 14:21:15]

On 2016/10/05 18:47:34, aleksandar.stojiljkovic wrote:
> On 2016/10/05 18:23:22, hubbe wrote:
> > On 2016/10/05 18:09:45, aleksandar.stojiljkovic wrote:
> > > +hubbe@
> > > for Owner review of media/video/gpu_memory_buffer_video_frame_pool.cc.
> > > Thanks
> > 
> > I think you'd be better off sending this to mailto:dcastagna@chromium.org,
> he's
> > currently reviewing my CL that touch this file... :)
> Thanks.
> +dcastagna@
> for Owner review of media/video/gpu_memory_buffer_video_frame_pool.cc.

I'm not able to fix the failing
WebRtcGetUserMediaBrowserTest.GetUserMediaWithMandatorySourceID without
rendering changes in original patch (2121043002) because it uses all fake
capture devices (also 16 bit stream). My idea of splitting 2121043002 for easier
review wasn't that good.
Closing this one and moving review to original patch.