OLD | NEW |
---|---|
(Empty) | |
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 | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef COMPONENTS_EXO_COMPOSITOR_FRAME_SINK_HOLDER_H_ | |
6 #define COMPONENTS_EXO_COMPOSITOR_FRAME_SINK_HOLDER_H_ | |
7 | |
8 #include <list> | |
9 #include <map> | |
10 #include <memory> | |
11 | |
12 #include "cc/ipc/mojo_compositor_frame_sink.mojom.h" | |
13 #include "cc/resources/single_release_callback.h" | |
14 #include "cc/resources/transferable_resource.h" | |
15 #include "cc/scheduler/begin_frame_source.h" | |
16 #include "components/exo/compositor_frame_sink.h" | |
17 #include "mojo/public/cpp/bindings/binding.h" | |
18 | |
19 namespace exo { | |
20 class Surface; | |
21 | |
22 using FrameCallback = base::Callback<void(base::TimeTicks frame_time)>; | |
reveman
2016/12/07 00:46:57
Can you move this into the class and just above Ac
Alex Z.
2016/12/07 20:09:35
Done.
| |
23 | |
24 class CompositorFrameSinkHolder | |
reveman
2016/12/07 00:46:57
Can you add a short class description here that ex
Alex Z.
2016/12/07 20:09:35
Done.
| |
25 : public base::RefCounted<CompositorFrameSinkHolder>, | |
26 public cc::ExternalBeginFrameSourceClient, | |
27 public cc::mojom::MojoCompositorFrameSinkClient, | |
28 public cc::BeginFrameObserver { | |
29 public: | |
30 CompositorFrameSinkHolder( | |
31 cc::mojom::MojoCompositorFrameSinkPtr compositor_frame_sink, | |
32 base::WeakPtr<Surface> surface, | |
33 cc::mojom::MojoCompositorFrameSinkClientRequest request); | |
34 | |
35 bool HasReleaseCallbacks(cc::ResourceId id); | |
reveman
2016/12/07 00:46:57
HasReleaseCallbacksForResource or ResourceHasRelea
Alex Z.
2016/12/07 20:09:35
Done.
| |
36 bool IsReleaseCallbacksEmpty(); | |
reveman
2016/12/07 00:46:57
and this can then be named HasReleaseCallbacks
Alex Z.
2016/12/07 20:09:35
It turns out this is no longer being used so I rem
| |
37 void AddResourceReleaseCallback( | |
38 cc::ResourceId id, | |
39 std::unique_ptr<cc::SingleReleaseCallback> callback); | |
40 cc::mojom::MojoCompositorFrameSink* GetCompositorFrameSink() { | |
reveman
2016/12/07 00:46:57
nit: blankline before this? doesn't seem like it s
Alex Z.
2016/12/07 20:09:35
Done.
| |
41 return compositor_frame_sink_.get(); | |
42 } | |
43 | |
44 void ActivateFrameCallbacks(std::list<FrameCallback>& frame_callbacks); | |
reveman
2016/12/07 00:46:57
"std::list<FrameCallback>* frame_callbacks" as it'
Alex Z.
2016/12/07 20:09:35
Done.
| |
45 void CancelFrameCallbacks(); | |
46 | |
47 void SetNeedsBeginFrame(bool needs_begin_frame); | |
48 | |
49 // cc::mojom::MojoCompositorFrameSinkClient: | |
reveman
2016/12/07 00:46:57
Overridden from cc::Mojo..
Alex Z.
2016/12/07 21:13:43
Done.
| |
50 void DidReceiveCompositorFrameAck() override; | |
51 void OnBeginFrame(const cc::BeginFrameArgs& args) override; | |
52 void ReclaimResources(const cc::ReturnedResourceArray& resources) override; | |
53 void WillDrawSurface() override; | |
54 | |
55 // cc::BeginFrameObserver: | |
reveman
2016/12/07 00:46:57
Overridden from..
Alex Z.
2016/12/07 20:09:35
Done.
| |
56 const cc::BeginFrameArgs& LastUsedBeginFrameArgs() const override; | |
57 void OnBeginFrameSourcePausedChanged(bool paused) override; | |
58 | |
59 private: | |
60 friend class base::RefCounted<CompositorFrameSinkHolder>; | |
61 | |
62 ~CompositorFrameSinkHolder() override; | |
63 | |
64 void UpdateNeedsBeginFrame(); | |
65 | |
66 // cc::ExternalBeginFrameSouceClient: | |
reveman
2016/12/07 00:46:57
"Overridden from.." and don't make this private
Alex Z.
2016/12/07 20:09:35
Done.
| |
67 void OnNeedsBeginFrames(bool needs_begin_frames) override; | |
68 | |
69 using ResourceReleaseCallbacks = | |
70 std::map<int, | |
71 std::pair<scoped_refptr<CompositorFrameSinkHolder>, | |
72 std::unique_ptr<cc::SingleReleaseCallback>>>; | |
73 ResourceReleaseCallbacks release_callbacks_; | |
74 | |
75 cc::mojom::MojoCompositorFrameSinkPtr compositor_frame_sink_; | |
76 | |
reveman
2016/12/07 00:46:57
nit: you can remove the blanklines between member
Alex Z.
2016/12/07 20:09:35
Done.
| |
77 base::WeakPtr<Surface> surface_; | |
78 | |
79 std::list<FrameCallback> active_frame_callbacks_; | |
80 | |
81 std::unique_ptr<cc::ExternalBeginFrameSource> begin_frame_source_; | |
82 | |
83 bool needs_begin_frame_ = false; | |
84 | |
85 cc::BeginFrameArgs last_begin_frame_args_; | |
86 | |
87 mojo::Binding<cc::mojom::MojoCompositorFrameSinkClient> binding_; | |
88 | |
89 DISALLOW_COPY_AND_ASSIGN(CompositorFrameSinkHolder); | |
90 }; | |
91 | |
92 } // namespace exo | |
93 | |
94 #endif // COMPONENTS_EXO_COMPOSITOR_FRAME_SINK_HOLDER_H_ | |
OLD | NEW |