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

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

Issue 1184863004: cc: Move timing history to the Scheduler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase, add include 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 1f50b4545e9aa5f5c6acf87f36ea4f59c4182f33..9f045a2a8df50c1af5e42d209f123a846668651c 100644
--- a/components/view_manager/surfaces/surfaces_scheduler.cc
+++ b/components/view_manager/surfaces/surfaces_scheduler.cc
@@ -4,14 +4,19 @@
#include "components/view_manager/surfaces/surfaces_scheduler.h"
+#include "cc/debug/rendering_stats_instrumentation.h"
#include "cc/surfaces/display.h"
+
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 +66,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;
}
@@ -96,18 +97,6 @@ void SurfacesScheduler::ScheduledActionPrepareTiles() {
void SurfacesScheduler::ScheduledActionInvalidateOutputSurface() {
}
-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.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