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_direct_rendering,
+                       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_;
+
+  // 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);
 };
 
 }  // namespace client
 }  // namespace blimp
 
 #endif  // BLIMP_CLIENT_CORE_CONTENTS_BLIMP_CONTENTS_MANAGER_H_