Index: cc/scheduler/scheduler.cc |
diff --git a/cc/scheduler/scheduler.cc b/cc/scheduler/scheduler.cc |
index 15e1fe0b3661b74517af8bae2f042ba33550b73b..082fbee328e398cd32bff66d0185101259ff9c8e 100644 |
--- a/cc/scheduler/scheduler.cc |
+++ b/cc/scheduler/scheduler.cc |
@@ -278,7 +278,6 @@ void Scheduler::SetupNextBeginFrameIfNeeded() { |
frame_source_->SetNeedsBeginFrames(true); |
devtools_instrumentation::NeedsBeginFrameChanged(layer_tree_host_id_, |
true); |
- UpdateCompositorTimingHistoryRecordingEnabled(); |
} else if (state_machine_.begin_impl_frame_state() == |
SchedulerStateMachine::BEGIN_IMPL_FRAME_STATE_IDLE) { |
// Call SetNeedsBeginFrames(false) in between frames only. |
@@ -286,7 +285,6 @@ void Scheduler::SetupNextBeginFrameIfNeeded() { |
client_->SendBeginMainFrameNotExpectedSoon(); |
devtools_instrumentation::NeedsBeginFrameChanged(layer_tree_host_id_, |
false); |
- UpdateCompositorTimingHistoryRecordingEnabled(); |
} |
} |
@@ -741,6 +739,8 @@ void Scheduler::AsValueInto(base::trace_event::TracedValue* state) const { |
state->BeginDictionary("scheduler_state"); |
state->SetDouble("estimated_parent_draw_time_ms", |
estimated_parent_draw_time_.InMillisecondsF()); |
+ state->SetBoolean("last_set_needs_begin_frame_", |
+ frame_source_->NeedsBeginFrames()); |
state->SetInteger("begin_retro_frame_args", |
static_cast<int>(begin_retro_frame_args_.size())); |
state->SetBoolean("begin_retro_frame_task", |
@@ -766,8 +766,7 @@ void Scheduler::AsValueInto(base::trace_event::TracedValue* state) const { |
void Scheduler::UpdateCompositorTimingHistoryRecordingEnabled() { |
compositor_timing_history_->SetRecordingEnabled( |
- state_machine_.HasInitializedOutputSurface() && |
- state_machine_.visible() && frame_source_->NeedsBeginFrames()); |
+ state_machine_.HasInitializedOutputSurface() && state_machine_.visible()); |
} |
bool Scheduler::ShouldRecoverMainLatency(const BeginFrameArgs& args) const { |