Remove unnecessary std::move() from a //media/gpu file.

Remove unnecessary std::move() from a //media/gpu file.

Compiling w/ clang on CrOS, the compiler caught an unnecessary std::move()
in a CrOS-only file with the -Wpessimizing-move warning.

R=posciak@chromium.org, llozano@chromium.org
BUG=626241
Committed: https://crrev.com/f3b2adee05ddfc032bf0000fdcbdc3cf17b8b965
Cr-Commit-Position: refs/heads/master@{#406128}

[Dirk Pranke, 2016-07-15 02:09:08]

Description was changed from

==========
Remove unnecessary std::move() from a //media/gpu file.

Compiling w/ clang on CrOS, the compiler caught an unnecessary std::move()
in a CrOS-only file with the -Wpessimisting-move warning.

R=dalecurtis@chromium.org, llozano@chromium.org
BUG=626241
==========

to

==========
Remove unnecessary std::move() from a //media/gpu file.

Compiling w/ clang on CrOS, the compiler caught an unnecessary std::move()
in a CrOS-only file with the -Wpessimisting-move warning.

R=posciak@chromium.org, llozano@chromium.org
BUG=626241
==========

[Dirk Pranke, 2016-07-15 02:09:08]

dpranke@chromium.org changed reviewers:
+ posciak@chromium.org
- dalecurtis@chromium.org

[Dirk Pranke, 2016-07-15 02:09:23]

Description was changed from

==========
Remove unnecessary std::move() from a //media/gpu file.

Compiling w/ clang on CrOS, the compiler caught an unnecessary std::move()
in a CrOS-only file with the -Wpessimisting-move warning.

R=posciak@chromium.org, llozano@chromium.org
BUG=626241
==========

to

==========
Remove unnecessary std::move() from a //media/gpu file.

Compiling w/ clang on CrOS, the compiler caught an unnecessary std::move()
in a CrOS-only file with the -Wpessimizing-move warning.

R=posciak@chromium.org, llozano@chromium.org
BUG=626241
==========

[Dirk Pranke, 2016-07-15 02:09:53]

I have no idea if this is correct or not, or if it makes the warning go away ...
llozano@, can you verify the fix?

[Dirk Pranke, 2016-07-18 21:31:45]

ping?

[DaleCurtis, 2016-07-18 21:33:42]

dalecurtis@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[DaleCurtis, 2016-07-18 21:33:43]

Doesn't change anything it's a vector <-> vector stack allocation which should
already have move constructor. so lgtm

[Dirk Pranke, 2016-07-18 21:35:43]

Thanks!

[Dirk Pranke, 2016-07-18 21:35:47]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2016-07-18 21:37:04]

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-07-18 23:26:59]

Description was changed from

==========
Remove unnecessary std::move() from a //media/gpu file.

Compiling w/ clang on CrOS, the compiler caught an unnecessary std::move()
in a CrOS-only file with the -Wpessimizing-move warning.

R=posciak@chromium.org, llozano@chromium.org
BUG=626241
==========

to

==========
Remove unnecessary std::move() from a //media/gpu file.

Compiling w/ clang on CrOS, the compiler caught an unnecessary std::move()
in a CrOS-only file with the -Wpessimizing-move warning.

R=posciak@chromium.org, llozano@chromium.org
BUG=626241
==========

[commit-bot: I haz the power, 2016-07-18 23:26:59]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-18 23:27:15]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-18 23:28:12]

Description was changed from

==========
Remove unnecessary std::move() from a //media/gpu file.

Compiling w/ clang on CrOS, the compiler caught an unnecessary std::move()
in a CrOS-only file with the -Wpessimizing-move warning.

R=posciak@chromium.org, llozano@chromium.org
BUG=626241
==========

to

==========
Remove unnecessary std::move() from a //media/gpu file.

Compiling w/ clang on CrOS, the compiler caught an unnecessary std::move()
in a CrOS-only file with the -Wpessimizing-move warning.

R=posciak@chromium.org, llozano@chromium.org
BUG=626241

Committed: https://crrev.com/f3b2adee05ddfc032bf0000fdcbdc3cf17b8b965
Cr-Commit-Position: refs/heads/master@{#406128}
==========

[commit-bot: I haz the power, 2016-07-18 23:28:13]

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