Revert of [Mojo Video Capture] Simplify media::VideoCaptureDevice::Client:Buffer to a struct

Revert of [Mojo Video Capture] Simplify media::VideoCaptureDevice::Client:Buffer to a struct (patchset #8 id:220001 of https://codereview.chromium.org/2573223002/ )

Reason for revert:
In order to revert https://codereview.chromium.org/2592303002/, need to revert this first, since it depends on it.

Original issue's description:
> [Mojo Video Capture] Simplify media::VideoCaptureDevice::Client:Buffer to a struct
> 
> In interface media::VideoCaptureDevice::Client, change interface Buffer to a 
> move-only struct. The new struct separates the concerns of providing access 
> handles (for in-process as well as for inter-process transit) from access 
> permissions. This allows clients to explicitly obtain in-process memory-mapped 
> handles when they need it. Clients who only want to pass along the buffer or 
> move it across process boundaries do not need memory-mapped handles.
> 
> In interface media::VideoCaptureBufferPool, renamed 
>  GetBufferHandle() -> GetHandleForInProcessAccess()
>  GetHandleForTransit() -> GetHandleForInterProcessTransit().
> 
> Removed outdated implementation in video_capture Mojo service. We leave the 
> implementation empty, until refactoring the interfaces has been completed. 
> 
> This CL is part of the Mojo Video Capture work. For the bigger picture, 
> see [1] CL1.9.10
> 
> For a peek at the follow-up CLs, please see 
> https://codereview.chromium.org/2607203002/
> https://codereview.chromium.org/2602983002/
> 
> BUG=584797
> TEST=
>   content_unittests --gtest_filter="*Video*", 
>   video_capture_unittests, 
>   Apprtc loopback on Debug, 
>   Desktop Capture Example extension on Release
> 
> [1] https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxsxWslJ_X-zpOhijvI8/edit?usp=sharing
> 
> Review-Url: https://codereview.chromium.org/2573223002
> Cr-Commit-Position: refs/heads/master@{#442066}
> Committed: https://chromium.googlesource.com/chromium/src/+/e1aac992b0399db7776661f4ac2c43094c265de7

TBR=miu@chromium.org,mcasas@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584797
Review-Url: https://codereview.chromium.org/2613793007
Cr-Commit-Position: refs/heads/master@{#442130}
Committed: https://chromium.googlesource.com/chromium/src/+/355fef64c80c70bf0787534f029ee84c7e58d171

[chfremer, 2017-01-07 01:04:56]

The CQ bit was checked by chfremer@chromium.org

[chfremer, 2017-01-07 01:04:56]

Created Revert of [Mojo Video Capture] Simplify
media::VideoCaptureDevice::Client:Buffer to a struct

[commit-bot: I haz the power, 2017-01-07 01:05:28]

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, 2017-01-07 01:21:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483751096431930, "parent_rev":
"a15badc41c7fa19740dfcff4075874c75af119bd", "commit_rev":
"355fef64c80c70bf0787534f029ee84c7e58d171"}

[commit-bot: I haz the power, 2017-01-07 01:21:44]

Description was changed from

==========
Revert of [Mojo Video Capture] Simplify media::VideoCaptureDevice::Client:Buffer
to a struct (patchset #8 id:220001 of
https://codereview.chromium.org/2573223002/ )

Reason for revert:
In order to revert https://codereview.chromium.org/2592303002/, need to revert
this first, since it depends on it.

Original issue's description:
> [Mojo Video Capture] Simplify media::VideoCaptureDevice::Client:Buffer to a
struct
> 
> In interface media::VideoCaptureDevice::Client, change interface Buffer to a 
> move-only struct. The new struct separates the concerns of providing access 
> handles (for in-process as well as for inter-process transit) from access 
> permissions. This allows clients to explicitly obtain in-process memory-mapped

> handles when they need it. Clients who only want to pass along the buffer or 
> move it across process boundaries do not need memory-mapped handles.
> 
> In interface media::VideoCaptureBufferPool, renamed 
>  GetBufferHandle() -> GetHandleForInProcessAccess()
>  GetHandleForTransit() -> GetHandleForInterProcessTransit().
> 
> Removed outdated implementation in video_capture Mojo service. We leave the 
> implementation empty, until refactoring the interfaces has been completed. 
> 
> This CL is part of the Mojo Video Capture work. For the bigger picture, 
> see [1] CL1.9.10
> 
> For a peek at the follow-up CLs, please see 
> https://codereview.chromium.org/2607203002/
> https://codereview.chromium.org/2602983002/
> 
> BUG=584797
> TEST=
>   content_unittests --gtest_filter="*Video*", 
>   video_capture_unittests, 
>   Apprtc loopback on Debug, 
>   Desktop Capture Example extension on Release
> 
> [1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
> 
> Review-Url: https://codereview.chromium.org/2573223002
> Cr-Commit-Position: refs/heads/master@{#442066}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e1aac992b0399db7776661f4ac2c...

TBR=miu@chromium.org,mcasas@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584797
==========

to

==========
Revert of [Mojo Video Capture] Simplify media::VideoCaptureDevice::Client:Buffer
to a struct (patchset #8 id:220001 of
https://codereview.chromium.org/2573223002/ )

Reason for revert:
In order to revert https://codereview.chromium.org/2592303002/, need to revert
this first, since it depends on it.

Original issue's description:
> [Mojo Video Capture] Simplify media::VideoCaptureDevice::Client:Buffer to a
struct
> 
> In interface media::VideoCaptureDevice::Client, change interface Buffer to a 
> move-only struct. The new struct separates the concerns of providing access 
> handles (for in-process as well as for inter-process transit) from access 
> permissions. This allows clients to explicitly obtain in-process memory-mapped

> handles when they need it. Clients who only want to pass along the buffer or 
> move it across process boundaries do not need memory-mapped handles.
> 
> In interface media::VideoCaptureBufferPool, renamed 
>  GetBufferHandle() -> GetHandleForInProcessAccess()
>  GetHandleForTransit() -> GetHandleForInterProcessTransit().
> 
> Removed outdated implementation in video_capture Mojo service. We leave the 
> implementation empty, until refactoring the interfaces has been completed. 
> 
> This CL is part of the Mojo Video Capture work. For the bigger picture, 
> see [1] CL1.9.10
> 
> For a peek at the follow-up CLs, please see 
> https://codereview.chromium.org/2607203002/
> https://codereview.chromium.org/2602983002/
> 
> BUG=584797
> TEST=
>   content_unittests --gtest_filter="*Video*", 
>   video_capture_unittests, 
>   Apprtc loopback on Debug, 
>   Desktop Capture Example extension on Release
> 
> [1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
> 
> Review-Url: https://codereview.chromium.org/2573223002
> Cr-Commit-Position: refs/heads/master@{#442066}
> Committed:
https://chromium.googlesource.com/chromium/src/+/e1aac992b0399db7776661f4ac2c...

TBR=miu@chromium.org,mcasas@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=584797

Review-Url: https://codereview.chromium.org/2613793007
Cr-Commit-Position: refs/heads/master@{#442130}
Committed:
https://chromium.googlesource.com/chromium/src/+/355fef64c80c70bf0787534f029e...
==========

[commit-bot: I haz the power, 2017-01-07 01:21:45]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/355fef64c80c70bf0787534f029e...