SkPixmap::setColorSpace

SkPixmap::setColorSpace

Landed for reed@ with bug fix:
Use default copy constructor for SkDraw

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

patch from issue 2057563002 at patchset 1 (http://crrev.com/2057563002#ps1)

Committed: https://skia.googlesource.com/skia/+/804b461bc4fe3cfd959fc065c1c0b524ef96e90b

[reed1, 2016-06-09 17:37:32]

reed@google.com changed reviewers:
+ reed@google.com

[reed1, 2016-06-09 17:37:32]

lgtm

https://codereview.chromium.org/2052943002/diff/1/src/core/SkPixmap.cpp
File src/core/SkPixmap.cpp (right):

https://codereview.chromium.org/2052943002/diff/1/src/core/SkPixmap.cpp#newco...
src/core/SkPixmap.cpp:56: fInfo = fInfo.makeColorSpace(cs);
std::move

[msarett, 2016-06-09 17:41:50]

Description was changed from

==========
Use default copy constructor for SkDraw

SkPixmap::setColorSpace

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

patch from issue 2057563002 at patchset 1 (http://crrev.com/2057563002#ps1)
==========

to

==========
SkPixmap::setColorSpace

Landed for reed@ with bug fix:
Use default copy constructor for SkDraw



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

patch from issue 2057563002 at patchset 1 (http://crrev.com/2057563002#ps1)
==========

[msarett, 2016-06-09 17:41:59]

Description was changed from

==========
SkPixmap::setColorSpace

Landed for reed@ with bug fix:
Use default copy constructor for SkDraw



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

patch from issue 2057563002 at patchset 1 (http://crrev.com/2057563002#ps1)
==========

to

==========
SkPixmap::setColorSpace

Landed for reed@ with bug fix:
Use default copy constructor for SkDraw

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

patch from issue 2057563002 at patchset 1 (http://crrev.com/2057563002#ps1)
==========

[msarett, 2016-06-09 17:42:13]

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

[commit-bot: I haz the power, 2016-06-09 17:42:24]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2052943002/20001

[msarett, 2016-06-09 17:42:44]

https://codereview.chromium.org/2052943002/diff/1/src/core/SkPixmap.cpp
File src/core/SkPixmap.cpp (right):

https://codereview.chromium.org/2052943002/diff/1/src/core/SkPixmap.cpp#newco...
src/core/SkPixmap.cpp:56: fInfo = fInfo.makeColorSpace(cs);
On 2016/06/09 17:37:32, reed1 wrote:
> std::move

Done.

[commit-bot: I haz the power, 2016-06-09 18:00:33]

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

[commit-bot: I haz the power, 2016-06-09 18:00:34]

Dry run: This issue passed the CQ dry run.

[msarett, 2016-06-09 18:02:19]

The CQ bit was checked by msarett@google.com

[msarett, 2016-06-09 18:02:20]

The patchset sent to the CQ was uploaded after l-g-t-m from reed@google.com
Link to the patchset: https://codereview.chromium.org/2052943002/#ps20001
(title: "std::move()")

[commit-bot: I haz the power, 2016-06-09 18:02:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2052943002/20001

[commit-bot: I haz the power, 2016-06-09 18:03:48]

Description was changed from

==========
SkPixmap::setColorSpace

Landed for reed@ with bug fix:
Use default copy constructor for SkDraw

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

patch from issue 2057563002 at patchset 1 (http://crrev.com/2057563002#ps1)
==========

to

==========
SkPixmap::setColorSpace

Landed for reed@ with bug fix:
Use default copy constructor for SkDraw

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

patch from issue 2057563002 at patchset 1 (http://crrev.com/2057563002#ps1)

Committed:
https://skia.googlesource.com/skia/+/804b461bc4fe3cfd959fc065c1c0b524ef96e90b
==========

[commit-bot: I haz the power, 2016-06-09 18:03:49]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/804b461bc4fe3cfd959fc065c1c0b524ef96e90b