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

Side by Side Diff: components/exo/compositor_frame_sink.cc

Issue 2807653003: Move Work From CompositorFrameSinkSupport() To Init() (Closed)
Patch Set: Set needs_sync_points Created 3 years, 8 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/exo/compositor_frame_sink.h" 5 #include "components/exo/compositor_frame_sink.h"
6 6
7 #include "base/memory/ptr_util.h" 7 #include "base/memory/ptr_util.h"
8 #include "cc/surfaces/surface.h" 8 #include "cc/surfaces/surface.h"
9 #include "cc/surfaces/surface_manager.h" 9 #include "cc/surfaces/surface_manager.h"
10 #include "components/exo/compositor_frame_sink_holder.h" 10 #include "components/exo/compositor_frame_sink_holder.h"
11 #include "mojo/public/cpp/bindings/strong_binding.h" 11 #include "mojo/public/cpp/bindings/strong_binding.h"
12 12
13 namespace exo { 13 namespace exo {
14 14
15 //////////////////////////////////////////////////////////////////////////////// 15 ////////////////////////////////////////////////////////////////////////////////
16 // ExoComopositorFrameSink, public: 16 // ExoComopositorFrameSink, public:
17 17
18 CompositorFrameSink::CompositorFrameSink(const cc::FrameSinkId& frame_sink_id, 18 CompositorFrameSink::CompositorFrameSink(const cc::FrameSinkId& frame_sink_id,
19 cc::SurfaceManager* surface_manager, 19 cc::SurfaceManager* surface_manager,
20 CompositorFrameSinkHolder* client) 20 CompositorFrameSinkHolder* client)
21 : support_(this, 21 : support_(this,
22 surface_manager,
23 frame_sink_id, 22 frame_sink_id,
24 false /* is_root */, 23 false /* is_root */,
25 true /* handles_frame_sink_id_invalidation */, 24 true /* handles_frame_sink_id_invalidation */),
26 true /* needs_sync_points */), 25 client_(client) {
27 client_(client) {} 26 constexpr bool needs_sync_points = true;
27 support_.Init(surface_manager, needs_sync_points);
28 }
28 29
29 CompositorFrameSink::~CompositorFrameSink() {} 30 CompositorFrameSink::~CompositorFrameSink() {}
30 31
31 //////////////////////////////////////////////////////////////////////////////// 32 ////////////////////////////////////////////////////////////////////////////////
32 // cc::mojom::MojoCompositorFrameSink overrides: 33 // cc::mojom::MojoCompositorFrameSink overrides:
33 34
34 void CompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) { 35 void CompositorFrameSink::SetNeedsBeginFrame(bool needs_begin_frame) {
35 support_.SetNeedsBeginFrame(needs_begin_frame); 36 support_.SetNeedsBeginFrame(needs_begin_frame);
36 } 37 }
37 38
(...skipping 22 matching lines...) Expand all
60 void CompositorFrameSink::OnBeginFrame(const cc::BeginFrameArgs& args) { 61 void CompositorFrameSink::OnBeginFrame(const cc::BeginFrameArgs& args) {
61 client_->OnBeginFrame(args); 62 client_->OnBeginFrame(args);
62 } 63 }
63 64
64 void CompositorFrameSink::ReclaimResources( 65 void CompositorFrameSink::ReclaimResources(
65 const cc::ReturnedResourceArray& resources) { 66 const cc::ReturnedResourceArray& resources) {
66 client_->ReclaimResources(resources); 67 client_->ReclaimResources(resources);
67 } 68 }
68 69
69 } // namespace exo 70 } // namespace exo
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698