Revert of cc : Make LayerImpl destruction independent of tree hierarchy

Revert of cc : Make LayerImpl destruction independent of tree hierarchy (patchset #2 id:20001 of https://codereview.chromium.org/2043963005/ )

Reason for revert:
Didn't fix anything.

Original issue's description:
> cc : Make LayerImpl destruction independent of tree hierarchy
>
> Instead of deleting all layers when the root is deleted, we delete
> all layers when the layer tree impl is shutdown.
>
> BUG=617791
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/e536d2292fa548ee6231ff23ac65203c261e4a14
> Cr-Commit-Position: refs/heads/master@{#398606}

TBR=ajuma@chromium.org,weiliangc@chromium.org,jaydasika@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=617791
Committed: https://crrev.com/ef1625ae9feafee9ddaa2a58b9476175d7956b8b
Cr-Commit-Position: refs/heads/master@{#398633}

[Ken Rockot(use gerrit already), 2016-06-08 19:06:37]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-06-08 19:06:37]

Created Revert of cc : Make LayerImpl destruction independent of tree hierarchy

[commit-bot: I haz the power, 2016-06-08 19:07:23]

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

[ajuma, 2016-06-08 19:08:55]

The CQ bit was unchecked by ajuma@chromium.org

[commit-bot: I haz the power, 2016-06-08 19:09:01]

Committed patchset #1 (id:1)

[ajuma, 2016-06-08 19:09:59]

On 2016/06/08 19:06:37, Ken Rockot wrote:
> Created Revert of cc : Make LayerImpl destruction independent of tree
hierarchy

Un-checking the commit box, assuming you meant to revert the revert rather than
re-revert the original.

[Ken Rockot(use gerrit already), 2016-06-08 19:10:18]

Yep thanks

On Wed, Jun 8, 2016 at 12:09 PM, <ajuma@chromium.org> wrote:

> On 2016/06/08 19:06:37, Ken Rockot wrote:
> > Created Revert of cc : Make LayerImpl destruction independent of tree
> hierarchy
>
> Un-checking the commit box, assuming you meant to revert the revert rather
> than
> re-revert the original.
>
> https://codereview.chromium.org/2044253004/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[ajuma, 2016-06-08 19:11:03]

Sadly I was too late :(

On 2016/06/08 19:10:18, Ken Rockot wrote:
> Yep thanks
> 
> On Wed, Jun 8, 2016 at 12:09 PM, <mailto:ajuma@chromium.org> wrote:
> 
> > On 2016/06/08 19:06:37, Ken Rockot wrote:
> > > Created Revert of cc : Make LayerImpl destruction independent of tree
> > hierarchy
> >
> > Un-checking the commit box, assuming you meant to revert the revert rather
> > than
> > re-revert the original.
> >
> > https://codereview.chromium.org/2044253004/
> >
> 
> -- 
> You received this message because you are subscribed to the Google Groups
> "Chromium-reviews" group.
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

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

Description was changed from

==========
Revert of cc : Make LayerImpl destruction independent of tree hierarchy
(patchset #2 id:20001 of https://codereview.chromium.org/2043963005/ )

Reason for revert:
Didn't fix anything.

Original issue's description:
> cc : Make LayerImpl destruction independent of tree hierarchy
> 
> Instead of deleting all layers when the root is deleted, we delete
> all layers when the layer tree impl is shutdown.
> 
> BUG=617791
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> 
> Committed: https://crrev.com/e536d2292fa548ee6231ff23ac65203c261e4a14
> Cr-Commit-Position: refs/heads/master@{#398606}

TBR=ajuma@chromium.org,weiliangc@chromium.org,jaydasika@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=617791
==========

to

==========
Revert of cc : Make LayerImpl destruction independent of tree hierarchy
(patchset #2 id:20001 of https://codereview.chromium.org/2043963005/ )

Reason for revert:
Didn't fix anything.

Original issue's description:
> cc : Make LayerImpl destruction independent of tree hierarchy
>
> Instead of deleting all layers when the root is deleted, we delete
> all layers when the layer tree impl is shutdown.
>
> BUG=617791
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/e536d2292fa548ee6231ff23ac65203c261e4a14
> Cr-Commit-Position: refs/heads/master@{#398606}

TBR=ajuma@chromium.org,weiliangc@chromium.org,jaydasika@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=617791

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

[commit-bot: I haz the power, 2016-06-08 19:11:26]

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

[Ken Rockot(use gerrit already), 2016-06-08 19:15:59]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2049883002/ by rockot@chromium.org.

The reason for reverting is: around and around we go :D.