aw: Share cc::Display instance

android_webview/browser/hardware_renderer.cc

Index: android_webview/browser/hardware_renderer.cc
diff --git a/android_webview/browser/hardware_renderer.cc b/android_webview/browser/hardware_renderer.cc
index acac292f4be6b8311fa1c56334dccd5710442810..5f7c115c7efa2a681e00ea5843b725fe7fe36bea 100644
--- a/android_webview/browser/hardware_renderer.cc
+++ b/android_webview/browser/hardware_renderer.cc
@@ -13,6 +13,7 @@
 #include "android_webview/browser/parent_compositor_draw_constraints.h"
 #include "android_webview/browser/parent_output_surface.h"
 #include "android_webview/browser/render_thread_manager.h"
+#include "android_webview/browser/surfaces_instance.h"
 #include "android_webview/public/browser/draw_gl.h"
 #include "base/auto_reset.h"
 #include "base/strings/string_number_conversions.h"
@@ -22,7 +23,6 @@
 #include "cc/output/renderer_settings.h"
 #include "cc/output/texture_mailbox_deleter.h"
 #include "cc/quads/shared_quad_state.h"
-#include "cc/quads/surface_draw_quad.h"
 #include "cc/scheduler/begin_frame_source.h"
 #include "cc/surfaces/display.h"
 #include "cc/surfaces/display_scheduler.h"
