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

blimp/client/core/contents/blimp_contents_manager.h

Index: blimp/client/core/contents/blimp_contents_manager.h
diff --git a/blimp/client/core/contents/blimp_contents_manager.h b/blimp/client/core/contents/blimp_contents_manager.h
index ea70960ffbccffc7eb48072e9d5784c2de681fab..653476e3210bebaf14681e36160a3256f8d5cdc3 100644
--- a/blimp/client/core/contents/blimp_contents_manager.h
+++ b/blimp/client/core/contents/blimp_contents_manager.h
@@ -5,17 +5,20 @@
 #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_direct_rendering,
+                       RenderWidgetFeature* render_widget_feature);
   ~BlimpContentsManager();
 
   std::unique_ptr<BlimpContentsImpl> CreateBlimpContents();
@@ -40,6 +43,12 @@ class BlimpContentsManager {
   // lifetime of the observers.
   std::map<int, std::unique_ptr<BlimpContentsDeletionObserver>> observer_map_;
 
+  // Compositor dependencies shared across all BlimpContents.
+  CompositorDepsProvider compositor_deps_provider_;
+
+  // The network bridge for a BlimpContents to the WebContents on the engine.
+  RenderWidgetFeature* render_widget_feature_;

[nyquist, 2016/08/12 07:34:28]

Do we want to comment on the lifetime of this here?

[Khushal, 2016/08/13 00:03:22]

I guess it should be implicit that this is expected to outlive the
BlimpContentsManager. In case its not then we should add a comment stating that
it can be null. WDYT?

+
   base::WeakPtrFactory<BlimpContentsManager> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BlimpContentsManager);