Cleanup of passing GpuMemoryBuffer backed video frames

Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames
-- From browser to renderer
-- From renderer to gpu

I opened a bug to remove gpu::SyncToken usage on crbug.com/653994.
That is also dead code but not related to GMBs, so leaving it for a separate CL.

BUG=440843, 653579
Committed: https://crrev.com/5471ad2f678cc6e57475ed5e44cc20891917e690
Cr-Commit-Position: refs/heads/master@{#424072}

[emircan, 2016-10-07 01:03:46]

Description was changed from

==========
frame cleanup

BUG=
==========

to

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #1 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

BUG=
==========

[emircan, 2016-10-07 01:03:46]

emircan@chromium.org changed reviewers:
+ chfremer@chromium.org, mcasas@chromium.org

[emircan, 2016-10-07 01:03:53]

Patchset #1 (id:1) has been deleted

[emircan, 2016-10-07 01:04:12]

Description was changed from

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #1 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

BUG=
==========

to

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

BUG=
==========

[emircan, 2016-10-07 17:43:31]

Description was changed from

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

BUG=
==========

to

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames

BUG=440843, 653579
==========

[emircan, 2016-10-07 18:40:27]

Description was changed from

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames

BUG=440843, 653579
==========

to

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames
-- From browser to renderer
-- From renderer to gpu

Note that there still exists gpu::SyncToken

BUG=440843, 653579
==========

[emircan, 2016-10-07 18:48:39]

The CQ bit was checked by emircan@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-07 18:48:59]

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

[emircan, 2016-10-07 18:51:53]

Patchset #1 (id:20001) has been deleted

[commit-bot: I haz the power, 2016-10-07 18:52:49]

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

[commit-bot: I haz the power, 2016-10-07 18:52:50]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[emircan, 2016-10-07 18:53:11]

Description was changed from

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames
-- From browser to renderer
-- From renderer to gpu

Note that there still exists gpu::SyncToken

BUG=440843, 653579
==========

to

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames
-- From browser to renderer
-- From renderer to gpu

I opened a seperate bug to remove gpu::SyncToken usage on crbug.com/653994.
That is also dead code but not related to GMBs, so leaving it for a separate CL.

BUG=440843, 653579
==========

[emircan, 2016-10-07 19:00:26]

The CQ bit was checked by emircan@chromium.org to run a CQ dry run

[emircan, 2016-10-07 19:00:30]

Patchset #1 (id:40001) has been deleted

[emircan, 2016-10-07 19:00:59]

Description was changed from

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames
-- From browser to renderer
-- From renderer to gpu

I opened a seperate bug to remove gpu::SyncToken usage on crbug.com/653994.
That is also dead code but not related to GMBs, so leaving it for a separate CL.

BUG=440843, 653579
==========

to

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames
-- From browser to renderer
-- From renderer to gpu

I opened a bug to remove gpu::SyncToken usage on crbug.com/653994.
That is also dead code but not related to GMBs, so leaving it for a separate CL.

BUG=440843, 653579
==========

[commit-bot: I haz the power, 2016-10-07 19:01:06]

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

[emircan, 2016-10-07 19:01:14]

PTAL.

[commit-bot: I haz the power, 2016-10-07 19:04:48]

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

[commit-bot: I haz the power, 2016-10-07 19:04:50]

Dry run: Exceeded global retry quota

[chfremer, 2016-10-07 19:52:38]

lgtm % bots

[emircan, 2016-10-07 21:06:17]

The CQ bit was checked by emircan@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-07 21:07:04]

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

[mcasas, 2016-10-07 22:14:03]

On 2016/10/07 19:52:38, chfremer wrote:
> lgtm % bots

lgtm, ship it

[emircan, 2016-10-07 22:21:27]

emircan@chromium.org changed reviewers:
+ sandersd@chromium.org, tsepez@chromium.org

[emircan, 2016-10-07 22:21:28]

Adding for OWNERs review

tsepez@ can you PTAL at content/common/media/video_capture_messages.h and
media/gpu/ipc/common/media_messages.h?
sandersd@ can you PTAL at media/*?

[Tom Sepez, 2016-10-07 22:32:40]

messages LGTM.

[sandersd (OOO until July 31), 2016-10-07 22:36:58]

media/ lgtm.

[commit-bot: I haz the power, 2016-10-07 23:14:03]

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

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

Dry run: This issue passed the CQ dry run.

[emircan, 2016-10-07 23:14:35]

The CQ bit was checked by emircan@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-07 23:15:02]

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-08 02:30:41]

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

[commit-bot: I haz the power, 2016-10-08 02:30:42]

Dry run: This issue passed the CQ dry run.

[mcasas, 2016-10-08 05:50:01]

The CQ bit was checked by mcasas@chromium.org

[mcasas, 2016-10-08 05:50:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
chfremer@chromium.org, tsepez@chromium.org, mcasas@chromium.org,
sandersd@chromium.org
Link to the patchset: https://codereview.chromium.org/2395913003/#ps100001
(title: "Rebase")

[commit-bot: I haz the power, 2016-10-08 05:50:14]

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-08 05:55:52]

Description was changed from

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames
-- From browser to renderer
-- From renderer to gpu

I opened a bug to remove gpu::SyncToken usage on crbug.com/653994.
That is also dead code but not related to GMBs, so leaving it for a separate CL.

BUG=440843, 653579
==========

to

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames
-- From browser to renderer
-- From renderer to gpu

I opened a bug to remove gpu::SyncToken usage on crbug.com/653994.
That is also dead code but not related to GMBs, so leaving it for a separate CL.

BUG=440843, 653579
==========

[commit-bot: I haz the power, 2016-10-08 05:55:53]

Committed patchset #3 (id:100001)

[commit-bot: I haz the power, 2016-10-08 05:57:22]

Description was changed from

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames
-- From browser to renderer
-- From renderer to gpu

I opened a bug to remove gpu::SyncToken usage on crbug.com/653994.
That is also dead code but not related to GMBs, so leaving it for a separate CL.

BUG=440843, 653579
==========

to

==========
Cleanup of passing GpuMemoryBuffer backed video frames

This is CL #2 of 2 for cleaning up GpuMemoryBuffer usage for video
capture. This project is superseded by capture using mojo.

This CL cleans up:
- VideoFrame::StorageType enum STORAGE_GPU_MEMORY_BUFFERS
- IPC messages added for passing GMB backed frames
-- From browser to renderer
-- From renderer to gpu

I opened a bug to remove gpu::SyncToken usage on crbug.com/653994.
That is also dead code but not related to GMBs, so leaving it for a separate CL.

BUG=440843, 653579

Committed: https://crrev.com/5471ad2f678cc6e57475ed5e44cc20891917e690
Cr-Commit-Position: refs/heads/master@{#424072}
==========

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

Patchset 3 (id:??) landed as
https://crrev.com/5471ad2f678cc6e57475ed5e44cc20891917e690
Cr-Commit-Position: refs/heads/master@{#424072}