Makes display roots be destroyed by client

Makes display roots be destroyed by client

Prior to this patch when a display was destroyed the client would get
a delete notification for the root. This is problematic as the client
may need to delete the root after a delay (as happens in ash). This
patch changes it so the client is notified of the display removal and
it's up to the client to then delete the window when they want to.

BUG=647512
TEST=covered by tests
R=sadrul@chromium.org

Committed: https://crrev.com/54ff2ccef79fedc2a991f9c9b66e64d0c781cfc5
Cr-Commit-Position: refs/heads/master@{#419658}

[sky, 2016-09-19 21:04:31]

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

[commit-bot: I haz the power, 2016-09-19 21:05:10]

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

[sky, 2016-09-19 21:19:04]

Description was changed from

==========
Makes display roots be destroyed by client

Prior to this patch when a display was destroyed the client would get
a delete notification for the root. This is problematic as the client
may need to delete the root after a delay (as happens in ash). This
patch changes it so the client is notified of the display removal and
it's up to the client to then delete the window when they want to.

BUG=647512
TEST=covered by tests
R=sadrul@chromium.org
==========

to

==========
Makes display roots be destroyed by client

Prior to this patch when a display was destroyed the client would get
a delete notification for the root. This is problematic as the client
may need to delete the root after a delay (as happens in ash). This
patch changes it so the client is notified of the display removal and
it's up to the client to then delete the window when they want to.

BUG=647512
TEST=covered by tests
R=sadrul@chromium.org
==========

[sky, 2016-09-19 21:19:04]

sky@chromium.org changed reviewers:
+ tsepez@chromium.org

[sky, 2016-09-19 21:19:12]

+tsepez for mojom

[Tom Sepez, 2016-09-19 21:54:08]

mojom LGTM

[commit-bot: I haz the power, 2016-09-19 21:57:26]

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

[commit-bot: I haz the power, 2016-09-19 21:57:27]

Dry run: This issue passed the CQ dry run.

[sadrul, 2016-09-20 01:37:10]

lgtm

The 'clients' referred to in the CL description are window managers, right? When
reviewing the CL, I started looking at DisplayBinding, but it looks like that's
used only in tests?

[sky, 2016-09-20 02:38:43]

You are right that this only applies to the WindowManager. DisplayBinding is
only used if connected to by way of WindowTreeHostFactory, which no one really
does.

[sky, 2016-09-20 02:38:56]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-09-20 02:39:11]

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-09-20 02:44:58]

Description was changed from

==========
Makes display roots be destroyed by client

Prior to this patch when a display was destroyed the client would get
a delete notification for the root. This is problematic as the client
may need to delete the root after a delay (as happens in ash). This
patch changes it so the client is notified of the display removal and
it's up to the client to then delete the window when they want to.

BUG=647512
TEST=covered by tests
R=sadrul@chromium.org
==========

to

==========
Makes display roots be destroyed by client

Prior to this patch when a display was destroyed the client would get
a delete notification for the root. This is problematic as the client
may need to delete the root after a delay (as happens in ash). This
patch changes it so the client is notified of the display removal and
it's up to the client to then delete the window when they want to.

BUG=647512
TEST=covered by tests
R=sadrul@chromium.org
==========

[commit-bot: I haz the power, 2016-09-20 02:44:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-20 02:47:38]

Description was changed from

==========
Makes display roots be destroyed by client

Prior to this patch when a display was destroyed the client would get
a delete notification for the root. This is problematic as the client
may need to delete the root after a delay (as happens in ash). This
patch changes it so the client is notified of the display removal and
it's up to the client to then delete the window when they want to.

BUG=647512
TEST=covered by tests
R=sadrul@chromium.org
==========

to

==========
Makes display roots be destroyed by client

Prior to this patch when a display was destroyed the client would get
a delete notification for the root. This is problematic as the client
may need to delete the root after a delay (as happens in ash). This
patch changes it so the client is notified of the display removal and
it's up to the client to then delete the window when they want to.

BUG=647512
TEST=covered by tests
R=sadrul@chromium.org

Committed: https://crrev.com/54ff2ccef79fedc2a991f9c9b66e64d0c781cfc5
Cr-Commit-Position: refs/heads/master@{#419658}
==========

[commit-bot: I haz the power, 2016-09-20 02:47:39]

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