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

blimp/client/app/linux/blimp_display_manager.h

 // Copyright 2015 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_APP_LINUX_BLIMP_DISPLAY_MANAGER_H_
 #define BLIMP_CLIENT_APP_LINUX_BLIMP_DISPLAY_MANAGER_H_
 
 #include <memory>
 
 #include "base/macros.h"
-#include "blimp/client/feature/compositor/blimp_compositor_manager.h"
+#include "blimp/client/core/compositor/compositor_deps_provider.h"

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

Needed?

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

Oh not anymore. Done.

+#include "blimp/client/core/contents/blimp_contents_manager.h"
 #include "ui/platform_window/platform_window_delegate.h"
 
 namespace gfx {
 class Size;
 }
 
 namespace ui {
 class PlatformWindow;
 }
 
 namespace blimp {
 namespace client {
-
-class BlimpCompositorManager;
+class BlimpContentsImpl;
+class CompositorDepsProvider;

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

Needed?

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

Done.

 class RenderWidgetFeature;
 class TabControlFeature;
 
 class BlimpDisplayManagerDelegate {
  public:
   virtual void OnClosed() = 0;
 };
 
-class BlimpDisplayManager : public ui::PlatformWindowDelegate,
-                            BlimpCompositorManagerClient {
+class BlimpDisplayManager : public ui::PlatformWindowDelegate {
  public:
   BlimpDisplayManager(const gfx::Size& window_size,
                       BlimpDisplayManagerDelegate* delegate,
                       RenderWidgetFeature* render_widget_feature,
                       TabControlFeature* tab_control_feature);
   ~BlimpDisplayManager() override;
 
   // ui::PlatformWindowDelegate:
   void OnBoundsChanged(const gfx::Rect& new_bounds) override;
   void OnDamageRect(const gfx::Rect& damaged_region) override;
   void DispatchEvent(ui::Event* event) override;
   void OnCloseRequest() override;
   void OnClosed() override;
   void OnWindowStateChanged(ui::PlatformWindowState new_state) override;
   void OnLostCapture() override;
   void OnAcceleratedWidgetAvailable(gfx::AcceleratedWidget widget,
                                     float device_pixel_ratio) override;
   void OnAcceleratedWidgetDestroyed() override;
   void OnActivationChanged(bool active) override;
 
  private:
-  // BlimpCompositorManagerClient implementation.
-  void OnSwapBuffersCompleted() override;
-  void DidCommitAndDrawFrame() override;
-
   float device_pixel_ratio_;
 
   BlimpDisplayManagerDelegate* delegate_;
   TabControlFeature* tab_control_feature_;
 
-  std::unique_ptr<BlimpCompositorManager> blimp_compositor_manager_;
+  BlimpContentsManager blimp_contents_manager_;

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

Will this become BlimpClientContext possibly in the future when
//blimp/client/app is an embedder?

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

Yup. That looks like where we should be asking blimp to create a contents.

+  std::unique_ptr<BlimpContentsImpl> blimp_contents_;
   std::unique_ptr<ui::PlatformWindow> platform_window_;
 
   DISALLOW_COPY_AND_ASSIGN(BlimpDisplayManager);
 };
 
 }  // namespace client
 }  // namespace blimp
 
 #endif  // BLIMP_CLIENT_APP_LINUX_BLIMP_DISPLAY_MANAGER_H_