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

Side by Side Diff: cc/surfaces/display_scheduler.h

Issue 2854163003: [cc] Plumb BeginFrameAcks through SurfaceManager to DisplayScheduler. (Closed)
Patch Set: fix clang compile error Created 3 years, 7 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 | « cc/surfaces/display.cc ('k') | cc/surfaces/display_scheduler.cc » ('j') | 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 #ifndef CC_SURFACES_DISPLAY_SCHEDULER_H_ 5 #ifndef CC_SURFACES_DISPLAY_SCHEDULER_H_
6 #define CC_SURFACES_DISPLAY_SCHEDULER_H_ 6 #define CC_SURFACES_DISPLAY_SCHEDULER_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/cancelable_callback.h" 10 #include "base/cancelable_callback.h"
11 #include "base/containers/flat_map.h"
11 #include "base/macros.h" 12 #include "base/macros.h"
12 #include "base/memory/ref_counted.h" 13 #include "base/memory/ref_counted.h"
13 #include "base/single_thread_task_runner.h" 14 #include "base/single_thread_task_runner.h"
14 #include "cc/output/renderer_settings.h" 15 #include "cc/output/renderer_settings.h"
15 #include "cc/scheduler/begin_frame_source.h" 16 #include "cc/scheduler/begin_frame_source.h"
16 #include "cc/surfaces/surface_id.h" 17 #include "cc/surfaces/surface_id.h"
17 #include "cc/surfaces/surfaces_export.h" 18 #include "cc/surfaces/surfaces_export.h"
18 19
19 namespace cc { 20 namespace cc {
20 21
21 class BeginFrameSource; 22 class BeginFrameSource;
23 class SurfaceInfo;
22 24
23 class CC_SURFACES_EXPORT DisplaySchedulerClient { 25 class CC_SURFACES_EXPORT DisplaySchedulerClient {
24 public: 26 public:
25 virtual ~DisplaySchedulerClient() {} 27 virtual ~DisplaySchedulerClient() {}
26 28
27 virtual bool DrawAndSwap() = 0; 29 virtual bool DrawAndSwap() = 0;
30 virtual bool SurfaceHasUndrawnFrame(const SurfaceId& surface_id) const = 0;
28 }; 31 };
29 32
30 class CC_SURFACES_EXPORT DisplayScheduler : public BeginFrameObserverBase { 33 class CC_SURFACES_EXPORT DisplayScheduler : public BeginFrameObserverBase {
31 public: 34 public:
32 DisplayScheduler(base::SingleThreadTaskRunner* task_runner, 35 DisplayScheduler(base::SingleThreadTaskRunner* task_runner,
33 int max_pending_swaps); 36 int max_pending_swaps);
34 ~DisplayScheduler() override; 37 ~DisplayScheduler() override;
35 38
36 void SetClient(DisplaySchedulerClient* client); 39 void SetClient(DisplaySchedulerClient* client);
37 void SetBeginFrameSource(BeginFrameSource* begin_frame_source); 40 void SetBeginFrameSource(BeginFrameSource* begin_frame_source);
38 41
39 void SetVisible(bool visible); 42 void SetVisible(bool visible);
40 void SetRootSurfaceResourcesLocked(bool locked); 43 void SetRootSurfaceResourcesLocked(bool locked);
41 void ForceImmediateSwapIfPossible(); 44 void ForceImmediateSwapIfPossible();
42 virtual void DisplayResized(); 45 virtual void DisplayResized();
43 virtual void SetNewRootSurface(const SurfaceId& root_surface_id); 46 virtual void SetNewRootSurface(const SurfaceId& root_surface_id);
44 virtual void SurfaceDamaged(const SurfaceId& surface_id); 47 virtual void SurfaceDamaged(const SurfaceId& surface_id,
48 const BeginFrameAck& ack,
49 bool display_damaged);
50 void SurfaceCreated(const SurfaceInfo& surface_info);
51 void SurfaceDestroyed(const SurfaceId& surface_id);
52 void SurfaceDamageExpected(const SurfaceId& surface_id,
53 const BeginFrameArgs& args);
45 54
46 virtual void DidSwapBuffers(); 55 virtual void DidSwapBuffers();
47 void DidReceiveSwapBuffersAck(); 56 void DidReceiveSwapBuffersAck();
48 57
49 void OutputSurfaceLost(); 58 void OutputSurfaceLost();
50 59
51 // BeginFrameObserverBase implementation 60 // BeginFrameObserverBase implementation
52 bool OnBeginFrameDerivedImpl(const BeginFrameArgs& args) override; 61 bool OnBeginFrameDerivedImpl(const BeginFrameArgs& args) override;
53 void OnBeginFrameSourcePausedChanged(bool paused) override; 62 void OnBeginFrameSourcePausedChanged(bool paused) override;
54 63
55 protected: 64 protected:
56 base::TimeTicks DesiredBeginFrameDeadlineTime(); 65 base::TimeTicks DesiredBeginFrameDeadlineTime();
57 virtual void ScheduleBeginFrameDeadline(); 66 virtual void ScheduleBeginFrameDeadline();
58 bool AttemptDrawAndSwap(); 67 bool AttemptDrawAndSwap();
59 void OnBeginFrameDeadline(); 68 void OnBeginFrameDeadline();
60 bool DrawAndSwap(); 69 bool DrawAndSwap();
61 void StartObservingBeginFrames(); 70 void StartObservingBeginFrames();
62 void StopObservingBeginFrames(); 71 void StopObservingBeginFrames();
63 bool ShouldDraw(); 72 bool ShouldDraw();
64 void DidFinishFrame(bool did_draw); 73 void DidFinishFrame(bool did_draw);
74 // Updates |has_pending_surfaces_| and returns whether its value changed.
75 bool UpdateHasPendingSurfaces();
65 76
66 DisplaySchedulerClient* client_; 77 DisplaySchedulerClient* client_;
67 BeginFrameSource* begin_frame_source_; 78 BeginFrameSource* begin_frame_source_;
68 base::SingleThreadTaskRunner* task_runner_; 79 base::SingleThreadTaskRunner* task_runner_;
69 80
70 BeginFrameArgs current_begin_frame_args_; 81 BeginFrameArgs current_begin_frame_args_;
71 base::Closure begin_frame_deadline_closure_; 82 base::Closure begin_frame_deadline_closure_;
72 base::CancelableClosure begin_frame_deadline_task_; 83 base::CancelableClosure begin_frame_deadline_task_;
73 base::TimeTicks begin_frame_deadline_task_time_; 84 base::TimeTicks begin_frame_deadline_task_time_;
74 85
75 base::CancelableClosure missed_begin_frame_task_; 86 base::CancelableClosure missed_begin_frame_task_;
76 bool inside_surface_damaged_; 87 bool inside_surface_damaged_;
77 88
78 bool visible_; 89 bool visible_;
79 bool output_surface_lost_; 90 bool output_surface_lost_;
80 bool root_surface_resources_locked_; 91 bool root_surface_resources_locked_;
81 92
82 bool inside_begin_frame_deadline_interval_; 93 bool inside_begin_frame_deadline_interval_;
83 bool needs_draw_; 94 bool needs_draw_;
84 bool expecting_root_surface_damage_because_of_resize_; 95 bool expecting_root_surface_damage_because_of_resize_;
85 bool all_active_child_surfaces_ready_to_draw_; 96 bool has_pending_surfaces_;
97
98 struct SurfaceBeginFrameState {
99 BeginFrameArgs last_args;
100 BeginFrameAck last_ack;
101 };
102 base::flat_map<SurfaceId, SurfaceBeginFrameState> surface_states_;
86 103
87 int next_swap_id_; 104 int next_swap_id_;
88 int pending_swaps_; 105 int pending_swaps_;
89 int max_pending_swaps_; 106 int max_pending_swaps_;
90 107
91 bool observing_begin_frame_source_; 108 bool observing_begin_frame_source_;
92 109
93 SurfaceId root_surface_id_; 110 SurfaceId root_surface_id_;
94 bool root_surface_damaged_;
95 bool expect_damage_from_root_surface_;
96
97 std::set<SurfaceId> child_surface_ids_damaged_;
98 std::set<SurfaceId> child_surface_ids_damaged_prev_;
99 std::vector<SurfaceId> child_surface_ids_to_expect_damage_from_;
100 111
101 base::WeakPtrFactory<DisplayScheduler> weak_ptr_factory_; 112 base::WeakPtrFactory<DisplayScheduler> weak_ptr_factory_;
102 113
103 private: 114 private:
104 DISALLOW_COPY_AND_ASSIGN(DisplayScheduler); 115 DISALLOW_COPY_AND_ASSIGN(DisplayScheduler);
105 }; 116 };
106 117
107 } // namespace cc 118 } // namespace cc
108 119
109 #endif // CC_SURFACES_DISPLAY_SCHEDULER_H_ 120 #endif // CC_SURFACES_DISPLAY_SCHEDULER_H_
OLDNEW
« no previous file with comments | « cc/surfaces/display.cc ('k') | cc/surfaces/display_scheduler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698