Passing scoped_ptr into ScopedVector

Substituting pattern push_back(ptr.release()) with push_back(ptr.Pass()) in media/

BUG=457697
Committed: https://crrev.com/04591163c31819c50d1ad762072979c5785b15bf
Cr-Commit-Position: refs/heads/master@{#329592}

[anujsharma, 2015-05-12 09:19:50]

anujk.sharma@samsung.com changed reviewers:
+ xhwang@chromium.org

[anujsharma, 2015-05-12 09:19:53]

PTAL

[xhwang, 2015-05-12 16:13:06]

lgtm % one comment

https://codereview.chromium.org/1138163002/diff/1/media/base/audio_block_fifo.cc
File media/base/audio_block_fifo.cc (right):

https://codereview.chromium.org/1138163002/diff/1/media/base/audio_block_fifo...
media/base/audio_block_fifo.cc:97: AudioBus::Create(channels_,
block_frames_).Pass());
.Pass() not needed?

[anujsharma, 2015-05-13 03:01:07]

Thanks xhwang for your review and lgtm.

I will remove the said change in the next patchset.

https://codereview.chromium.org/1138163002/diff/1/media/base/audio_block_fifo.cc
File media/base/audio_block_fifo.cc (right):

https://codereview.chromium.org/1138163002/diff/1/media/base/audio_block_fifo...
media/base/audio_block_fifo.cc:97: AudioBus::Create(channels_,
block_frames_).Pass());
On 2015/05/12 16:13:06, xhwang wrote:
> .Pass() not needed?

Ok, I will remove it in next patchset.

[anujsharma, 2015-05-13 03:20:49]

@Xhwang

I have removed the change from "media/base/audio_block_fifo.cc" file.

I am now proceeding to commit.

Thanks!!

[anujsharma, 2015-05-13 03:20:52]

The CQ bit was checked by anujk.sharma@samsung.com

[anujsharma, 2015-05-13 03:20:53]

The patchset sent to the CQ was uploaded after l-g-t-m from xhwang@chromium.org
Link to the patchset: https://codereview.chromium.org/1138163002/#ps20001
(title: "Removed file as suggested")

[commit-bot: I haz the power, 2015-05-13 03:21:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1138163002/20001

[anujsharma, 2015-05-13 03:54:23]

The CQ bit was unchecked by anujk.sharma@samsung.com

[anujsharma, 2015-05-13 04:42:49]

@xhwang

I have checked with the builds, everything going well.

So, could you please give a go ahead for the change.

Thanks!!

[xhwang, 2015-05-13 05:51:39]

The CQ bit was checked by xhwang@chromium.org

[xhwang, 2015-05-13 05:51:40]

lgtm

LGTM

[commit-bot: I haz the power, 2015-05-13 05:52:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1138163002/20001

[commit-bot: I haz the power, 2015-05-13 05:55:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-13 05:56:33]

Patchset 2 (id:??) landed as
https://crrev.com/04591163c31819c50d1ad762072979c5785b15bf
Cr-Commit-Position: refs/heads/master@{#329592}