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

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

Powered by Google App Engine
This is Rietveld 408576698