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

blimp/client/app/android/blimp_view.h

Index: blimp/client/app/android/blimp_view.h
diff --git a/blimp/client/app/android/blimp_view.h b/blimp/client/app/android/blimp_view.h
index fa60dec50657243c9888f81bf011805b8a3b3abe..8719fcb84e03e060601c5da36c82ce137b046d3e 100644
--- a/blimp/client/app/android/blimp_view.h
+++ b/blimp/client/app/android/blimp_view.h
@@ -9,7 +9,8 @@
 
 #include "base/android/jni_android.h"
 #include "base/macros.h"
-#include "blimp/client/app/android/blimp_compositor_manager_android.h"
+#include "base/memory/weak_ptr.h"
+#include "blimp/client/core/contents/blimp_contents_manager.h"
 #include "ui/gfx/native_widget_types.h"
 
 namespace gfx {
@@ -18,13 +19,13 @@ class Size;
 
 namespace blimp {
 namespace client {
-
+class BlimpContents;
 class RenderWidgetFeature;
 
 // The native component of org.chromium.blimp.BlimpView.  This builds and
 // maintains a BlimpCompositorAndroid and handles notifying the compositor of
 // SurfaceView surface changes (size, creation, destruction, etc.).
-class BlimpView : public BlimpCompositorManagerClient {
+class BlimpView {
  public:
   static bool RegisterJni(JNIEnv* env);
 
@@ -91,17 +92,15 @@ class BlimpView : public BlimpCompositorManagerClient {
   virtual ~BlimpView();
 
   void ReleaseAcceleratedWidget();
-
-  // BlimpCompositorManagerClient implementation.
-  void OnSwapBuffersCompleted() override;
-  void DidCommitAndDrawFrame() override;
+  void OnSwapBuffersCompleted();
 
   // Reference to the Java object which owns this class.
   base::android::ScopedJavaGlobalRef<jobject> java_obj_;
 
   const float device_scale_factor_;
 
-  std::unique_ptr<BlimpCompositorManagerAndroid> compositor_manager_;
+  BlimpContentsManager blimp_contents_manager_;
+  std::unique_ptr<BlimpContentsImpl> blimp_contents_;

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

I know this is temporary, but could you add a comment explaining the
relationship between these two and that we only support a single BlimpContents?

[Khushal, 2016/08/18 03:16:32]

Done. Though I think more detailed comments about these should go on
BlimpContentsManager and BlimpContents instead.

 
   // The format of the current surface owned by |compositor_|.  See
   // android.graphics.PixelFormat.java.
@@ -109,6 +108,9 @@ class BlimpView : public BlimpCompositorManagerClient {
 
   gfx::AcceleratedWidget window_;
 
+  // BlimpConnectionStatistics* blimp_connection_statistics_;
+  base::WeakPtrFactory<BlimpView> weak_ptr_factory_;
+
   DISALLOW_COPY_AND_ASSIGN(BlimpView);
 };