Relanding "Reset the root layer on the compositor when recreating a root layer."

Relanding "Reset the root layer on the compositor when recreating a root layer."

Initial attempt found here: https://codereview.chromium.org/983933002/

This is going to be used by https://codereview.chromium.org/975943002/

TEST=LayerOwnerTest.RecreateRootLayerWithNullCompositor
TEST=LayerOwnerTestWithCompositor.RecreateRootLayerWithCompositor

BUG=337596
Committed: https://crrev.com/0df296f41a987c04a2aad2d19e9d5095c3a525c2
Cr-Commit-Position: refs/heads/master@{#319665}

[bruthig, 2015-03-09 15:21:27]

bruthig@chromium.org changed reviewers:
+ sadrul@chromium.org

[bruthig, 2015-03-09 15:21:28]

sadrul@ can you please have a look?  I tried to land this fix on Friday but it
was reverted due to a memory leak in the tests.

[sadrul, 2015-03-09 15:31:25]

lgtm

https://codereview.chromium.org/987043004/diff/1/ui/compositor/layer_owner_un...
File ui/compositor/layer_owner_unittest.cc (right):

https://codereview.chromium.org/987043004/diff/1/ui/compositor/layer_owner_un...
ui/compositor/layer_owner_unittest.cc:29: scoped_ptr<ui::Compositor>
compositor_;
Make this private and add a public/protected accessor instead?

[bruthig, 2015-03-09 15:59:22]

I updated the compositor_'s access.  No need to review.

https://codereview.chromium.org/987043004/diff/1/ui/compositor/layer_owner_un...
File ui/compositor/layer_owner_unittest.cc (right):

https://codereview.chromium.org/987043004/diff/1/ui/compositor/layer_owner_un...
ui/compositor/layer_owner_unittest.cc:29: scoped_ptr<ui::Compositor>
compositor_;
On 2015/03/09 15:31:25, sadrul wrote:
> Make this private and add a public/protected accessor instead?

Done.

[bruthig, 2015-03-09 15:59:26]

The CQ bit was checked by bruthig@chromium.org

[bruthig, 2015-03-09 15:59:26]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/987043004/#ps20001 (title:
"Changed LayerOwnerTestWithCompositor::compositor_'s access from protected to
private.")

[commit-bot: I haz the power, 2015-03-09 15:59:59]

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

[commit-bot: I haz the power, 2015-03-09 17:19:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-09 17:20:29]

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