Remove ForceReclaimResources

content/test/layouttest_support.cc

 // Copyright (c) 2012 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/public/test/layouttest_support.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/callback.h"
@@ skipping 337 matching lines @@
       task_runner = base::ThreadTaskRunnerHandle::Get().get();
 
     cc::LayerTreeSettings settings =
         RenderWidgetCompositor::GenerateLayerTreeSettings(
             *base::CommandLine::ForCurrentProcess(), deps, 1.f);
 
     auto compositor_frame_sink = base::MakeUnique<cc::TestCompositorFrameSink>(
         std::move(compositor_context_provider),
         std::move(worker_context_provider), nullptr /* shared_bitmap_manager */,
         gpu_memory_buffer_manager, settings.renderer_settings, task_runner,
-        synchronous_composite, false /* force_disable_reclaim_resources */);
+        synchronous_composite);
     compositor_frame_sink->SetClient(this);
+    // Always swap (regardless of damage) to ensure that we can capture frames.
+    compositor_frame_sink->SetAlwaysSwap();

[danakj, 2017/01/11 19:28:24]

Instead we could have the code that inserts a copy request also set damage on
the viewport?

[ericrk, 2017/01/11 23:10:50]

Added the invalidation to the code that triggers the copy request in
RenderWidgetCompositor.

     compositor_frame_sinks_[routing_id] = compositor_frame_sink.get();
     return std::move(compositor_frame_sink);
   }
 
   std::unique_ptr<cc::SwapPromise> RequestCopyOfOutput(
       int32_t routing_id,
       std::unique_ptr<cc::CopyOutputRequest> request) override {
     // Note that we can't immediately check compositor_frame_sinks_, since it
     // may not have been created yet. Instead, we wait until OnCommit to find
     // the currently active CompositorFrameSink for the given RenderWidget
@@ skipping 229 matching lines @@
 
 void ForceTextInputStateUpdateForRenderFrame(RenderFrame* frame) {
   if (auto* render_widget =
           static_cast<RenderFrameImpl*>(frame)->GetRenderWidget()) {
     render_widget->UpdateTextInputState(ShowIme::IF_NEEDED,
                                         ChangeSource::FROM_NON_IME);
   }
 }
 
 }  // namespace content