Simplify VectorMover<true, T>::swap()

Simplify VectorMover<true, T>::swap()

Simplify VectorMover<true, T>::swap() - use std::swap_ranges instead
of creating the loop ourselves, plus the patch removes a "FIXME"
comment - compiler already produces the most efficient code for POD
types swap, so there is no need to introduce compex algorithms for
that (checked with GCC 4.7.2).

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169747

[Mikhail, 2014-03-21 15:00:31]

PTAL

[kouhei (in TOK), 2014-03-21 15:02:37]

On 2014/03/21 15:00:31, mikhail.pozdnyakov wrote:
> PTAL

lgtm

[eseidel, 2014-03-21 16:12:52]

lgtm

OK.

[Mikhail, 2014-03-21 16:17:28]

The CQ bit was checked by mikhail.pozdnyakov@intel.com

[commit-bot: I haz the power, 2014-03-21 16:18:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mikhail.pozdnyakov@intel.com/208253003/1

[commit-bot: I haz the power, 2014-03-21 16:57:50]

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

[commit-bot: I haz the power, 2014-03-21 16:57:51]

Try jobs failed on following builders:
  tryserver.blink on mac_blink_rel

[Mikhail, 2014-03-21 17:03:00]

The CQ bit was checked by mikhail.pozdnyakov@intel.com

[commit-bot: I haz the power, 2014-03-21 17:03:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mikhail.pozdnyakov@intel.com/208253003/1

[commit-bot: I haz the power, 2014-03-21 17:32:19]

Change committed as 169747