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

Side by Side Diff: components/display_compositor/gpu_compositor_frame_sink.cc

Issue 2651213004: Revert of Decouple GpuCompositorFrameSink from DisplayCompositor (Closed)
Patch Set: Created 3 years, 10 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "components/display_compositor/gpu_compositor_frame_sink.h"
6
7 #include "cc/surfaces/surface_reference.h"
8
9 namespace display_compositor {
10
11 GpuCompositorFrameSink::GpuCompositorFrameSink(
12 GpuCompositorFrameSinkDelegate* delegate,
13 cc::SurfaceManager* surface_manager,
14 const cc::FrameSinkId& frame_sink_id,
15 std::unique_ptr<cc::Display> display,
16 std::unique_ptr<cc::BeginFrameSource> begin_frame_source,
17 cc::mojom::MojoCompositorFrameSinkPrivateRequest
18 compositor_frame_sink_private_request,
19 cc::mojom::MojoCompositorFrameSinkClientPtr client)
20 : delegate_(delegate),
21 support_(this,
22 surface_manager,
23 frame_sink_id,
24 std::move(display),
25 std::move(begin_frame_source)),
26 surface_manager_(surface_manager),
27 surface_tracker_(frame_sink_id),
28 client_(std::move(client)),
29 compositor_frame_sink_private_binding_(
30 this,
31 std::move(compositor_frame_sink_private_request)) {
32 compositor_frame_sink_private_binding_.set_connection_error_handler(
33 base::Bind(&GpuCompositorFrameSink::OnPrivateConnectionLost,
34 base::Unretained(this)));
35 }
36
37 GpuCompositorFrameSink::~GpuCompositorFrameSink() {
38 // For display root surfaces, remove the reference from top level root to
39 // indicate the display root surface is no longer visible.
40 if (support_.display() && surface_tracker_.current_surface_id().is_valid()) {
41 const cc::SurfaceId top_level_root_surface_id =
42 surface_manager_->GetRootSurfaceId();
43 std::vector<cc::SurfaceReference> references_to_remove{cc::SurfaceReference(
44 top_level_root_surface_id, surface_tracker_.current_surface_id())};
45 surface_manager_->RemoveSurfaceReferences(references_to_remove);
46 }
47 }
48
49 void GpuCompositorFrameSink::EvictFrame() {
50 support_.EvictFrame();
51 }
52
53 void GpuCompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) {
54 support_.SetNeedsBeginFrame(needs_begin_frame);
55 }
56
57 void GpuCompositorFrameSink::SubmitCompositorFrame(
58 const cc::LocalFrameId& local_frame_id,
59 cc::CompositorFrame frame) {
60 cc::SurfaceId start_surface_id = surface_tracker_.current_surface_id();
61 surface_tracker_.UpdateReferences(local_frame_id,
62 frame.metadata.referenced_surfaces);
63 // TODO(kylechar): Move adding top-level root references to
64 // GpuDisplayCompositorFrameSink.
65
66 support_.SubmitCompositorFrame(local_frame_id, std::move(frame));
67
68 // Get the list of surfaces to add/remove from |surface_tracker_| so we can
69 // append to them before adding/removing.
70 std::vector<cc::SurfaceReference>& references_to_add =
71 surface_tracker_.references_to_add();
72 std::vector<cc::SurfaceReference>& references_to_remove =
73 surface_tracker_.references_to_remove();
74
75 // Append TLR references for the display root surfaces when display root
76 // surface changes.
77 if (support_.display() &&
78 start_surface_id != surface_tracker_.current_surface_id()) {
79 const cc::SurfaceId top_level_root_surface_id =
80 surface_manager_->GetRootSurfaceId();
81
82 // The first frame will not have a valid |start_surface_id| and there will
83 // be no surface to remove.
84 if (start_surface_id.local_frame_id().is_valid()) {
85 references_to_remove.push_back(
86 cc::SurfaceReference(top_level_root_surface_id, start_surface_id));
87 }
88
89 references_to_add.push_back(cc::SurfaceReference(
90 top_level_root_surface_id, surface_tracker_.current_surface_id()));
91 }
92
93 if (!references_to_add.empty())
94 surface_manager_->AddSurfaceReferences(references_to_add);
95 if (!references_to_remove.empty())
96 surface_manager_->RemoveSurfaceReferences(references_to_remove);
97 }
98
99 void GpuCompositorFrameSink::Require(const cc::LocalFrameId& local_frame_id,
100 const cc::SurfaceSequence& sequence) {
101 support_.Require(local_frame_id, sequence);
102 }
103
104 void GpuCompositorFrameSink::Satisfy(const cc::SurfaceSequence& sequence) {
105 support_.Satisfy(sequence);
106 }
107
108 void GpuCompositorFrameSink::DidReceiveCompositorFrameAck() {
109 if (client_)
110 client_->DidReceiveCompositorFrameAck();
111 }
112
113 void GpuCompositorFrameSink::AddChildFrameSink(
114 const cc::FrameSinkId& child_frame_sink_id) {
115 support_.AddChildFrameSink(child_frame_sink_id);
116 }
117
118 void GpuCompositorFrameSink::RemoveChildFrameSink(
119 const cc::FrameSinkId& child_frame_sink_id) {
120 support_.RemoveChildFrameSink(child_frame_sink_id);
121 }
122
123 void GpuCompositorFrameSink::OnBeginFrame(const cc::BeginFrameArgs& args) {
124 if (client_)
125 client_->OnBeginFrame(args);
126 }
127
128 void GpuCompositorFrameSink::ReclaimResources(
129 const cc::ReturnedResourceArray& resources) {
130 if (client_)
131 client_->ReclaimResources(resources);
132 }
133
134 void GpuCompositorFrameSink::WillDrawSurface() {
135 if (client_)
136 client_->WillDrawSurface();
137 }
138
139 void GpuCompositorFrameSink::OnClientConnectionLost() {
140 client_connection_lost_ = true;
141 // Request destruction of |this| only if both connections are lost.
142 delegate_->OnClientConnectionLost(support_.frame_sink_id(),
143 private_connection_lost_);
144 }
145
146 void GpuCompositorFrameSink::OnPrivateConnectionLost() {
147 private_connection_lost_ = true;
148 // Request destruction of |this| only if both connections are lost.
149 delegate_->OnPrivateConnectionLost(support_.frame_sink_id(),
150 client_connection_lost_);
151 }
152
153 } // namespace display_compositor
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698