Move when swapping, if possible.

Move when swapping, if possible.

This change was avoided in the past because vc++ 2013 (12.0)
did not properly create default move constructors and
move assignment operators.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2454763002

TBR=reed
Verbal lgtm

Committed: https://skia.googlesource.com/skia/+/6f4293af6906721f09a12818ff3adc767badb3c7

[bungeman-skia, 2016-10-26 15:34:55]

Description was changed from

==========
Move when swapping, if possible.
==========

to

==========
Move when swapping, if possible.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2454763002
==========

[bungeman-skia, 2016-10-26 15:35:13]

Description was changed from

==========
Move when swapping, if possible.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2454763002
==========

to

==========
Move when swapping, if possible.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2454763002
==========

[bungeman-skia, 2016-10-26 15:41:06]

Description was changed from

==========
Move when swapping, if possible.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2454763002
==========

to

==========
Move when swapping, if possible.

This change was avoided in the past because vc++ 2013 (12.0)
did not properly create default move constructors and
move assignment operators.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2454763002
==========

[bungeman-skia, 2016-10-26 16:02:36]

The CQ bit was checked by bungeman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-26 16:02:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bungeman-skia, 2016-10-26 16:10:32]

The CQ bit was unchecked by bungeman@google.com

[bungeman-skia, 2016-10-26 18:03:38]

bungeman@google.com changed reviewers:
+ mtklein@google.com, reed@google.com

[mtklein_C, 2016-10-26 18:20:47]

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

[mtklein_C, 2016-10-26 18:20:47]

lgtm

[bungeman-skia, 2016-10-26 19:10:18]

Description was changed from

==========
Move when swapping, if possible.

This change was avoided in the past because vc++ 2013 (12.0)
did not properly create default move constructors and
move assignment operators.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2454763002
==========

to

==========
Move when swapping, if possible.

This change was avoided in the past because vc++ 2013 (12.0)
did not properly create default move constructors and
move assignment operators.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2454763002

TBR=reed
Verbal lgtm
==========

[bungeman-skia, 2016-10-26 19:10:23]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2016-10-26 19:10:33]

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-10-26 19:11:29]

Description was changed from

==========
Move when swapping, if possible.

This change was avoided in the past because vc++ 2013 (12.0)
did not properly create default move constructors and
move assignment operators.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2454763002

TBR=reed
Verbal lgtm
==========

to

==========
Move when swapping, if possible.

This change was avoided in the past because vc++ 2013 (12.0)
did not properly create default move constructors and
move assignment operators.

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2454763002

TBR=reed
Verbal lgtm

Committed:
https://skia.googlesource.com/skia/+/6f4293af6906721f09a12818ff3adc767badb3c7
==========

[commit-bot: I haz the power, 2016-10-26 19:11:30]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/6f4293af6906721f09a12818ff3adc767badb3c7