Adds a CloneAndAnimate function to WindowManagerInternalClient

mojo/services/public/interfaces/window_manager/window_manager_internal.mojom

Index: mojo/services/public/interfaces/window_manager/window_manager_internal.mojom
diff --git a/mojo/services/public/interfaces/window_manager/window_manager_internal.mojom b/mojo/services/public/interfaces/window_manager/window_manager_internal.mojom
index 54867f9f44336036b8cb0431eabd0fccdab1d1db..6a5d642eb186736252feebeed1fd902eee679837 100644
--- a/mojo/services/public/interfaces/window_manager/window_manager_internal.mojom
+++ b/mojo/services/public/interfaces/window_manager/window_manager_internal.mojom
@@ -27,4 +27,11 @@ interface WindowManagerInternalClient {
 
   // Sets the native viewport size.
   SetViewportSize(mojo.Size size);
+
+  // Clones the tree rooted at |view_id|. When the animation completes the clone
+  // is destroyed.
+  // TODO(sky): add actual animation.
+  // TODO(sky): I think this only makes sense when destroying (view is
+  // already visible), should it be named to indicate this?
+  CloneAndAnimate(uint32 view_id) => (bool success);

[msw, 2014/11/18 23:37:41]

q: is the success response here valuable? (would the ViewManager do something
different based on the success or failure? or is it useful as a callback for
when the animation completes?) It seems odd if that has more value than the
success or failure of the functions above.

[sky, 2014/11/19 00:49:39]

I'm not sure if I'll end up needing the callback, so I nuked it.

 };