cc: Remove Display::SetExternalViewport.

content/renderer/android/synchronous_compositor_frame_sink.cc

 // Copyright 2013 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.
 
 #include "content/renderer/android/synchronous_compositor_frame_sink.h"
 
 #include <vector>
 
 #include "base/auto_reset.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "cc/output/compositor_frame.h"
 #include "cc/output/compositor_frame_sink_client.h"
 #include "cc/output/context_provider.h"
 #include "cc/output/output_surface.h"
 #include "cc/output/renderer_settings.h"
 #include "cc/output/software_output_device.h"
 #include "cc/output/texture_mailbox_deleter.h"
+#include "cc/quads/render_pass.h"
+#include "cc/quads/surface_draw_quad.h"
 #include "cc/surfaces/display.h"
 #include "cc/surfaces/surface_factory.h"
 #include "cc/surfaces/surface_id_allocator.h"
 #include "cc/surfaces/surface_manager.h"
 #include "content/common/android/sync_compositor_messages.h"
 #include "content/renderer/android/synchronous_compositor_filter.h"
 #include "content/renderer/android/synchronous_compositor_registry.h"
 #include "content/renderer/gpu/frame_swap_message_queue.h"
 #include "content/renderer/render_thread_impl.h"
 #include "gpu/command_buffer/client/context_support.h"
@@ skipping 46 matching lines @@
 
   // cc::OutputSurface implementation.
   void EnsureBackbuffer() override {}
   void DiscardBackbuffer() override {}
   void BindFramebuffer() override {}
   void SwapBuffers(cc::CompositorFrame frame) override {}
   void Reshape(const gfx::Size& size,
                float scale_factor,
                const gfx::ColorSpace& color_space,
                bool has_alpha) override {
-    // Intentional no-op. Surface size controlled by embedder.
+    surface_size_ = size;
   }
   uint32_t GetFramebufferCopyTextureFormat() override { return 0; }
   cc::OverlayCandidateValidator* GetOverlayCandidateValidator() const override {
     return nullptr;
   }
   bool IsDisplayedAsOverlayPlane() const override { return false; }
   unsigned GetOverlayTextureId() const override { return 0; }
   bool SurfaceIsSuspendForRecycle() const override { return false; }
   bool HasExternalStencilTest() const override { return false; }
   void ApplyExternalStencil() override {}
-
-  void SetSurfaceSize(const gfx::Size surface_size) {
-    surface_size_ = surface_size;
-  }
 };
 
 SynchronousCompositorFrameSink::SynchronousCompositorFrameSink(
     scoped_refptr<cc::ContextProvider> context_provider,
     scoped_refptr<cc::ContextProvider> worker_context_provider,
     int routing_id,
     uint32_t compositor_frame_sink_id,
     std::unique_ptr<cc::BeginFrameSource> begin_frame_source,
     SynchronousCompositorRegistry* registry,
     scoped_refptr<FrameSwapMessageQueue> frame_swap_message_queue)
@@ skipping 52 matching lines @@
       base::Bind(&SynchronousCompositorFrameSink::DidActivatePendingTree,
                  base::Unretained(this)));
   registry_->RegisterCompositorFrameSink(routing_id_, this);
   registered_ = true;
 
   surface_manager_->RegisterFrameSinkId(kFrameSinkId);
   surface_manager_->RegisterSurfaceFactoryClient(kFrameSinkId, this);
 
   cc::RendererSettings software_renderer_settings;
 
