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

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/ba5ad2e37e2277f82d560caf172992d739e1b7c6
Cr-Commit-Position: refs/heads/master@{#432568}

[Sarmad Hashmi, 2016-11-16 16:07:17]

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

[commit-bot: I haz the power, 2016-11-16 16:07:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Sarmad Hashmi, 2016-11-16 16:07:59]

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

[Sarmad Hashmi, 2016-11-16 16:07:59]

Trying to reland https://codereview.chromium.org/2476353002/ without the tests

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

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

[commit-bot: I haz the power, 2016-11-16 16:44:12]

Dry run: This issue passed the CQ dry run.

[Sarmad Hashmi, 2016-11-16 17:41:30]

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

[commit-bot: I haz the power, 2016-11-16 17:42:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Sarmad Hashmi, 2016-11-16 17:43:12]

Added tests back in but disabled on windows.

[sadrul, 2016-11-16 18:06:58]

lgtm

[commit-bot: I haz the power, 2016-11-16 18:16:58]

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

[commit-bot: I haz the power, 2016-11-16 18:16:59]

Dry run: This issue passed the CQ dry run.

[Sarmad Hashmi, 2016-11-16 18:25:43]

The CQ bit was checked by mhashmi@chromium.org

[commit-bot: I haz the power, 2016-11-16 18:26:41]

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 18:36:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-16 18:52:47]

Description was changed from

==========
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
==========

to

==========
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/ba5ad2e37e2277f82d560caf172992d739e1b7c6
Cr-Commit-Position: refs/heads/master@{#432568}
==========

[commit-bot: I haz the power, 2016-11-16 18:52:48]

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