Handle frames sent from peer connection in VideoTrackRecorder

Handle frames sent from peer connection in VideoTrackRecorder

This CL changes the way we handle frames received from peer connection in
VideoTrackRecorder. If we forward those frames directly to GPU process, they
might not be accessible if they aren't backed by STORAGE_SHMEM. We need
to copy those frames into new buffers for these cases.

BUG=644526
TEST=Added new content_browsertest which fails before the changes.

Committed: https://crrev.com/83287174008c789d2e3ed450fa5e2a4236dd5040
Cr-Commit-Position: refs/heads/master@{#416983}

[emircan, 2016-09-07 00:14:59]

Description was changed from

==========
vt rec

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

to

==========
Handle frames sent from peer connection in VideoTrackRecorder

BUG=644526
==========

[emircan, 2016-09-07 00:14:59]

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

[emircan, 2016-09-07 00:42:49]

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

[commit-bot: I haz the power, 2016-09-07 00:43:13]

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

[emircan, 2016-09-07 00:44:21]

Description was changed from

==========
Handle frames sent from peer connection in VideoTrackRecorder

BUG=644526
==========

to

==========
Handle frames sent from peer connection in VideoTrackRecorder

This CL changes the way we handle frames received from peer connection in 
VideoTrackRecorder. If we forward those frames directly to GPU process, they
might not be accessible if they aren't backed by STORAGE_SHMEM. We need
to copy those frames into new buffers for these cases.

BUG=644526
TEST=Added new content_browsertest which fails before the changes.
==========

[emircan, 2016-09-07 00:44:28]

Patchset #1 (id:1) has been deleted

[emircan, 2016-09-07 00:44:34]

PTAL.

[commit-bot: I haz the power, 2016-09-07 02:08:20]

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

[commit-bot: I haz the power, 2016-09-07 02:08:21]

Dry run: This issue passed the CQ dry run.

[mcasas, 2016-09-07 16:07:59]

lgtm

[emircan, 2016-09-07 17:07:35]

The CQ bit was checked by emircan@chromium.org

[commit-bot: I haz the power, 2016-09-07 17:07:58]

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-09-07 17:14:16]

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

[commit-bot: I haz the power, 2016-09-07 17:14:17]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[emircan, 2016-09-07 17:31:12]

emircan@chromium.org changed reviewers:
+ avi@chromium.org

[emircan, 2016-09-07 17:31:12]

avi@chromium.org: Please RS review changes in
content/browser/webrtc/webrtc_media_recorder_browsertest.cc

[Avi (use Gerrit), 2016-09-07 17:33:58]

lgtm

stampity stamp

[emircan, 2016-09-07 17:35:13]

The CQ bit was checked by emircan@chromium.org

[commit-bot: I haz the power, 2016-09-07 17:35:41]

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-09-07 17:39:52]

Description was changed from

==========
Handle frames sent from peer connection in VideoTrackRecorder

This CL changes the way we handle frames received from peer connection in 
VideoTrackRecorder. If we forward those frames directly to GPU process, they
might not be accessible if they aren't backed by STORAGE_SHMEM. We need
to copy those frames into new buffers for these cases.

BUG=644526
TEST=Added new content_browsertest which fails before the changes.
==========

to

==========
Handle frames sent from peer connection in VideoTrackRecorder

This CL changes the way we handle frames received from peer connection in 
VideoTrackRecorder. If we forward those frames directly to GPU process, they
might not be accessible if they aren't backed by STORAGE_SHMEM. We need
to copy those frames into new buffers for these cases.

BUG=644526
TEST=Added new content_browsertest which fails before the changes.
==========

[commit-bot: I haz the power, 2016-09-07 17:39:53]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-09-07 17:43:16]

Description was changed from

==========
Handle frames sent from peer connection in VideoTrackRecorder

This CL changes the way we handle frames received from peer connection in 
VideoTrackRecorder. If we forward those frames directly to GPU process, they
might not be accessible if they aren't backed by STORAGE_SHMEM. We need
to copy those frames into new buffers for these cases.

BUG=644526
TEST=Added new content_browsertest which fails before the changes.
==========

to

==========
Handle frames sent from peer connection in VideoTrackRecorder

This CL changes the way we handle frames received from peer connection in
VideoTrackRecorder. If we forward those frames directly to GPU process, they
might not be accessible if they aren't backed by STORAGE_SHMEM. We need
to copy those frames into new buffers for these cases.

BUG=644526
TEST=Added new content_browsertest which fails before the changes.

Committed: https://crrev.com/83287174008c789d2e3ed450fa5e2a4236dd5040
Cr-Commit-Position: refs/heads/master@{#416983}
==========

[commit-bot: I haz the power, 2016-09-07 17:43:18]

Patchset 1 (id:??) landed as
https://crrev.com/83287174008c789d2e3ed450fa5e2a4236dd5040
Cr-Commit-Position: refs/heads/master@{#416983}

[emircan, 2016-09-08 23:10:32]

Description was changed from

==========
Handle frames sent from peer connection in VideoTrackRecorder

This CL changes the way we handle frames received from peer connection in
VideoTrackRecorder. If we forward those frames directly to GPU process, they
might not be accessible if they aren't backed by STORAGE_SHMEM. We need
to copy those frames into new buffers for these cases.

BUG=644526
TEST=Added new content_browsertest which fails before the changes.

Committed: https://crrev.com/83287174008c789d2e3ed450fa5e2a4236dd5040
Cr-Commit-Position: refs/heads/master@{#416983}
==========

to

==========
Handle frames sent from peer connection in VideoTrackRecorder

This CL changes the way we handle frames received from peer connection in
VideoTrackRecorder. If we forward those frames directly to GPU process, they
might not be accessible if they aren't backed by STORAGE_SHMEM. We need
to copy those frames into new buffers for these cases.

BUG=644526
TEST=Added new content_browsertest which fails before the changes.

Committed: https://crrev.com/83287174008c789d2e3ed450fa5e2a4236dd5040
Cr-Commit-Position: refs/heads/master@{#416983}

NOTRY=true
NOPRESUBMIT=true
TBR=avi@chromium.org,mcasas@chromium.org
==========

[emircan, 2016-09-08 23:13:08]

Description was changed from

==========
Handle frames sent from peer connection in VideoTrackRecorder

This CL changes the way we handle frames received from peer connection in
VideoTrackRecorder. If we forward those frames directly to GPU process, they
might not be accessible if they aren't backed by STORAGE_SHMEM. We need
to copy those frames into new buffers for these cases.

BUG=644526
TEST=Added new content_browsertest which fails before the changes.

Committed: https://crrev.com/83287174008c789d2e3ed450fa5e2a4236dd5040
Cr-Commit-Position: refs/heads/master@{#416983}

NOTRY=true
NOPRESUBMIT=true
TBR=avi@chromium.org,mcasas@chromium.org
==========

to

==========
Handle frames sent from peer connection in VideoTrackRecorder

This CL changes the way we handle frames received from peer connection in
VideoTrackRecorder. If we forward those frames directly to GPU process, they
might not be accessible if they aren't backed by STORAGE_SHMEM. We need
to copy those frames into new buffers for these cases.

BUG=644526
TEST=Added new content_browsertest which fails before the changes.

Committed: https://crrev.com/83287174008c789d2e3ed450fa5e2a4236dd5040
Cr-Commit-Position: refs/heads/master@{#416983}
==========