-  std::unique_ptr<SoftwareOutputSurface> compositor_frame_sink(
-      new SoftwareOutputSurface(
-          base::MakeUnique<SoftwareDevice>(&current_sw_canvas_)));
-  software_compositor_frame_sink_ = compositor_frame_sink.get();
+  auto output_surface = base::MakeUnique<SoftwareOutputSurface>(
+      base::MakeUnique<SoftwareDevice>(&current_sw_canvas_));
+  software_output_surface_ = output_surface.get();
 
   // The shared_bitmap_manager and gpu_memory_buffer_manager here are null as
   // this Display is only used for resourcesless software draws, where no
   // resources are included in the frame swapped from the compositor. So there
   // is no need for these.
   display_.reset(new cc::Display(
       nullptr /* shared_bitmap_manager */,
       nullptr /* gpu_memory_buffer_manager */, software_renderer_settings,
-      nullptr /* begin_frame_source */, std::move(compositor_frame_sink),
+      nullptr /* begin_frame_source */, std::move(output_surface),
       nullptr /* scheduler */, nullptr /* texture_mailbox_deleter */));
   display_->Initialize(&display_client_, surface_manager_.get(), kFrameSinkId);
   display_->SetVisible(true);
   return true;
 }
 
 void SynchronousCompositorFrameSink::DetachFromClient() {
   DCHECK(CalledOnValidThread());
   client_->SetBeginFrameSource(nullptr);
   // Destroy the begin frame source on the same thread it was bound on.
   begin_frame_source_ = nullptr;
   if (registered_)
     registry_->UnregisterCompositorFrameSink(routing_id_, this);
   client_->SetTreeActivationCallback(base::Closure());
-  if (!delegated_surface_id_.is_null())
-    surface_factory_->Destroy(delegated_surface_id_);
+  if (!root_surface_id_.is_null()) {
+    surface_factory_->Destroy(root_surface_id_);
+    surface_factory_->Destroy(child_surface_id_);
+  }
   surface_manager_->UnregisterSurfaceFactoryClient(kFrameSinkId);
   surface_manager_->InvalidateFrameSinkId(kFrameSinkId);
-  software_compositor_frame_sink_ = nullptr;
+  software_output_surface_ = nullptr;
   display_ = nullptr;
   surface_factory_ = nullptr;
   surface_id_allocator_ = nullptr;
   surface_manager_ = nullptr;
   cc::CompositorFrameSink::DetachFromClient();
   CancelFallbackTick();
 }
 
 static void NoOpDrawCallback() {}
 
 void SynchronousCompositorFrameSink::SwapBuffers(cc::CompositorFrame frame) {
   DCHECK(CalledOnValidThread());
   DCHECK(sync_client_);
 
   if (fallback_tick_running_) {
     DCHECK(frame.delegated_frame_data->resource_list.empty());
     cc::ReturnedResourceArray return_resources;
     ReturnResources(return_resources);
     did_swap_ = true;
     return;
   }
 
   cc::CompositorFrame swap_frame;
 
   if (in_software_draw_) {
     // The frame we send to the client is actually just the metadata. Preserve
     // the |frame| for the software path below.
     swap_frame.metadata = frame.metadata.Clone();
 
-    if (delegated_surface_id_.is_null()) {
-      delegated_surface_id_ = surface_id_allocator_->GenerateId();
-      surface_factory_->Create(delegated_surface_id_);
+    if (root_surface_id_.is_null()) {
+      root_surface_id_ = surface_id_allocator_->GenerateId();
+      surface_factory_->Create(root_surface_id_);
+      child_surface_id_ = surface_id_allocator_->GenerateId();
+      surface_factory_->Create(child_surface_id_);
     }
 
-    display_->SetSurfaceId(delegated_surface_id_,
+    display_->SetSurfaceId(root_surface_id_,
                            frame.metadata.device_scale_factor);
 
-    gfx::Size frame_size =
-        frame.delegated_frame_data->render_pass_list.back()->output_rect.size();
-    display_->Resize(frame_size);
+    // The layer compositor should be giving a frame that covers the
+    // |sw_viewport_for_current_draw_| but at 0,0.
+    gfx::Size child_size = sw_viewport_for_current_draw_.size();
+    DCHECK(gfx::Rect(child_size) ==
+           frame.delegated_frame_data->render_pass_list.back()->output_rect);
 
-    surface_factory_->SubmitCompositorFrame(
-        delegated_surface_id_, std::move(frame), base::Bind(&NoOpDrawCallback));
+    // Make a size that covers from 0,0 and includes the area coming from the
+    // layer compositor.
+    gfx::Size display_size(sw_viewport_for_current_draw_.right(),
+                           sw_viewport_for_current_draw_.bottom());
+    display_->Resize(display_size);
+
+    // The offset for the child frame relative to the origin of the canvas being
+    // drawn into.
+    gfx::Transform child_transform;
+    child_transform.Translate(
+        gfx::Vector2dF(sw_viewport_for_current_draw_.OffsetFromOrigin()));
+
+    // Make a root frame that embeds the frame coming from the layer compositor
+    // and positions it based on the provided viewport.
+    // TODO(danakj): We could apply the transform here instead of passing it to
+    // the CompositorFrameSink client too? (We'd have to do the same for
+    // hardware frames in SurfacesInstance?)
+    cc::CompositorFrame embed_frame;
+    embed_frame.delegated_frame_data =
+        base::MakeUnique<cc::DelegatedFrameData>();
+    embed_frame.delegated_frame_data->render_pass_list.push_back(
+        cc::RenderPass::Create());
+
+    // The embedding RenderPass covers the entire Display's area.
+    const auto& embed_render_pass =
+        embed_frame.delegated_frame_data->render_pass_list.back();
+    embed_render_pass->SetAll(cc::RenderPassId(1, 1), gfx::Rect(display_size),
+                              gfx::Rect(display_size), gfx::Transform(), false);
+
+    // The RenderPass has a single SurfaceDrawQuad (and SharedQuadState for it).
+    auto* shared_quad_state =
+        embed_render_pass->CreateAndAppendSharedQuadState();
+    auto* surface_quad =
+        embed_render_pass->CreateAndAppendDrawQuad<cc::SurfaceDrawQuad>();
+    shared_quad_state->SetAll(
+        child_transform, child_size, gfx::Rect(child_size),
+        gfx::Rect() /* clip_rect */, false /* is_clipped */, 1.f /* opacity */,
+        SkXfermode::kSrcOver_Mode, 0 /* sorting_context_id */);
+    surface_quad->SetNew(shared_quad_state, gfx::Rect(child_size),
+                         gfx::Rect(child_size), child_surface_id_);
+
+    surface_factory_->SubmitCompositorFrame(child_surface_id_, std::move(frame),
+                                            base::Bind(&NoOpDrawCallback));
+    surface_factory_->SubmitCompositorFrame(root_surface_id_,
+                                            std::move(embed_frame),
+                                            base::Bind(&NoOpDrawCallback));
     display_->DrawAndSwap();
   } else {
     // For hardware draws we send the whole frame to the client so it can draw
     // the content in it.
     swap_frame = std::move(frame);
   }
 
   sync_client_->SwapBuffers(compositor_frame_sink_id_, std::move(swap_frame));
   DeliverMessages();
   did_swap_ = true;
@@ skipping 54 matching lines @@
 
   base::AutoReset<SkCanvas*> canvas_resetter(&current_sw_canvas_, canvas);
 
   SkIRect canvas_clip;
   canvas->getClipDeviceBounds(&canvas_clip);
   gfx::Rect viewport = gfx::SkIRectToRect(canvas_clip);
 
   gfx::Transform transform(gfx::Transform::kSkipInitialization);
   transform.matrix() = canvas->getTotalMatrix();  // Converts 3x3 matrix to 4x4.
 
+  // We will resize the Display to ensure it covers the entire |viewport|, so
+  // save it for later.
+  sw_viewport_for_current_draw_ = viewport;
+
   base::AutoReset<bool> set_in_software_draw(&in_software_draw_, true);
-  display_->SetExternalViewport(viewport);
-  display_->SetExternalClip(viewport);
-  software_compositor_frame_sink_->SetSurfaceSize(
-      gfx::SkISizeToSize(canvas->getBaseLayerSize()));
   InvokeComposite(transform, viewport);
 }
 
 void SynchronousCompositorFrameSink::InvokeComposite(
     const gfx::Transform& transform,
     const gfx::Rect& viewport) {
+  did_swap_ = false;
+  // Adjust transform so that the layer compositor draws the |viewport| rect
+  // at its origin. The offset of the |viewport| we pass to the layer compositor
+  // is ignored for drawing, so its okay to not match the transform.
+  // TODO(danakj): Why do we pass a viewport origin and then not really use it
+  // (only for comparing to the viewport passed in
+  // SetExternalTilePriorityConstraints), surely this could be more clear?
   gfx::Transform adjusted_transform = transform;
   adjusted_transform.matrix().postTranslate(-viewport.x(), -viewport.y(), 0);
-  did_swap_ = false;
   client_->OnDraw(adjusted_transform, viewport, in_software_draw_);
 
   if (did_swap_) {
     // This must happen after unwinding the stack and leaving the compositor.
     // Usually it is a separate task but we just defer it until OnDraw completes
     // instead.
     client_->DidSwapBuffersComplete();
   }
 }
 
@@ skipping 61 matching lines @@
   client_->ReclaimResources(resources);
 }
 
 void SynchronousCompositorFrameSink::SetBeginFrameSource(
     cc::BeginFrameSource* begin_frame_source) {
   // Software output is synchronous and doesn't use a BeginFrameSource.
   NOTREACHED();
 }
 
 }  // namespace content