cc: Add gfx::ColorSpace to cc::ResourceProvider resource creation

cc: Add gfx::ColorSpace to cc::ResourceProvider resource creation

Updated all callers to specify a color space. Almost all of these are
the default gfx::ColorSpace, but some of them in cc::DirectRenderer and
its subclasses use the cc::OutputSurface's color space, as appropriate.

Update tests for cc::ResourcePool to take color space into account for
resource recycling.

Add tests for plumbing through to queryable properts in locks.

The addition of this argument does not change any behaviors, and this
patch should have no observable effects.

BUG=44872
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/b4d9236157069f52950e8f4d84cbc0f0df5cd9ca
Cr-Commit-Position: refs/heads/master@{#411575}

[ccameron, 2016-08-10 21:09:26]

Description was changed from

==========
cc: Add gfx::ColorSpace to cc::ResourceProvider resource creation

Updated all callers to specify a color space. Almost all of these are
the default gfx::ColorSpace, but some of them in cc::DirectRenderer and
its subclasses use the cc::OutputSurface's color space, as appropriate.

Update tests for cc::ResourcePool to take color space into account for
resource recycling.

Add tests for plumbing through to queryable properts in locks.

The addition of this argument does not change any behaviors, and this
patch should have no observable effects.

BUG=44872
==========

to

==========
cc: Add gfx::ColorSpace to cc::ResourceProvider resource creation

Updated all callers to specify a color space. Almost all of these are
the default gfx::ColorSpace, but some of them in cc::DirectRenderer and
its subclasses use the cc::OutputSurface's color space, as appropriate.

Update tests for cc::ResourcePool to take color space into account for
resource recycling.

Add tests for plumbing through to queryable properts in locks.

The addition of this argument does not change any behaviors, and this
patch should have no observable effects.

BUG=44872
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[ccameron, 2016-08-10 21:09:28]

The CQ bit was checked by ccameron@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-10 21:10:23]

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

[ccameron, 2016-08-10 21:24:00]

ccameron@chromium.org changed reviewers:
+ enne@chromium.org

[ccameron, 2016-08-10 21:24:01]

Lots of plumbing, has no effect on the running code. Couple of tests to make
sure that things are the same on both sides of the pipes.

[commit-bot: I haz the power, 2016-08-10 22:41:32]

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

[commit-bot: I haz the power, 2016-08-10 22:41:33]

Dry run: This issue passed the CQ dry run.

[ccameron, 2016-08-11 22:41:22]

ping -- does this look reasonable?

[enne (OOO), 2016-08-11 22:45:12]

lgtm

[ccameron, 2016-08-12 06:34:24]

The CQ bit was checked by ccameron@chromium.org

[ccameron, 2016-08-12 06:34:24]

The patchset sent to the CQ was uploaded after l-g-t-m from enne@chromium.org
Link to the patchset: https://codereview.chromium.org/2235623003/#ps20001
(title: "Rebase")

[commit-bot: I haz the power, 2016-08-12 06:34:34]

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-08-12 07:33:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-12 07:35:33]

Description was changed from

==========
cc: Add gfx::ColorSpace to cc::ResourceProvider resource creation

Updated all callers to specify a color space. Almost all of these are
the default gfx::ColorSpace, but some of them in cc::DirectRenderer and
its subclasses use the cc::OutputSurface's color space, as appropriate.

Update tests for cc::ResourcePool to take color space into account for
resource recycling.

Add tests for plumbing through to queryable properts in locks.

The addition of this argument does not change any behaviors, and this
patch should have no observable effects.

BUG=44872
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
cc: Add gfx::ColorSpace to cc::ResourceProvider resource creation

Updated all callers to specify a color space. Almost all of these are
the default gfx::ColorSpace, but some of them in cc::DirectRenderer and
its subclasses use the cc::OutputSurface's color space, as appropriate.

Update tests for cc::ResourcePool to take color space into account for
resource recycling.

Add tests for plumbing through to queryable properts in locks.

The addition of this argument does not change any behaviors, and this
patch should have no observable effects.

BUG=44872
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

[commit-bot: I haz the power, 2016-08-12 07:35:34]

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