cc: Pass LayerTreeHostImpl to LayerImpl constructor

cc: Pass LayerTreeHostImpl to LayerImpl constructor

In order to allow for LayerImpl to notify its host when one of its property
changes, it needs to more robustly have its host set (which is not the case in
most tests).  To avoid a recursive pass when adding children or after layer
synchronization, instead just pass it to the constructor of LayerImpl, since a
given LayerImpl will never move from one host to another.

NOTRY=true
BUG=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=171830

[enne (OOO), 2012-12-07 07:14:37]

One shaved yak, as ordered.

[jamesr, 2012-12-07 07:28:39]

lgtm

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_impl_proxy.h
File cc/test/fake_impl_proxy.h (right):

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_impl_proxy.h#new...
cc/test/fake_impl_proxy.h:11: namespace cc {
can you put this in namespace cc::test ?

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_layer_tree_host_...
File cc/test/fake_layer_tree_host_impl.cc (right):

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_layer_tree_host_...
cc/test/fake_layer_tree_host_impl.cc:11: {
chromium style?

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_layer_tree_host_...
File cc/test/fake_layer_tree_host_impl.h (right):

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_layer_tree_host_...
cc/test/fake_layer_tree_host_impl.h:12: namespace cc {
and test ?

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_layer_tree_host_...
cc/test/fake_layer_tree_host_impl.h:15: {
chromium style for new file plz

[enne (OOO), 2012-12-07 08:16:37]

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_impl_proxy.h
File cc/test/fake_impl_proxy.h (right):

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_impl_proxy.h#new...
cc/test/fake_impl_proxy.h:11: namespace cc {
On 2012/12/07 07:28:40, jamesr wrote:
> can you put this in namespace cc::test ?

It doesn't look like anything else is in the cc::test namespace.  There's a mix
of cc, WebKit, and WebKitTests (seriously).   I'd prefer to punt and clean up
cc/test namespaces in a separate patch.

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_layer_tree_host_...
File cc/test/fake_layer_tree_host_impl.cc (right):

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_layer_tree_host_...
cc/test/fake_layer_tree_host_impl.cc:11: {
On 2012/12/07 07:28:40, jamesr wrote:
> chromium style?

Done.

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_layer_tree_host_...
File cc/test/fake_layer_tree_host_impl.h (right):

https://codereview.chromium.org/11472021/diff/1/cc/test/fake_layer_tree_host_...
cc/test/fake_layer_tree_host_impl.h:15: {
On 2012/12/07 07:28:40, jamesr wrote:
> chromium style for new file plz

Done.

[nduca, 2012-12-07 17:51:16]

holy yakshave batman

[commit-bot: I haz the power, 2012-12-07 20:28:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/enne@chromium.org/11472021/8076

[commit-bot: I haz the power, 2012-12-07 20:31:48]

Change committed as 171830