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

Side by Side Diff: blimp/client/core/compositor/blimp_compositor.h

Issue 2383373002: Reduce SurfaceIdAllocator usage and tie SurfaceFactory to a single FrameSinkId (Closed)
Patch Set: Rebased Created 4 years, 2 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 #ifndef BLIMP_CLIENT_CORE_COMPOSITOR_BLIMP_COMPOSITOR_H_ 5 #ifndef BLIMP_CLIENT_CORE_COMPOSITOR_BLIMP_COMPOSITOR_H_
6 #define BLIMP_CLIENT_CORE_COMPOSITOR_BLIMP_COMPOSITOR_H_ 6 #define BLIMP_CLIENT_CORE_COMPOSITOR_BLIMP_COMPOSITOR_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 171 matching lines...) Expand 10 before | Expand all | Expand 10 after
182 // of the count. 182 // of the count.
183 void CheckPendingCommitCounts(bool flush); 183 void CheckPendingCommitCounts(bool flush);
184 184
185 // The unique identifier for the render widget for this compositor. 185 // The unique identifier for the render widget for this compositor.
186 const int render_widget_id_; 186 const int render_widget_id_;
187 187
188 BlimpCompositorClient* client_; 188 BlimpCompositorClient* client_;
189 189
190 BlimpCompositorDependencies* compositor_dependencies_; 190 BlimpCompositorDependencies* compositor_dependencies_;
191 191
192 cc::FrameSinkId frame_sink_id_;
193
192 std::unique_ptr<cc::LayerTreeHost> host_; 194 std::unique_ptr<cc::LayerTreeHost> host_;
193 195
194 // The SurfaceFactory is bound to the lifetime of the |proxy_client_|. When 196 // The SurfaceFactory is bound to the lifetime of the |proxy_client_|. When
195 // detached, the surface factory will be destroyed. 197 // detached, the surface factory will be destroyed.
196 std::unique_ptr<cc::SurfaceFactory> surface_factory_; 198 std::unique_ptr<cc::SurfaceFactory> surface_factory_;
197 base::WeakPtr<BlimpCompositorFrameSinkProxyClient> proxy_client_; 199 base::WeakPtr<BlimpCompositorFrameSinkProxyClient> proxy_client_;
198 200
199 // Whether or not |host_| has asked for a new CompositorFrameSink. 201 // Whether or not |host_| has asked for a new CompositorFrameSink.
200 bool compositor_frame_sink_request_pending_; 202 bool compositor_frame_sink_request_pending_;
201 203
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
234 236
235 base::WeakPtrFactory<BlimpCompositor> weak_ptr_factory_; 237 base::WeakPtrFactory<BlimpCompositor> weak_ptr_factory_;
236 238
237 DISALLOW_COPY_AND_ASSIGN(BlimpCompositor); 239 DISALLOW_COPY_AND_ASSIGN(BlimpCompositor);
238 }; 240 };
239 241
240 } // namespace client 242 } // namespace client
241 } // namespace blimp 243 } // namespace blimp
242 244
243 #endif // BLIMP_CLIENT_CORE_COMPOSITOR_BLIMP_COMPOSITOR_H_ 245 #endif // BLIMP_CLIENT_CORE_COMPOSITOR_BLIMP_COMPOSITOR_H_
OLDNEW
« no previous file with comments | « android_webview/browser/surfaces_instance.cc ('k') | blimp/client/core/compositor/blimp_compositor.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698