@@ -40,57 +40,23 @@ namespace android_webview {
 HardwareRenderer::HardwareRenderer(RenderThreadManager* state)
     : render_thread_manager_(state),
       last_egl_context_(eglGetCurrentContext()),
-      gl_surface_(new AwGLSurface),
+      surfaces_(SurfacesInstance::GetOrCreateInstance()),
       last_committed_output_surface_id_(0u),
-      last_submitted_output_surface_id_(0u),
-      output_surface_(NULL) {
+      last_submitted_output_surface_id_(0u) {
   DCHECK(last_egl_context_);
 
-  cc::RendererSettings settings;
-
-  // Should be kept in sync with compositor_impl_android.cc.
-  settings.allow_antialiasing = false;
-  settings.highp_threshold_min = 2048;
-
-  // Webview does not own the surface so should not clear it.
-  settings.should_clear_root_render_pass = false;
-
-  surface_manager_.reset(new cc::SurfaceManager);
-  surface_id_allocator_.reset(new cc::SurfaceIdAllocator(1));
-  surface_id_allocator_->RegisterSurfaceIdNamespace(surface_manager_.get());
-  surface_manager_->RegisterSurfaceFactoryClient(
+  surface_id_allocator_ = surfaces_->CreateSurfaceIdAllocator();
+  surfaces_->GetSurfaceManager()->RegisterSurfaceFactoryClient(
       surface_id_allocator_->id_namespace(), this);
-
-  std::unique_ptr<cc::BeginFrameSource> begin_frame_source(
-      new cc::StubBeginFrameSource);
-  std::unique_ptr<cc::TextureMailboxDeleter> texture_mailbox_deleter(
-      new cc::TextureMailboxDeleter(nullptr));
-  std::unique_ptr<ParentOutputSurface> output_surface_holder(
-      new ParentOutputSurface(AwRenderThreadContextProvider::Create(
-          gl_surface_, DeferredGpuCommandService::GetInstance())));
-  output_surface_ = output_surface_holder.get();
-  std::unique_ptr<cc::DisplayScheduler> scheduler(new cc::DisplayScheduler(
-      begin_frame_source.get(), nullptr,
-      output_surface_holder->capabilities().max_frames_pending));
-  display_.reset(new cc::Display(
-      surface_manager_.get(), nullptr /* shared_bitmap_manager */,
-      nullptr /* gpu_memory_buffer_manager */, settings,
-      surface_id_allocator_->id_namespace(), std::move(begin_frame_source),
-      std::move(output_surface_holder), std::move(scheduler),
-      std::move(texture_mailbox_deleter)));
-  display_->Initialize(this);
 }
 
 HardwareRenderer::~HardwareRenderer() {
   // Must reset everything before |surface_factory_| to ensure all
   // resources are returned before resetting.
-  if (!root_id_.is_null())
-    surface_factory_->Destroy(root_id_);
   if (!child_id_.is_null())
     surface_factory_->Destroy(child_id_);
-  display_.reset();
   surface_factory_.reset();
-  surface_manager_->UnregisterSurfaceFactoryClient(
+  surfaces_->GetSurfaceManager()->UnregisterSurfaceFactoryClient(
       surface_id_allocator_->id_namespace());
 
   // Reset draw constraints.
@@ -134,12 +100,9 @@ void HardwareRenderer::DrawGL(AwDrawGLInfo* draw_info,
   if (child_frame_.get() && child_frame_->frame.get()) {
     if (!compositor_id_.Equals(child_frame_->compositor_id) ||
         last_submitted_output_surface_id_ != child_frame_->output_surface_id) {
-      if (!root_id_.is_null())
-        surface_factory_->Destroy(root_id_);
       if (!child_id_.is_null())
         surface_factory_->Destroy(child_id_);
 
-      root_id_ = cc::SurfaceId();
       child_id_ = cc::SurfaceId();
 
       // This will return all the resources to the previous compositor.
@@ -147,7 +110,7 @@ void HardwareRenderer::DrawGL(AwDrawGLInfo* draw_info,
       compositor_id_ = child_frame_->compositor_id;
       last_submitted_output_surface_id_ = child_frame_->output_surface_id;
       surface_factory_.reset(
-          new cc::SurfaceFactory(surface_manager_.get(), this));
+          new cc::SurfaceFactory(surfaces_->GetSurfaceManager(), this));
     }
 
     std::unique_ptr<cc::CompositorFrame> child_compositor_frame =
@@ -197,6 +160,7 @@ void HardwareRenderer::DrawGL(AwDrawGLInfo* draw_info,
                  draw_info->clip_right - draw_info->clip_left,
                  draw_info->clip_bottom - draw_info->clip_top);
 
+  /*
   // Create a frame with a single SurfaceDrawQuad referencing the child
   // Surface and transformed using the given transform.
   std::unique_ptr<cc::RenderPass> render_pass = cc::RenderPass::Create();
@@ -221,19 +185,23 @@ void HardwareRenderer::DrawGL(AwDrawGLInfo* draw_info,
   std::unique_ptr<cc::CompositorFrame> frame(new cc::CompositorFrame);
   frame->delegated_frame_data = std::move(delegated_frame);
 
+  cc::Display* display = surfaces_->GetDisplay();
   if (root_id_.is_null()) {
     root_id_ = surface_id_allocator_->GenerateId();
     surface_factory_->Create(root_id_);
-    display_->SetSurfaceId(root_id_, 1.f);
+    display->SetSurfaceId(root_id_, 1.f);
   }
   surface_factory_->SubmitCompositorFrame(root_id_, std::move(frame),
                                           cc::SurfaceFactory::DrawCallback());
 
-  display_->Resize(viewport);
+  display->Resize(viewport);
 
-  output_surface_->SetGLState(gl_state);
-  display_->SetExternalClip(clip);
-  display_->DrawAndSwap();
+  surfaces_->GetOutputSurface()->SetGLState(gl_state);
+  display->SetExternalClip(clip);
+  display->DrawAndSwap();
+  */
+  surfaces_->DrawAndSwap(viewport, clip, transform, frame_size_, child_id_,
+                         gl_state);
 }
 
 void HardwareRenderer::ReturnResources(
@@ -249,7 +217,7 @@ void HardwareRenderer::SetBeginFrameSource(
 
 void HardwareRenderer::SetBackingFrameBufferObject(
     int framebuffer_binding_ext) {
-  gl_surface_->SetBackingFrameBufferObject(framebuffer_binding_ext);
+  surfaces_->SetBackingFrameBufferObject(framebuffer_binding_ext);
 }
 
 void HardwareRenderer::ReturnResourcesInChildFrame() {