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

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: add cc:: 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
« no previous file with comments | « components/view_manager/surfaces/surfaces_scheduler.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 f2a413fd3dcf25818363c9f497807384da1cb20b..bee565ac9442c1a1adbae246a179bafaf2418e30 100644
--- a/components/view_manager/surfaces/surfaces_scheduler.cc
+++ b/components/view_manager/surfaces/surfaces_scheduler.cc
@@ -4,15 +4,21 @@
#include "components/view_manager/surfaces/surfaces_scheduler.h"
+#include "cc/debug/rendering_stats_instrumentation.h"
+#include "cc/scheduler/compositor_timing_history.h"
#include "cc/surfaces/display.h"
namespace surfaces {
-SurfacesScheduler::SurfacesScheduler() {
+SurfacesScheduler::SurfacesScheduler()
+ : rendering_stats_instrumentation_(
+ cc::RenderingStatsInstrumentation::Create()) {
cc::SchedulerSettings settings;
+ scoped_ptr<cc::CompositorTimingHistory> compositor_timing_history(
+ new cc::CompositorTimingHistory(rendering_stats_instrumentation_.get()));
scheduler_ = cc::Scheduler::Create(
this, settings, 0, base::MessageLoop::current()->task_runner().get(),
- nullptr);
+ nullptr, compositor_timing_history.Pass());
scheduler_->SetCanStart();
scheduler_->SetVisible(true);
scheduler_->SetCanDraw(true);
@@ -62,13 +68,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;
}
@@ -97,18 +99,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) {
}
« no previous file with comments | « 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