cc: Make sure we don't insert or remove layers during painting

cc: Make sure we don't insert or remove layers during painting

Calling Layer::Update should not cause changes to tree topology.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/2b92d0faeb76efa29112da48ed7a193827d62017
Cr-Commit-Position: refs/heads/master@{#338385}

[enne (OOO), 2015-07-10 20:29:21]

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

[enne (OOO), 2015-07-10 20:29:21]

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

[enne (OOO), 2015-07-10 20:29:21]

https://codereview.chromium.org/1232573005/diff/1/cc/layers/layer.cc
File cc/layers/layer.cc (right):

https://codereview.chromium.org/1232573005/diff/1/cc/layers/layer.cc#newcode138
cc/layers/layer.cc:138: DCHECK(!host->in_update());
Reuse host->in_paint_layer_contents()?

[ajuma, 2015-07-10 20:52:53]

https://codereview.chromium.org/1232573005/diff/1/cc/layers/layer.cc
File cc/layers/layer.cc (right):

https://codereview.chromium.org/1232573005/diff/1/cc/layers/layer.cc#newcode138
cc/layers/layer.cc:138: DCHECK(!host->in_update());
On 2015/07/10 20:29:21, enne wrote:
> Reuse host->in_paint_layer_contents()?

Done, and moved the DCHECK over to LayerTreeHost::{Register, Unregister}Layer.

[enne (OOO), 2015-07-10 21:01:19]

lgtm

[ajuma, 2015-07-10 22:32:40]

The CQ bit was checked by ajuma@chromium.org

[commit-bot: I haz the power, 2015-07-10 22:33:19]

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

[commit-bot: I haz the power, 2015-07-10 22:37:50]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-07-10 22:39:50]

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