Provide an API to the example window manager app that supports closing a window.

Provide an API to the example window manager app that supports closing a window.
Listen for clicks in the embedded app and close the corresponding view when clicked.

R=sky@chromium.org
http://crbug.com/365012

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275322

[Ben Goodger (Google), 2014-06-05 22:35:42]

https://codereview.chromium.org/311423003/diff/100001/mojo/examples/embedded_...
File mojo/examples/embedded_app/embedded_app.cc (right):

https://codereview.chromium.org/311423003/diff/100001/mojo/examples/embedded_...
mojo/examples/embedded_app/embedded_app.cc:63: views_to_reap_[node] = old_view;
I have to do this because on the server side when a node is deleted you reset
the active view, and tell the client about it, so by the time I get to
OnRootRemoved below I've lost the association from node->view.

I think we should consider not notifying the client in this case so that the
client has an opportunity to clean up the view.

[sky, 2014-06-05 22:51:31]

LGTM

https://codereview.chromium.org/311423003/diff/100001/mojo/examples/embedded_...
File mojo/examples/embedded_app/embedded_app.cc (right):

https://codereview.chromium.org/311423003/diff/100001/mojo/examples/embedded_...
mojo/examples/embedded_app/embedded_app.cc:63: views_to_reap_[node] = old_view;
On 2014/06/05 22:35:42, Ben Goodger (Google) wrote:
> I have to do this because on the server side when a node is deleted you reset
> the active view, and tell the client about it, so by the time I get to
> OnRootRemoved below I've lost the association from node->view.
> 
> I think we should consider not notifying the client in this case so that the
> client has an opportunity to clean up the view.

Sounds reasonable. I'll do that for the client that originated the change.

[Ben Goodger (Google), 2014-06-06 04:50:42]

Committed patchset #8 manually as r275322 (presubmit successful).