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

Side by Side Diff: components/viz/frame_sinks/gpu_compositor_frame_sink.cc

Issue 2807653003: Move Work From CompositorFrameSinkSupport() To Init() (Closed)
Patch Set: More Init Created 3 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/viz/frame_sinks/gpu_compositor_frame_sink.h" 5 #include "components/viz/frame_sinks/gpu_compositor_frame_sink.h"
6 6
7 namespace viz { 7 namespace viz {
8 8
9 GpuCompositorFrameSink::GpuCompositorFrameSink( 9 GpuCompositorFrameSink::GpuCompositorFrameSink(
10 GpuCompositorFrameSinkDelegate* delegate, 10 GpuCompositorFrameSinkDelegate* delegate,
11 cc::SurfaceManager* surface_manager, 11 cc::SurfaceManager* surface_manager,
12 const cc::FrameSinkId& frame_sink_id, 12 const cc::FrameSinkId& frame_sink_id,
13 cc::mojom::MojoCompositorFrameSinkRequest request, 13 cc::mojom::MojoCompositorFrameSinkRequest request,
14 cc::mojom::MojoCompositorFrameSinkPrivateRequest 14 cc::mojom::MojoCompositorFrameSinkPrivateRequest
15 compositor_frame_sink_private_request, 15 compositor_frame_sink_private_request,
16 cc::mojom::MojoCompositorFrameSinkClientPtr client) 16 cc::mojom::MojoCompositorFrameSinkClientPtr client)
17 : delegate_(delegate), 17 : delegate_(delegate),
18 support_(base::MakeUnique<cc::CompositorFrameSinkSupport>( 18 support_(base::MakeUnique<cc::CompositorFrameSinkSupport>(
19 this, 19 this,
20 surface_manager, 20 surface_manager,
21 frame_sink_id, 21 frame_sink_id,
22 false /* is_root */, 22 false /* is_root */,
23 true /* handles_frame_sink_id_invalidation */, 23 true /* handles_frame_sink_id_invalidation */,
24 true /* needs_sync_points */)), 24 true /* needs_sync_points */)),
25 client_(std::move(client)), 25 client_(std::move(client)),
26 compositor_frame_sink_binding_(this, std::move(request)), 26 compositor_frame_sink_binding_(this, std::move(request)),
27 compositor_frame_sink_private_binding_( 27 compositor_frame_sink_private_binding_(
28 this, 28 this,
29 std::move(compositor_frame_sink_private_request)) { 29 std::move(compositor_frame_sink_private_request)) {
30 support_->Init();
30 compositor_frame_sink_binding_.set_connection_error_handler(base::Bind( 31 compositor_frame_sink_binding_.set_connection_error_handler(base::Bind(
31 &GpuCompositorFrameSink::OnClientConnectionLost, base::Unretained(this))); 32 &GpuCompositorFrameSink::OnClientConnectionLost, base::Unretained(this)));
32 compositor_frame_sink_private_binding_.set_connection_error_handler( 33 compositor_frame_sink_private_binding_.set_connection_error_handler(
33 base::Bind(&GpuCompositorFrameSink::OnPrivateConnectionLost, 34 base::Bind(&GpuCompositorFrameSink::OnPrivateConnectionLost,
34 base::Unretained(this))); 35 base::Unretained(this)));
35 } 36 }
36 37
37 GpuCompositorFrameSink::~GpuCompositorFrameSink() {} 38 GpuCompositorFrameSink::~GpuCompositorFrameSink() {}
38 39
39 void GpuCompositorFrameSink::EvictFrame() { 40 void GpuCompositorFrameSink::EvictFrame() {
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 } 97 }
97 98
98 void GpuCompositorFrameSink::OnPrivateConnectionLost() { 99 void GpuCompositorFrameSink::OnPrivateConnectionLost() {
99 private_connection_lost_ = true; 100 private_connection_lost_ = true;
100 // Request destruction of |this| only if both connections are lost. 101 // Request destruction of |this| only if both connections are lost.
101 delegate_->OnPrivateConnectionLost(support_->frame_sink_id(), 102 delegate_->OnPrivateConnectionLost(support_->frame_sink_id(),
102 client_connection_lost_); 103 client_connection_lost_);
103 } 104 }
104 105
105 } // namespace viz 106 } // namespace viz
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698