cc: Remove direct calls to LayerImpl::SetHasRenderSurface from tests

cc: Remove direct calls to LayerImpl::SetHasRenderSurface from tests

This replaces most calls to LayerImpl::SetHasRenderSurface in unit
tests with setting LayerImpl::test_properties::force_render_surface,
except for calls that involve the root layer and calls in LayerImplTests.
Calls that involve the root layer are just deleted without replacement,
since the root layer always has a render surface. Calls in LayerImplTests
can be deleted when LayerImpl::SetHasRenderSurface is itself deleted.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2659883003
Cr-Commit-Position: refs/heads/master@{#446776}
Committed: https://chromium.googlesource.com/chromium/src/+/35b24f6319f8516d7393dd8cd68225cf0cedaa2f

[ajuma, 2017-01-27 19:41:23]

Description was changed from

==========
cc: Remove direct calls to LayerImpl::SetHasRenderSurface from tests

This replaces most calls to LayerImpl::SetHasRenderSurface in unit
tests with setting LayerImpl::test_properties::force_render_surface,
except for calls that involve the root layer and calls in LayerImplTests.
Calls that involve the root layer are just deleted without replacement,
since the root layer always has a render surface. Calls in LayerImplTests
can be deleted when LayerImpl::SetHasRenderSurface is itself deleted.

BUG=611883
==========

to

==========
cc: Remove direct calls to LayerImpl::SetHasRenderSurface from tests

This replaces most calls to LayerImpl::SetHasRenderSurface in unit
tests with setting LayerImpl::test_properties::force_render_surface,
except for calls that involve the root layer and calls in LayerImplTests.
Calls that involve the root layer are just deleted without replacement,
since the root layer always has a render surface. Calls in LayerImplTests
can be deleted when LayerImpl::SetHasRenderSurface is itself deleted.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ajuma, 2017-01-27 19:49:38]

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

[commit-bot: I haz the power, 2017-01-27 19:50:32]

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

[ajuma, 2017-01-27 19:51:16]

Description was changed from

==========
cc: Remove direct calls to LayerImpl::SetHasRenderSurface from tests

This replaces most calls to LayerImpl::SetHasRenderSurface in unit
tests with setting LayerImpl::test_properties::force_render_surface,
except for calls that involve the root layer and calls in LayerImplTests.
Calls that involve the root layer are just deleted without replacement,
since the root layer always has a render surface. Calls in LayerImplTests
can be deleted when LayerImpl::SetHasRenderSurface is itself deleted.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Remove direct calls to LayerImpl::SetHasRenderSurface from tests

This replaces most calls to LayerImpl::SetHasRenderSurface in unit
tests with setting LayerImpl::test_properties::force_render_surface,
except for calls that involve the root layer and calls in LayerImplTests.
Calls that involve the root layer are just deleted without replacement,
since the root layer always has a render surface. Calls in LayerImplTests
can be deleted when LayerImpl::SetHasRenderSurface is itself deleted.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ajuma, 2017-01-27 19:51:16]

ajuma@chromium.org changed reviewers:
+ jaydasika@chromium.org, weiliangc@chromium.org

[weiliangc, 2017-01-27 19:55:52]

LGTM

[commit-bot: I haz the power, 2017-01-27 20:46:07]

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

[commit-bot: I haz the power, 2017-01-27 20:46:08]

Dry run: This issue passed the CQ dry run.

[ajuma, 2017-01-27 21:04:04]

The CQ bit was checked by ajuma@chromium.org

[commit-bot: I haz the power, 2017-01-27 21:04:55]

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, 2017-01-27 21:23:14]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485551044887990, "parent_rev":
"05209f3fb8784a0f322a0cd2dcd634469b3cbf4e", "commit_rev":
"35b24f6319f8516d7393dd8cd68225cf0cedaa2f"}

[commit-bot: I haz the power, 2017-01-27 21:23:48]

Description was changed from

==========
cc: Remove direct calls to LayerImpl::SetHasRenderSurface from tests

This replaces most calls to LayerImpl::SetHasRenderSurface in unit
tests with setting LayerImpl::test_properties::force_render_surface,
except for calls that involve the root layer and calls in LayerImplTests.
Calls that involve the root layer are just deleted without replacement,
since the root layer always has a render surface. Calls in LayerImplTests
can be deleted when LayerImpl::SetHasRenderSurface is itself deleted.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
cc: Remove direct calls to LayerImpl::SetHasRenderSurface from tests

This replaces most calls to LayerImpl::SetHasRenderSurface in unit
tests with setting LayerImpl::test_properties::force_render_surface,
except for calls that involve the root layer and calls in LayerImplTests.
Calls that involve the root layer are just deleted without replacement,
since the root layer always has a render surface. Calls in LayerImplTests
can be deleted when LayerImpl::SetHasRenderSurface is itself deleted.

BUG=611883
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2659883003
Cr-Commit-Position: refs/heads/master@{#446776}
Committed:
https://chromium.googlesource.com/chromium/src/+/35b24f6319f8516d7393dd8cd682...
==========

[commit-bot: I haz the power, 2017-01-27 21:23:49]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/35b24f6319f8516d7393dd8cd682...