Prepare CompositorFrameSinkSupport To Be Used By DirectCompositorFrameSink

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"
@@ skipping 150 matching lines @@
     return false;
 
   DCHECK(begin_frame_source_);
   client_->SetBeginFrameSource(begin_frame_source_.get());
   client_->SetMemoryPolicy(memory_policy_);
   client_->SetTreeActivationCallback(
       base::Bind(&SynchronousCompositorFrameSink::DidActivatePendingTree,
                  base::Unretained(this)));
   registry_->RegisterCompositorFrameSink(routing_id_, this);
 
+  const bool root_submits_to_display_compositor = true;

[danakj, 2017/02/14 16:54:01]

We were talking about names with fsamuel the other day and he mentioned that
this concept of "offscreen" really means "not-root". I wonder if that applies
here too.. in that children also submit frames to a display compositor too
right? They are just not the root of the display compositor. So I wonder if
these should be named in those terms (root vs child)?

+  const bool child_submits_to_display_compositor = false;
+  const bool handles_frame_sink_id_invalidation = true;
+  const bool needs_sync_points = true;
   root_support_.reset(new cc::CompositorFrameSinkSupport(
       this, surface_manager_.get(), kRootFrameSinkId,
-      true /* submits_to_display_compositor */));
+      root_submits_to_display_compositor, handles_frame_sink_id_invalidation,
+      needs_sync_points));
   child_support_.reset(new cc::CompositorFrameSinkSupport(
       this, surface_manager_.get(), kChildFrameSinkId,
-      false /* submits_to_display_compositor */));
+      child_submits_to_display_compositor, handles_frame_sink_id_invalidation,
+      needs_sync_points));
 
   cc::RendererSettings software_renderer_settings;
 
   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
@@ skipping 271 matching lines @@
 
 void SynchronousCompositorFrameSink::ReclaimResources(
     const cc::ReturnedResourceArray& resources) {
   DCHECK(resources.empty());
   client_->ReclaimResources(resources);
 }
 
 void SynchronousCompositorFrameSink::WillDrawSurface() {}
 
 }  // namespace content