blimp: Move compositing, input and render widget feature to client/core.

blimp/client/core/contents/blimp_contents_manager.h

 // Copyright 2016 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 BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_
 #define BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_
 
+#include "blimp/client/core/compositor/compositor_deps_provider.h"
 #include "blimp/client/core/contents/blimp_contents_impl.h"
 
 namespace blimp {
 namespace client {
+class RenderWidgetFeature;
 
 // BlimpContentsManager does the real work of creating BlimpContentsImpl, and
 // then passes the ownership to the caller. It also owns the observers to
 // monitor the life time of the contents it creates.
 class BlimpContentsManager {
  public:
-  BlimpContentsManager();
+  BlimpContentsManager(bool use_internal_display,
+                       RenderWidgetFeature* render_widget_feature);
   ~BlimpContentsManager();
 
   std::unique_ptr<BlimpContentsImpl> CreateBlimpContents();
 
   // The caller can query the contents through its id.
   BlimpContentsImpl* GetBlimpContents(int id);
 
  private:
   class BlimpContentsDeletionObserver;
   friend class BlimpContentsDeletionObserver;
 
   // When creating the BlimpContentsImpl, an id is created for the content.
   int CreateBlimpContentsId();
 
   // When a BlimpContentsImpl is destroyed, its observer is able to retrieve the
   // id, and use this to destroy the observer entry from the observer_map_.
   void EraseObserverFromMap(int id);
   base::WeakPtr<BlimpContentsManager> GetWeakPtr();
 
   // BlimpContentsManager owns the BlimpContentsDeletionObserver for the
   // contents it creates, with the content id being the key to help manage the
   // lifetime of the observers.
   std::map<int, std::unique_ptr<BlimpContentsDeletionObserver>> observer_map_;
 
+  // Compositor dependencies shared across all BlimpContents.
+  CompositorDepsProvider compositor_deps_provider_;

[nyquist, 2016/08/16 23:14:57]

Does his happen to provide state that will be shared across profiles, etc? If
so, should this be a singleton?

[Khushal, 2016/08/18 02:01:46]

Yup, it has the compositor thread and the task graph runner, which should be
shared for all compositors. I've added it as a single instance, since I think
for Clank integration we would want the tests to be able to have a clean
shutdown and terminate all this state. So I've added methods to the public API
that will set up all such dependencies for us. Let me know if that should go
somewhere else instead.

+
+  // The network bridge for a BlimpContents to the WebContents on the engine.
+  RenderWidgetFeature* render_widget_feature_;
+
   base::WeakPtrFactory<BlimpContentsManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BlimpContentsManager);
 };
 
 }  // namespace client
 }  // namespace blimp
 
 #endif  // BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_