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

Unified Diff: cc/scheduler/compositor_timing_history.h

Issue 1460923002: cc: CompositorTimingHistory throughput UMAs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@didDraw
Patch Set: Remove debug TRACE_EVENTS Created 5 years, 1 month 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | cc/scheduler/compositor_timing_history.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/scheduler/compositor_timing_history.h
diff --git a/cc/scheduler/compositor_timing_history.h b/cc/scheduler/compositor_timing_history.h
index b09923ab308c53fededda6df230a35f77ca50212..1523aeb4b3ca42c17a9670bd84207d073a981f99 100644
--- a/cc/scheduler/compositor_timing_history.h
+++ b/cc/scheduler/compositor_timing_history.h
@@ -50,6 +50,9 @@ class CC_EXPORT CompositorTimingHistory {
void SetRecordingEnabled(bool enabled);
+ void WillBeginImplFrame(bool new_active_tree_is_likely);
+ void WillFinishImplFrame(bool needs_redraw);
+ void BeginImplFrameNotExpectedSoon();
void WillBeginMainFrame(bool on_critical_path);
void BeginMainFrameStarted(base::TimeTicks main_thread_start_time);
void BeginMainFrameAborted();
@@ -60,17 +63,32 @@ class CC_EXPORT CompositorTimingHistory {
void WillActivate();
void DidActivate();
void WillDraw();
- void DidDraw();
+ void DidDraw(bool used_new_active_tree);
void DidSwapBuffers();
void DidSwapBuffersComplete();
void DidSwapBuffersReset();
protected:
+ void DidBeginMainFrame();
+
+ void SetBeginMainFrameNeededContinuously(bool active);
+ void SetBeginMainFrameCommittingContinuously(bool active);
+ void SetCompositorDrawingContinuously(bool active);
+
static scoped_ptr<UMAReporter> CreateUMAReporter(UMACategory category);
virtual base::TimeTicks Now() const;
bool enabled_;
+ // Used to calculate frame rates of Main and Impl threads.
+ bool did_send_begin_main_frame_;
+ bool begin_main_frame_needed_continuously_;
+ bool begin_main_frame_committing_continuously_;
+ bool compositor_drawing_continuously_;
+ base::TimeTicks begin_main_frame_end_time_prev_;
+ base::TimeTicks new_active_tree_draw_end_time_prev_;
+ base::TimeTicks draw_end_time_prev_;
+
RollingTimeDeltaHistory begin_main_frame_sent_to_commit_duration_history_;
RollingTimeDeltaHistory begin_main_frame_queue_duration_critical_history_;
RollingTimeDeltaHistory begin_main_frame_queue_duration_not_critical_history_;
@@ -83,10 +101,10 @@ class CC_EXPORT CompositorTimingHistory {
bool begin_main_frame_on_critical_path_;
base::TimeTicks begin_main_frame_sent_time_;
base::TimeTicks begin_main_frame_start_time_;
- base::TimeTicks commit_time_;
- base::TimeTicks start_prepare_tiles_time_;
- base::TimeTicks start_activate_time_;
- base::TimeTicks start_draw_time_;
+ base::TimeTicks begin_main_frame_end_time_;
+ base::TimeTicks prepare_tiles_start_time_;
+ base::TimeTicks activate_start_time_;
+ base::TimeTicks draw_start_time_;
base::TimeTicks swap_start_time_;
scoped_ptr<UMAReporter> uma_reporter_;
« no previous file with comments | « no previous file | cc/scheduler/compositor_timing_history.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698