Revert of Fix bug where removed (but not deleted) windows are not reflected in the tree properly

Revert of Fix bug where removed (but not deleted) windows are not reflected in the tree properly (patchset #2 id:20001 of https://codereview.chromium.org/2495533005/ )

Reason for revert:
SystemTrayTest.NullDefaultViewIsNotRecorded failing (https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/builds/60026)

Original issue's description:
> Fix bug where removed (but not deleted) windows are not reflected in the tree properly
>
> When there are windows which are removed as children of some parent, and
> they still exist, OnWindowDestroying is not called because the child window
> was never destroyed. However, OnWindowTreeChanging is called, if
> the target has an old_parent, aka the parent it is being removed from.
> OnWindowTreeChanged is called if the target has a new_parent, aka the
> parent it is being added to.
>
> This CL removes nodes on OnWindowTreeChanging and adds them back in
> OnWindowTreeChanged if necessary. Since we only want them to trigger once,
> we check to see in either callback if the window that the callback is
> being called on is the old_parent or the new_parent.
>
> BUG=648701
>
> Committed: https://crrev.com/f555669c992a0bf22b8994d6a6f5a3f27e9f32c4
> Cr-Commit-Position: refs/heads/master@{#432366}

TBR=sadrul@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=648701
Committed: https://crrev.com/e75795096a81f406c92afce687ef4aa54900845f
Cr-Commit-Position: refs/heads/master@{#432374}

[Sarmad Hashmi, 2016-11-16 04:23:58]

The CQ bit was checked by mhashmi@chromium.org

[Sarmad Hashmi, 2016-11-16 04:23:58]

Created Revert of Fix bug where removed (but not deleted) windows are not
reflected in the tree properly

[commit-bot: I haz the power, 2016-11-16 04:24:22]

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-11-16 04:24:22]

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

[commit-bot: I haz the power, 2016-11-16 04:24:23]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Sarmad Hashmi, 2016-11-16 04:24:47]

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

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

Dry run: 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-11-16 04:25:14]

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

[commit-bot: I haz the power, 2016-11-16 04:25:15]

Dry run: This issue passed the CQ dry run.

[sadrul, 2016-11-16 04:42:46]

The CQ bit was checked by sadrul@chromium.org

[sadrul, 2016-11-16 04:42:47]

lgtm

[commit-bot: I haz the power, 2016-11-16 04:43:28]

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-11-16 04:47:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-16 04:49:34]

Description was changed from

==========
Revert of Fix bug where removed (but not deleted) windows are not reflected in
the tree properly (patchset #2 id:20001 of
https://codereview.chromium.org/2495533005/ )

Reason for revert:
SystemTrayTest.NullDefaultViewIsNotRecorded failing
(https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...)

Original issue's description:
> Fix bug where removed (but not deleted) windows are not reflected in the tree
properly
> 
> When there are windows which are removed as children of some parent, and
> they still exist, OnWindowDestroying is not called because the child window
> was never destroyed. However, OnWindowTreeChanging is called, if
> the target has an old_parent, aka the parent it is being removed from.
> OnWindowTreeChanged is called if the target has a new_parent, aka the
> parent it is being added to.
> 
> This CL removes nodes on OnWindowTreeChanging and adds them back in
> OnWindowTreeChanged if necessary. Since we only want them to trigger once,
> we check to see in either callback if the window that the callback is
> being called on is the old_parent or the new_parent.
> 
> BUG=648701
> 
> Committed: https://crrev.com/f555669c992a0bf22b8994d6a6f5a3f27e9f32c4
> Cr-Commit-Position: refs/heads/master@{#432366}

TBR=sadrul@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=648701
==========

to

==========
Revert of Fix bug where removed (but not deleted) windows are not reflected in
the tree properly (patchset #2 id:20001 of
https://codereview.chromium.org/2495533005/ )

Reason for revert:
SystemTrayTest.NullDefaultViewIsNotRecorded failing
(https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%281%29/bui...)

Original issue's description:
> Fix bug where removed (but not deleted) windows are not reflected in the tree
properly
>
> When there are windows which are removed as children of some parent, and
> they still exist, OnWindowDestroying is not called because the child window
> was never destroyed. However, OnWindowTreeChanging is called, if
> the target has an old_parent, aka the parent it is being removed from.
> OnWindowTreeChanged is called if the target has a new_parent, aka the
> parent it is being added to.
>
> This CL removes nodes on OnWindowTreeChanging and adds them back in
> OnWindowTreeChanged if necessary. Since we only want them to trigger once,
> we check to see in either callback if the window that the callback is
> being called on is the old_parent or the new_parent.
>
> BUG=648701
>
> Committed: https://crrev.com/f555669c992a0bf22b8994d6a6f5a3f27e9f32c4
> Cr-Commit-Position: refs/heads/master@{#432366}

TBR=sadrul@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=648701

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

[commit-bot: I haz the power, 2016-11-16 04:49:35]

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