[spv2] Do not manage effect node render surfaces from blink

[spv2] Do not manage effect node render surfaces from blink

Render surface determination will happen on the compositor thread so we
will not need to make that decision in blink/PaintArtifactCompositor.
This patch switches PaintArtifactCompositor to not set the render
surface bit on effect nodes.

With this patch, test failures with spv2 and layer lists [1] are reduced
from 1103 to 16. No new tests are included in this patch but this brings
us very close to being able to turn on --enable-layer-lists by default
for spv2.

[1] --additional-drt-flag=--enable-slimming-paint-v2 --additional-drt-flag=--enable-layer-lists

BUG=635691
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

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

[pdr., 2016-08-10 23:37:13]

Description was changed from

==========
[spv2] Do not manage effect node render surfaces from blink

Render surface determination will happen on the compositor thread so we
will not need to make that decision in blink/PaintArtifactCompositor.
This patch switches PaintArtifactCompositor to not set the render
surface bit on effect nodes.

With this patch, test failures with spv2 and layer lists [1] are reduced
from 1103 to 16. No new tests are included in this patch but this brings
us very close to being able to turn on --enable-layer-lists by default
for spv2.

[1] --additional-drt-flag=--enable-slimming-paint-v2
--additional-drt-flag=--enable-layer-lists

BUG=635691
==========

to

==========
[spv2] Do not manage effect node render surfaces from blink

Render surface determination will happen on the compositor thread so we
will not need to make that decision in blink/PaintArtifactCompositor.
This patch switches PaintArtifactCompositor to not set the render
surface bit on effect nodes.

With this patch, test failures with spv2 and layer lists [1] are reduced
from 1103 to 16. No new tests are included in this patch but this brings
us very close to being able to turn on --enable-layer-lists by default
for spv2.

[1] --additional-drt-flag=--enable-slimming-paint-v2
--additional-drt-flag=--enable-layer-lists

BUG=635691
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[pdr., 2016-08-10 23:38:04]

pdr@chromium.org changed reviewers:
+ ajuma@chromium.org, chrishtr@chromium.org, trchen@chromium.org

[pdr., 2016-08-10 23:46:03]

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

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

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

[chrishtr, 2016-08-11 00:22:55]

lgtm

[commit-bot: I haz the power, 2016-08-11 02:46:25]

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

[commit-bot: I haz the power, 2016-08-11 02:46:26]

Dry run: This issue passed the CQ dry run.

[pdr., 2016-08-11 03:14:37]

The CQ bit was checked by pdr@chromium.org

[commit-bot: I haz the power, 2016-08-11 03:15:08]

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-11 03:21:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-11 03:25:04]

Description was changed from

==========
[spv2] Do not manage effect node render surfaces from blink

Render surface determination will happen on the compositor thread so we
will not need to make that decision in blink/PaintArtifactCompositor.
This patch switches PaintArtifactCompositor to not set the render
surface bit on effect nodes.

With this patch, test failures with spv2 and layer lists [1] are reduced
from 1103 to 16. No new tests are included in this patch but this brings
us very close to being able to turn on --enable-layer-lists by default
for spv2.

[1] --additional-drt-flag=--enable-slimming-paint-v2
--additional-drt-flag=--enable-layer-lists

BUG=635691
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
[spv2] Do not manage effect node render surfaces from blink

Render surface determination will happen on the compositor thread so we
will not need to make that decision in blink/PaintArtifactCompositor.
This patch switches PaintArtifactCompositor to not set the render
surface bit on effect nodes.

With this patch, test failures with spv2 and layer lists [1] are reduced
from 1103 to 16. No new tests are included in this patch but this brings
us very close to being able to turn on --enable-layer-lists by default
for spv2.

[1] --additional-drt-flag=--enable-slimming-paint-v2
--additional-drt-flag=--enable-layer-lists

BUG=635691
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

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

[commit-bot: I haz the power, 2016-08-11 03:25:06]

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