cc: Use setter methods on layer instead of SetLayerPropertiesForTesting

cc: Use setter methods on layer instead of SetLayerPropertiesForTesting

This changes half the LayerTreeHostCommon tests to just set things on
layers instead of calling SetLayerPropertiesForTesting with a bunch of
default values that are just noise. Once the others are done we can
remove those methods.

R=ajuma@chromium.org, jaydasika@chromium.org
BUG=630754
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel

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

[danakj, 2016-07-25 21:00:28]

Description was changed from

==========
cc: Use setter methods on layer instead of SetLayerPropertiesForTesting

This changes half the LayerTreeHostCommon tests to just set things on
layers instead of calling SetLayerPropertiesForTesting with a bunch of
default values that are just noise. Once the others are done we can
remove those methods.

R=ajuma@chromium.org, jaydasika@chromium.org
BUG=630754
==========

to

==========
cc: Use setter methods on layer instead of SetLayerPropertiesForTesting

This changes half the LayerTreeHostCommon tests to just set things on
layers instead of calling SetLayerPropertiesForTesting with a bunch of
default values that are just noise. Once the others are done we can
remove those methods.

R=ajuma@chromium.org, jaydasika@chromium.org
BUG=630754
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel
==========

[danakj, 2016-07-25 21:01:09]

I feel like this actually makes the tests more clear too. It's easier to tell
what is/isn't being set on each layer. And when one thing changes in the test
later, it's clear that you set that one thing instead of having to compare all
the arguments to the previous call.

[danakj, 2016-07-25 21:01:13]

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

[commit-bot: I haz the power, 2016-07-25 21:02:46]

Dry run: 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-07-25 21:46:46]

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

[commit-bot: I haz the power, 2016-07-25 21:46:47]

Dry run: This issue passed the CQ dry run.

[jaydasika, 2016-07-25 21:53:53]

lgtm

[danakj, 2016-07-25 21:55:56]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2016-07-25 21:56:29]

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-07-25 22:00:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-25 22:02:31]

Description was changed from

==========
cc: Use setter methods on layer instead of SetLayerPropertiesForTesting

This changes half the LayerTreeHostCommon tests to just set things on
layers instead of calling SetLayerPropertiesForTesting with a bunch of
default values that are just noise. Once the others are done we can
remove those methods.

R=ajuma@chromium.org, jaydasika@chromium.org
BUG=630754
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel
==========

to

==========
cc: Use setter methods on layer instead of SetLayerPropertiesForTesting

This changes half the LayerTreeHostCommon tests to just set things on
layers instead of calling SetLayerPropertiesForTesting with a bunch of
default values that are just noise. Once the others are done we can
remove those methods.

R=ajuma@chromium.org, jaydasika@chromium.org
BUG=630754
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel

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

[commit-bot: I haz the power, 2016-07-25 22:02:32]

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