Merge 58: Avoid calling VEA::Destroy() in VideoTrackRecorder

Merge 58: Avoid calling VEA::Destroy() in VideoTrackRecorder

The default deleter for VEA calls Destroy() by default[0]. Running it multiple times can
cause a crash, see bug.
[0] https://cs.chromium.org/chromium/src/media/video/video_encode_accelerator.h?rcl=575512843787644524196d04076a59ec9415cb89&l=186

BUG=701450
TEST=Fixes the crash described in the bug.

Review-Url: https://codereview.chromium.org/2753593002
Cr-Commit-Position: refs/heads/master@{#456851}
(cherry picked from commit 98557de3bc84264df66670ca5baa7e0765f2186f)
NOTRY=true
NOPRESUBMIT=true
TBR=mcasas@chromium.org

Review-Url: https://codereview.chromium.org/2753513008
Cr-Commit-Position: refs/branch-heads/3029@{#219}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed: https://chromium.googlesource.com/chromium/src/+/813c97a414763063352914dd35fe9ea426852acf

[emircan, 2017-03-15 22:04:48]

Description was changed from

==========
Avoid calling VEA::Destroy() in VideoTrackRecorder

The default deleter for VEA calls Destroy() by default[0]. Running it multiple
times can
cause a crash, see bug.
[0]
https://cs.chromium.org/chromium/src/media/video/video_encode_accelerator.h?r...

BUG=701450
TEST=Fixes the crash described in the bug.

Review-Url: https://codereview.chromium.org/2753593002
Cr-Commit-Position: refs/heads/master@{#456851}
(cherry picked from commit 98557de3bc84264df66670ca5baa7e0765f2186f)
==========

to

==========
Merge 58: Avoid calling VEA::Destroy() in VideoTrackRecorder

The default deleter for VEA calls Destroy() by default[0]. Running it multiple
times can
cause a crash, see bug.
[0]
https://cs.chromium.org/chromium/src/media/video/video_encode_accelerator.h?r...

BUG=701450
TEST=Fixes the crash described in the bug.

Review-Url: https://codereview.chromium.org/2753593002
Cr-Commit-Position: refs/heads/master@{#456851}
(cherry picked from commit 98557de3bc84264df66670ca5baa7e0765f2186f)
NOTRY=true
NOPRESUBMIT=true
TBR=mcasas@chromium.org
==========

[emircan, 2017-03-15 22:05:06]

The CQ bit was checked by emircan@chromium.org

[commit-bot: I haz the power, 2017-03-15 22:05:57]

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-03-15 22:08:30]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489615506365220, "parent_rev":
"b3c1d9d6170108539e5ba8662692e0478366daac", "commit_rev":
"813c97a414763063352914dd35fe9ea426852acf"}

[commit-bot: I haz the power, 2017-03-15 22:09:10]

Description was changed from

==========
Merge 58: Avoid calling VEA::Destroy() in VideoTrackRecorder

The default deleter for VEA calls Destroy() by default[0]. Running it multiple
times can
cause a crash, see bug.
[0]
https://cs.chromium.org/chromium/src/media/video/video_encode_accelerator.h?r...

BUG=701450
TEST=Fixes the crash described in the bug.

Review-Url: https://codereview.chromium.org/2753593002
Cr-Commit-Position: refs/heads/master@{#456851}
(cherry picked from commit 98557de3bc84264df66670ca5baa7e0765f2186f)
NOTRY=true
NOPRESUBMIT=true
TBR=mcasas@chromium.org
==========

to

==========
Merge 58: Avoid calling VEA::Destroy() in VideoTrackRecorder

The default deleter for VEA calls Destroy() by default[0]. Running it multiple
times can
cause a crash, see bug.
[0]
https://cs.chromium.org/chromium/src/media/video/video_encode_accelerator.h?r...

BUG=701450
TEST=Fixes the crash described in the bug.

Review-Url: https://codereview.chromium.org/2753593002
Cr-Commit-Position: refs/heads/master@{#456851}
(cherry picked from commit 98557de3bc84264df66670ca5baa7e0765f2186f)
NOTRY=true
NOPRESUBMIT=true
TBR=mcasas@chromium.org

Review-Url: https://codereview.chromium.org/2753513008
Cr-Commit-Position: refs/branch-heads/3029@{#219}
Cr-Branched-From:
939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed:
https://chromium.googlesource.com/chromium/src/+/813c97a414763063352914dd35fe...
==========

[commit-bot: I haz the power, 2017-03-15 22:09:11]

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