| Index: cc/test/test_compositor_frame_sink.cc
|
| diff --git a/cc/test/test_compositor_frame_sink.cc b/cc/test/test_compositor_frame_sink.cc
|
| index 1037fba98a6a5c9c482c9f7985c795b92cb919ba..52d50347dc7b9878c077b5331afd14f580f0efac 100644
|
| --- a/cc/test/test_compositor_frame_sink.cc
|
| +++ b/cc/test/test_compositor_frame_sink.cc
|
| @@ -35,6 +35,8 @@ TestCompositorFrameSink::TestCompositorFrameSink(
|
| surface_id_allocator_(new SurfaceIdAllocator(frame_sink_id_)),
|
| surface_factory_(
|
| new SurfaceFactory(frame_sink_id_, surface_manager_.get(), this)),
|
| + display_context_shared_with_compositor_(
|
| + display_output_surface->context_provider() == context_provider()),
|
| weak_ptrs_(this) {
|
| std::unique_ptr<SyntheticBeginFrameSource> begin_frame_source;
|
| std::unique_ptr<DisplayScheduler> scheduler;
|
| @@ -53,8 +55,6 @@ TestCompositorFrameSink::TestCompositorFrameSink(
|
| begin_frame_source.get(), task_runner,
|
| display_output_surface->capabilities().max_frames_pending));
|
| }
|
| - const bool context_shared_with_compositor =
|
| - display_output_surface->context_provider() == context_provider();
|
| display_.reset(
|
| new Display(shared_bitmap_manager, gpu_memory_buffer_manager,
|
| renderer_settings, std::move(begin_frame_source),
|
| @@ -66,8 +66,9 @@ TestCompositorFrameSink::TestCompositorFrameSink(
|
| // the Display. But we allow tests to disable this to mimic an out-of-process
|
| // Display.
|
| capabilities_.can_force_reclaim_resources = !force_disable_reclaim_resources;
|
| - capabilities_.delegated_sync_points_required =
|
| - !context_shared_with_compositor;
|
| + // Always use sync tokens so that code paths in resource provider that deal
|
| + // with sync tokens are tested.
|
| + capabilities_.delegated_sync_points_required = true;
|
| }
|
|
|
| TestCompositorFrameSink::~TestCompositorFrameSink() {
|
| @@ -83,11 +84,10 @@ bool TestCompositorFrameSink::BindToClient(CompositorFrameSinkClient* client) {
|
| if (!CompositorFrameSink::BindToClient(client))
|
| return false;
|
|
|
| - // We want the Display's OutputSurface to hear about lost context, and since
|
| - // this shares a context with it (when delegated_sync_points_required is
|
| - // false), we should not be listening for lost context callbacks on the
|
| - // context here.
|
| - if (!capabilities_.delegated_sync_points_required && context_provider())
|
| + // We want the Display's OutputSurface to hear about lost context, and when
|
| + // this shares a context with it we should not be listening for lost context
|
| + // callbacks on the context here.
|
| + if (display_context_shared_with_compositor_ && context_provider())
|
| context_provider()->SetLostContextCallback(base::Closure());
|
|
|
| surface_manager_->RegisterFrameSinkId(frame_sink_id_);
|
|
|