Fixes bug in handling window deletion originating from server

Fixes bug in handling window deletion originating from server

When the server tells the client a window was deleted the client would
schedule deletion again with the server. This can cause problems as
clients generally assume deletion succeeds and check otherwise. I'm
also adding logging to window_tree in this patch. I think this is
going to prove helpful for tracking down other similar issues and so
I'm leading it in (DVLOG).

BUG=659155
TEST=covered by test
R=msw@chromium.org
TBR=msw@chromium.org

Committed: https://crrev.com/82255b3581f426db199117c3449bfe00bc38cb26
Cr-Commit-Position: refs/heads/master@{#433315}

[sky, 2016-11-18 20:43:34]

I'm TBRing as you reviewed this in https://codereview.chromium.org/2500973002/ .
I'm spinning this fix out of the other patch in hopes of keeping that patch to
just test_wm. Also, I'm hitting another issue with converting test_wm and I
don't want to expand the other patch more.

[sky, 2016-11-18 20:43:40]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-11-18 20:44:57]

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

[msw, 2016-11-18 21:06:11]

still lgtm

[commit-bot: I haz the power, 2016-11-18 22:34:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-18 22:36:09]

Description was changed from

==========
Fixes bug in handling window deletion originating from server

When the server tells the client a window was deleted the client would
schedule deletion again with the server. This can cause problems as
clients generally assume deletion succeeds and check otherwise. I'm
also adding logging to window_tree in this patch. I think this is
going to prove helpful for tracking down other similar issues and so
I'm leading it in (DVLOG).

BUG=659155
TEST=covered by test
R=msw@chromium.org
TBR=msw@chromium.org
==========

to

==========
Fixes bug in handling window deletion originating from server

When the server tells the client a window was deleted the client would
schedule deletion again with the server. This can cause problems as
clients generally assume deletion succeeds and check otherwise. I'm
also adding logging to window_tree in this patch. I think this is
going to prove helpful for tracking down other similar issues and so
I'm leading it in (DVLOG).

BUG=659155
TEST=covered by test
R=msw@chromium.org
TBR=msw@chromium.org

Committed: https://crrev.com/82255b3581f426db199117c3449bfe00bc38cb26
Cr-Commit-Position: refs/heads/master@{#433315}
==========

[commit-bot: I haz the power, 2016-11-18 22:36:10]

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