Adds a CloneAndAnimate function to WindowManagerInternalClient

mojo/services/window_manager/window_manager_app.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MOJO_SERVICES_WINDOW_MANAGER_WINDOW_MANAGER_APP_H_
 #define MOJO_SERVICES_WINDOW_MANAGER_WINDOW_MANAGER_APP_H_
 
 #include <set>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 65 matching lines @@
   FocusController* focus_controller() { return focus_controller_.get(); }
 
   void InitFocus(scoped_ptr<FocusRules> rules);
 
   // WindowManagerImpl::Embed() forwards to this. If connected to ViewManager
   // then forwards to delegate, otherwise waits for connection to establish then
   // forwards.
   void Embed(const String& url,
              InterfaceRequest<ServiceProvider> service_provider);
 
+  // Calls through to WindowManagerInternalClient::CloneAndAnimate(); see mojom

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

nit "to WindowManagerInternalClient; see "

+  // for description.
+  void CloneAndAnimate(Id view);

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

Is this actually called somewhere?

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

I was adding it now to cut down on size of future patches. But I nuked it for
now.

+
   // Overridden from ApplicationDelegate:
   void Initialize(ApplicationImpl* impl) override;
   bool ConfigureIncomingConnection(ApplicationConnection* connection) override;
 
  private:
   // TODO(sky): rename this. Connections is ambiguous.
   typedef std::set<WindowManagerImpl*> Connections;
   typedef std::set<Id> RegisteredViewIdSet;
 
   struct PendingEmbed;
@@ skipping 65 matching lines @@
   scoped_ptr<ViewManagerClient> view_manager_client_;
 
   scoped_ptr<ViewEventDispatcher> view_event_dispatcher_;
 
   DISALLOW_COPY_AND_ASSIGN(WindowManagerApp);
 };
 
 }  // namespace mojo
 
 #endif  // MOJO_SERVICES_WINDOW_MANAGER_WINDOW_MANAGER_APP_H_