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

Unified Diff: components/view_manager/surfaces/surfaces_scheduler.cc

Issue 1192663005: cc: Measure compositor timing with finer granularity (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@modeTimingHistory3
Patch Set: fixes Created 5 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: components/view_manager/surfaces/surfaces_scheduler.cc
diff --git a/components/view_manager/surfaces/surfaces_scheduler.cc b/components/view_manager/surfaces/surfaces_scheduler.cc
index 2757642d10dee43afeb9c4d92c3c7f83eccbfd87..7cf6cebc861068ee715bba9afc5ddd08aa946ef4 100644
--- a/components/view_manager/surfaces/surfaces_scheduler.cc
+++ b/components/view_manager/surfaces/surfaces_scheduler.cc
@@ -8,10 +8,13 @@
namespace surfaces {
-SurfacesScheduler::SurfacesScheduler() {
+SurfacesScheduler::SurfacesScheduler()
+ : rendering_stats_instrumentation_(
+ RenderingStatsInstrumentation::Create()) {
cc::SchedulerSettings settings;
scheduler_ = cc::Scheduler::Create(
- this, settings, 0, base::MessageLoop::current()->task_runner(), nullptr);
+ this, settings, 0, base::MessageLoop::current()->task_runner(), nullptr,
+ rendering_stats_instrumentation_.get());
scheduler_->SetCanStart();
scheduler_->SetVisible(true);
scheduler_->SetCanDraw(true);
@@ -61,13 +64,9 @@ void SurfacesScheduler::ScheduledActionSendBeginMainFrame() {
}
cc::DrawResult SurfacesScheduler::ScheduledActionDrawAndSwapIfPossible() {
- base::TimeTicks start = base::TimeTicks::Now();
for (const auto& it : displays_) {
it->DrawAndSwap();
}
- base::TimeDelta duration = base::TimeTicks::Now() - start;
-
- draw_estimate_ = (duration + draw_estimate_) / 2;
return cc::DRAW_SUCCESS;
}
@@ -98,18 +97,6 @@ void SurfacesScheduler::ScheduledActionInvalidateOutputSurface() {
void SurfacesScheduler::DidAnticipatedDrawTimeChange(base::TimeTicks time) {
}
-base::TimeDelta SurfacesScheduler::DrawDurationEstimate() {
- return draw_estimate_;
-}
-
-base::TimeDelta SurfacesScheduler::BeginMainFrameToCommitDurationEstimate() {
- return base::TimeDelta();
-}
-
-base::TimeDelta SurfacesScheduler::CommitToActivateDurationEstimate() {
- return base::TimeDelta();
-}
-
void SurfacesScheduler::SendBeginFramesToChildren(
const cc::BeginFrameArgs& args) {
}
« cc/scheduler/scheduler_unittest.cc ('K') | « components/view_manager/surfaces/surfaces_scheduler.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698