[M-52] Close bitstream buffer shared memory handles on Android.

[M-52] Close bitstream buffer shared memory handles on Android.

AndroidVideoDecodeAccelerator would clear references to pending
bitstream buffers in ::Reset and, implicitly, in the destructor.
However, each of those may have a file descriptor for shared memory
that isn't closed.  During a seek, for example, this can cause leaks.

This CL wraps the queue of pending bitstream buffers with a new
BitstreamRecord type, which has a unique_ptr to the (unmapped)
shared memory region, if any.  This will automatically close the
underlying handle when the unique_ptr is released.  Note that no
expensive operation is performed in creating the shared memory
region.  Unless it is later mapped, it just keeps the handle
and ::close()s it on destruction.

This is a merge of https://codereview.chromium.org/2011653002 .

BUG=614489
TEST=observed /proc/fd/[gpu process] while seeking in a video
NOPRESUBMIT=true
NOTRY=true
TBR=dalecurtis@chromium.org

[liberato (no reviews please), 2016-05-25 21:47:24]

The CQ bit was checked by liberato@chromium.org

[commit-bot: I haz the power, 2016-05-25 21:48:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2008373003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2008373003/1

[commit-bot: I haz the power, 2016-05-25 21:48:01]

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

[commit-bot: I haz the power, 2016-05-25 21:48:02]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[liberato (no reviews please), 2016-05-25 21:51:47]

Description was changed from

==========
[M-52] Close bitstream buffer shared memory handles on Android.

AndroidVideoDecodeAccelerator would clear references to pending
bitstream buffers in ::Reset and, implicitly, in the destructor.
However, each of those may have a file descriptor for shared memory
that isn't closed.  During a seek, for example, this can cause leaks.

This CL wraps the queue of pending bitstream buffers with a new
BitstreamRecord type, which has a unique_ptr to the (unmapped)
shared memory region, if any.  This will automatically close the
underlying handle when the unique_ptr is released.  Note that no
expensive operation is performed in creating the shared memory
region.  Unless it is later mapped, it just keeps the handle
and ::close()s it on destruction.

This is a merge of https://codereview.chromium.org/2011653002 .

BUG=614489
TEST=observed /proc/fd/[gpu process] while seeking in a video
NOPRESUBMIT=true
NOTRY=true
==========

to

==========
[M-52] Close bitstream buffer shared memory handles on Android.

AndroidVideoDecodeAccelerator would clear references to pending
bitstream buffers in ::Reset and, implicitly, in the destructor.
However, each of those may have a file descriptor for shared memory
that isn't closed.  During a seek, for example, this can cause leaks.

This CL wraps the queue of pending bitstream buffers with a new
BitstreamRecord type, which has a unique_ptr to the (unmapped)
shared memory region, if any.  This will automatically close the
underlying handle when the unique_ptr is released.  Note that no
expensive operation is performed in creating the shared memory
region.  Unless it is later mapped, it just keeps the handle
and ::close()s it on destruction.

This is a merge of https://codereview.chromium.org/2011653002 .

BUG=614489
TEST=observed /proc/fd/[gpu process] while seeking in a video
NOPRESUBMIT=true
NOTRY=true
TBR=dalecurtis@chromium.org
==========

[liberato (no reviews please), 2016-05-25 21:51:51]

The CQ bit was checked by liberato@chromium.org

[commit-bot: I haz the power, 2016-05-25 21:52:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2008373003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2008373003/1

[commit-bot: I haz the power, 2016-05-25 21:54:41]

Description was changed from

==========
[M-52] Close bitstream buffer shared memory handles on Android.

AndroidVideoDecodeAccelerator would clear references to pending
bitstream buffers in ::Reset and, implicitly, in the destructor.
However, each of those may have a file descriptor for shared memory
that isn't closed.  During a seek, for example, this can cause leaks.

This CL wraps the queue of pending bitstream buffers with a new
BitstreamRecord type, which has a unique_ptr to the (unmapped)
shared memory region, if any.  This will automatically close the
underlying handle when the unique_ptr is released.  Note that no
expensive operation is performed in creating the shared memory
region.  Unless it is later mapped, it just keeps the handle
and ::close()s it on destruction.

This is a merge of https://codereview.chromium.org/2011653002 .

BUG=614489
TEST=observed /proc/fd/[gpu process] while seeking in a video
NOPRESUBMIT=true
NOTRY=true
TBR=dalecurtis@chromium.org
==========

to

==========
[M-52] Close bitstream buffer shared memory handles on Android.

AndroidVideoDecodeAccelerator would clear references to pending
bitstream buffers in ::Reset and, implicitly, in the destructor.
However, each of those may have a file descriptor for shared memory
that isn't closed.  During a seek, for example, this can cause leaks.

This CL wraps the queue of pending bitstream buffers with a new
BitstreamRecord type, which has a unique_ptr to the (unmapped)
shared memory region, if any.  This will automatically close the
underlying handle when the unique_ptr is released.  Note that no
expensive operation is performed in creating the shared memory
region.  Unless it is later mapped, it just keeps the handle
and ::close()s it on destruction.

This is a merge of https://codereview.chromium.org/2011653002 .

BUG=614489
TEST=observed /proc/fd/[gpu process] while seeking in a video
NOPRESUBMIT=true
NOTRY=true
TBR=dalecurtis@chromium.org
==========

[commit-bot: I haz the power, 2016-05-25 21:54:42]

Committed patchset #1 (id:1)