Add defaulted move constructors for some types that just got copy constructors

Add defaulted move constructors for some types that just got copy constructors

They can all benefit from moving, since they contain std::string and
std::vector. We intended to add these in
https://codereview.webrtc.org/1896953004/, but got compiler errors we
couldn't make sense of, so we skipped them. It turns out that what the
compiler was complaining about was that when we said we'd have a
user-defined move constructor, it stopped generating a copy assignment
operator for us. This CL solves the problem by outfitting the types
with defaulted copy and move assignment operators too.

Committed: https://crrev.com/4fb3d2bcca968e8a0f29d8a20f67238f64f40102
Cr-Commit-Position: refs/heads/master@{#12469}

[kwiberg-webrtc, 2016-04-19 11:22:15]

kwiberg@webrtc.org changed reviewers:
+ henrika@webrtc.org, kjellander@webrtc.org

[kjellander_webrtc, 2016-04-19 16:36:05]

lgtm. thanks!

[kwiberg-webrtc, 2016-04-22 02:09:41]

On 2016/04/19 16:36:05, kjellander (webrtc) wrote:
> lgtm. thanks!

henrika@: ping?

[henrika_webrtc, 2016-04-22 08:10:05]

LGTM

[kwiberg-webrtc, 2016-04-22 09:23:59]

The CQ bit was checked by kwiberg@webrtc.org

[commit-bot: I haz the power, 2016-04-22 09:24:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1899173002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1899173002/1

[commit-bot: I haz the power, 2016-04-22 11:24:48]

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

[commit-bot: I haz the power, 2016-04-22 11:24:50]

Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[kjellander_webrtc, 2016-04-22 11:42:31]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2016-04-22 11:42:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1899173002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1899173002/1

[commit-bot: I haz the power, 2016-04-22 11:59:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 11:59:42]

Description was changed from

==========
Add defaulted move constructors for some types that just got copy constructors

They can all benefit from moving, since they contain std::string and
std::vector. We intended to add these in
https://codereview.webrtc.org/1896953004/, but got compiler errors we
couldn't make sense of, so we skipped them. It turns out that what the
compiler was complaining about was that when we said we'd have a
user-defined move constructor, it stopped generating a copy assignment
operator for us. This CL solves the problem by outfitting the types
with defaulted copy and move assignment operators too.
==========

to

==========
Add defaulted move constructors for some types that just got copy constructors

They can all benefit from moving, since they contain std::string and
std::vector. We intended to add these in
https://codereview.webrtc.org/1896953004/, but got compiler errors we
couldn't make sense of, so we skipped them. It turns out that what the
compiler was complaining about was that when we said we'd have a
user-defined move constructor, it stopped generating a copy assignment
operator for us. This CL solves the problem by outfitting the types
with defaulted copy and move assignment operators too.

Committed: https://crrev.com/4fb3d2bcca968e8a0f29d8a20f67238f64f40102
Cr-Commit-Position: refs/heads/master@{#12469}
==========

[commit-bot: I haz the power, 2016-04-22 11:59:43]

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