Remove useless std::move().

Remove useless std::move().

https://codereview.chromium.org/1579583004/ converted a Pass() to
std::move(), which enabled the compiler to tell us that it's not needed
and actually harmful:

..\..\chrome\installer\util\shell_util.cc(277,10) :
    error: moving a local object in a return statement prevents copy elision
    [-Werror,-Wpessimizing-move]
  return std::move(entries);
         ^
..\..\chrome\installer\util\shell_util.cc(277,10) :
    note: remove std::move call here
  return std::move(entries);
         ^~~~~~~~~~
BUG=82385
TBR=mgiuca

[Nico, 2016-01-13 15:25:24]

Description was changed from

==========
Remove useless std::move().

https://codereview.chromium.org/1579583004/ converted a Pass() to
std::move(), which enabled the compiler to tell us that it's not needed
and actually harmful:

..\..\chrome\installer\util\shell_util.cc(277,10) :
    error: moving a local object in a return statement prevents copy elision
    [-Werror,-Wpessimizing-move]
  return std::move(entries);
         ^
..\..\chrome\installer\util\shell_util.cc(277,10) :
    note: remove std::move call here
  return std::move(entries);
         ^~~~~~~~~~
BUG=82385
==========

to

==========
Remove useless std::move().

https://codereview.chromium.org/1579583004/ converted a Pass() to
std::move(), which enabled the compiler to tell us that it's not needed
and actually harmful:

..\..\chrome\installer\util\shell_util.cc(277,10) :
    error: moving a local object in a return statement prevents copy elision
    [-Werror,-Wpessimizing-move]
  return std::move(entries);
         ^
..\..\chrome\installer\util\shell_util.cc(277,10) :
    note: remove std::move call here
  return std::move(entries);
         ^~~~~~~~~~
BUG=82385
TBR=mgiuca
==========

[Nico, 2016-01-13 15:25:31]

thakis@chromium.org changed reviewers:
+ mgiuca@chromium.org

[commit-bot: I haz the power, 2016-01-13 15:34:29]

Description was changed from

==========
Remove useless std::move().

https://codereview.chromium.org/1579583004/ converted a Pass() to
std::move(), which enabled the compiler to tell us that it's not needed
and actually harmful:

..\..\chrome\installer\util\shell_util.cc(277,10) :
    error: moving a local object in a return statement prevents copy elision
    [-Werror,-Wpessimizing-move]
  return std::move(entries);
         ^
..\..\chrome\installer\util\shell_util.cc(277,10) :
    note: remove std::move call here
  return std::move(entries);
         ^~~~~~~~~~
BUG=82385
TBR=mgiuca
==========

to

==========
Remove useless std::move().

https://codereview.chromium.org/1579583004/ converted a Pass() to
std::move(), which enabled the compiler to tell us that it's not needed
and actually harmful:

..\..\chrome\installer\util\shell_util.cc(277,10) :
    error: moving a local object in a return statement prevents copy elision
    [-Werror,-Wpessimizing-move]
  return std::move(entries);
         ^
..\..\chrome\installer\util\shell_util.cc(277,10) :
    note: remove std::move call here
  return std::move(entries);
         ^~~~~~~~~~
BUG=82385
TBR=mgiuca

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

[commit-bot: I haz the power, 2016-01-13 15:34:30]

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

[Nico, 2016-01-13 15:34:32]

Description was changed from

==========
Remove useless std::move().

https://codereview.chromium.org/1579583004/ converted a Pass() to
std::move(), which enabled the compiler to tell us that it's not needed
and actually harmful:

..\..\chrome\installer\util\shell_util.cc(277,10) :
    error: moving a local object in a return statement prevents copy elision
    [-Werror,-Wpessimizing-move]
  return std::move(entries);
         ^
..\..\chrome\installer\util\shell_util.cc(277,10) :
    note: remove std::move call here
  return std::move(entries);
         ^~~~~~~~~~
BUG=82385
TBR=mgiuca

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

to

==========
Remove useless std::move().

https://codereview.chromium.org/1579583004/ converted a Pass() to
std::move(), which enabled the compiler to tell us that it's not needed
and actually harmful:

..\..\chrome\installer\util\shell_util.cc(277,10) :
    error: moving a local object in a return statement prevents copy elision
    [-Werror,-Wpessimizing-move]
  return std::move(entries);
         ^
..\..\chrome\installer\util\shell_util.cc(277,10) :
    note: remove std::move call here
  return std::move(entries);
         ^~~~~~~~~~
BUG=82385
TBR=mgiuca
==========

[Nico, 2016-01-13 15:34:33]

Committed patchset #1 (id:1) to pending queue manually as
53acfe90cf7e1c06a34f38783256d4c386dd1d16 (presubmit successful).

[Matt Giuca, 2016-01-13 23:21:21]

lgtm thanks!

Sorry for breaking Clang Windows.