Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2658)

Unified Diff: content/test/layouttest_support.cc

Issue 2161323002: cc: Make LayerTreeHostImpl unittests use a delegating output surface. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: display-lthi-tests: . Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/trees/layer_tree_host_impl_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/test/layouttest_support.cc
diff --git a/content/test/layouttest_support.cc b/content/test/layouttest_support.cc
index e50757b14a791a747006036d4565a24250d2f355..816a48084ecbe9214fcdb4c55d9614fd6863afca 100644
--- a/content/test/layouttest_support.cc
+++ b/content/test/layouttest_support.cc
@@ -12,9 +12,6 @@
#include "base/memory/ptr_util.h"
#include "base/strings/string_util.h"
#include "build/build_config.h"
-#include "cc/output/texture_mailbox_deleter.h"
-#include "cc/scheduler/begin_frame_source.h"
-#include "cc/scheduler/delay_based_time_source.h"
#include "cc/test/pixel_test_output_surface.h"
#include "cc/test/test_delegating_output_surface.h"
#include "components/scheduler/test/renderer_scheduler_test_support.h"
@@ -214,36 +211,19 @@ class LayoutTestDependenciesImpl : public LayoutTestDependencies {
nullptr, flipped_output_surface));
auto* task_runner = deps->GetCompositorImplThreadTaskRunner().get();
- bool synchronous_compositor = !task_runner;
+ bool synchronous_composite = !task_runner;
if (!task_runner)
task_runner = base::ThreadTaskRunnerHandle::Get().get();
- std::unique_ptr<cc::SyntheticBeginFrameSource> begin_frame_source;
- std::unique_ptr<cc::DisplayScheduler> scheduler;
- if (!synchronous_compositor) {
- begin_frame_source.reset(new cc::DelayBasedBeginFrameSource(
- base::MakeUnique<cc::DelayBasedTimeSource>(task_runner)));
- scheduler.reset(new cc::DisplayScheduler(
- begin_frame_source.get(), task_runner,
- display_output_surface->capabilities().max_frames_pending));
- }
-
cc::LayerTreeSettings settings =
RenderWidgetCompositor::GenerateLayerTreeSettings(
*base::CommandLine::ForCurrentProcess(), deps, 1.f);
- std::unique_ptr<cc::Display> display(new cc::Display(
- deps->GetSharedBitmapManager(), deps->GetGpuMemoryBufferManager(),
- settings.renderer_settings, std::move(begin_frame_source),
- std::move(display_output_surface), std::move(scheduler),
- base::MakeUnique<cc::TextureMailboxDeleter>(task_runner)));
-
- const bool context_shared_with_compositor = false;
- const bool allow_force_reclaim_resources = false;
return base::MakeUnique<cc::TestDelegatingOutputSurface>(
std::move(compositor_context_provider),
- std::move(worker_context_provider), std::move(display),
- context_shared_with_compositor, allow_force_reclaim_resources);
+ std::move(worker_context_provider), std::move(display_output_surface),
+ deps->GetSharedBitmapManager(), deps->GetGpuMemoryBufferManager(),
+ settings.renderer_settings, task_runner, synchronous_composite);
}
};
« no previous file with comments | « cc/trees/layer_tree_host_impl_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698