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

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

Issue 2918883002: cc: Get rid of SurfaceDestroyed and use SurfaceDiscarded instead
Patch Set: Updated Created 3 years, 6 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/mojo_frame_sink_manager.h" 5 #include "components/viz/frame_sinks/mojo_frame_sink_manager.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
121 client_->OnSurfaceCreated(surface_info); 121 client_->OnSurfaceCreated(surface_info);
122 } 122 }
123 123
124 void MojoFrameSinkManager::OnSurfaceDamaged(const cc::SurfaceId& surface_id, 124 void MojoFrameSinkManager::OnSurfaceDamaged(const cc::SurfaceId& surface_id,
125 const cc::BeginFrameAck& ack, 125 const cc::BeginFrameAck& ack,
126 bool* changed) {} 126 bool* changed) {}
127 127
128 void MojoFrameSinkManager::OnSurfaceDiscarded(const cc::SurfaceId& surface_id) { 128 void MojoFrameSinkManager::OnSurfaceDiscarded(const cc::SurfaceId& surface_id) {
129 } 129 }
130 130
131 void MojoFrameSinkManager::OnSurfaceDestroyed(const cc::SurfaceId& surface_id) {
132 }
133
134 void MojoFrameSinkManager::OnSurfaceDamageExpected( 131 void MojoFrameSinkManager::OnSurfaceDamageExpected(
135 const cc::SurfaceId& surface_id, 132 const cc::SurfaceId& surface_id,
136 const cc::BeginFrameArgs& args) {} 133 const cc::BeginFrameArgs& args) {}
137 134
138 void MojoFrameSinkManager::OnClientConnectionLost( 135 void MojoFrameSinkManager::OnClientConnectionLost(
139 const cc::FrameSinkId& frame_sink_id, 136 const cc::FrameSinkId& frame_sink_id,
140 bool destroy_compositor_frame_sink) { 137 bool destroy_compositor_frame_sink) {
141 DCHECK(thread_checker_.CalledOnValidThread()); 138 DCHECK(thread_checker_.CalledOnValidThread());
142 if (destroy_compositor_frame_sink) 139 if (destroy_compositor_frame_sink)
143 DestroyCompositorFrameSink(frame_sink_id); 140 DestroyCompositorFrameSink(frame_sink_id);
144 // TODO(fsamuel): Tell the frame sink manager host that the client connection 141 // TODO(fsamuel): Tell the frame sink manager host that the client connection
145 // has been lost so that it can drop its private connection and allow a new 142 // has been lost so that it can drop its private connection and allow a new
146 // client instance to create a new CompositorFrameSink. 143 // client instance to create a new CompositorFrameSink.
147 } 144 }
148 145
149 void MojoFrameSinkManager::OnPrivateConnectionLost( 146 void MojoFrameSinkManager::OnPrivateConnectionLost(
150 const cc::FrameSinkId& frame_sink_id, 147 const cc::FrameSinkId& frame_sink_id,
151 bool destroy_compositor_frame_sink) { 148 bool destroy_compositor_frame_sink) {
152 DCHECK(thread_checker_.CalledOnValidThread()); 149 DCHECK(thread_checker_.CalledOnValidThread());
153 if (destroy_compositor_frame_sink) 150 if (destroy_compositor_frame_sink)
154 DestroyCompositorFrameSink(frame_sink_id); 151 DestroyCompositorFrameSink(frame_sink_id);
155 } 152 }
156 153
157 } // namespace viz 154 } // namespace viz
OLDNEW
« cc/surfaces/display_scheduler.cc ('K') | « components/viz/frame_sinks/mojo_frame_sink_manager.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698