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

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

Issue 2938833002: Add SurfaceWillDraw notification (Closed)
Patch Set: 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
« no previous file with comments | « components/viz/frame_sinks/mojo_frame_sink_manager.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
140 const cc::FrameSinkId& frame_sink_id, 140 const cc::FrameSinkId& frame_sink_id,
141 bool destroy_compositor_frame_sink) { 141 bool destroy_compositor_frame_sink) {
142 DCHECK(thread_checker_.CalledOnValidThread()); 142 DCHECK(thread_checker_.CalledOnValidThread());
143 if (destroy_compositor_frame_sink) 143 if (destroy_compositor_frame_sink)
144 DestroyCompositorFrameSink(frame_sink_id); 144 DestroyCompositorFrameSink(frame_sink_id);
145 // TODO(fsamuel): Tell the frame sink manager host that the client connection 145 // TODO(fsamuel): Tell the frame sink manager host that the client connection
146 // has been lost so that it can drop its private connection and allow a new 146 // has been lost so that it can drop its private connection and allow a new
147 // client instance to create a new CompositorFrameSink. 147 // client instance to create a new CompositorFrameSink.
148 } 148 }
149 149
150 void OnSurfaceWillDraw(const cc::SurfaceId& surface_id) {}
rjkroege 2017/06/14 13:59:12 why is this not in MojoFrameSinkManager::?
Fady Samuel 2017/06/14 15:15:25 Yea, I'm kind of surprised this compiles?
gklassen 2017/06/14 15:16:35 Done.
151
150 void MojoFrameSinkManager::OnPrivateConnectionLost( 152 void MojoFrameSinkManager::OnPrivateConnectionLost(
151 const cc::FrameSinkId& frame_sink_id, 153 const cc::FrameSinkId& frame_sink_id,
152 bool destroy_compositor_frame_sink) { 154 bool destroy_compositor_frame_sink) {
153 DCHECK(thread_checker_.CalledOnValidThread()); 155 DCHECK(thread_checker_.CalledOnValidThread());
154 if (destroy_compositor_frame_sink) 156 if (destroy_compositor_frame_sink)
155 DestroyCompositorFrameSink(frame_sink_id); 157 DestroyCompositorFrameSink(frame_sink_id);
156 } 158 }
157 159
158 } // namespace viz 160 } // namespace viz
OLDNEW
« no previous file with comments | « 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