Fix scheduler bug in skipping main frames

cc/scheduler/scheduler.cc

 // Copyright 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/scheduler/scheduler.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/logging.h"
@@ skipping 301 matching lines @@
 void Scheduler::BeginImplFrameWithDeadline(const BeginFrameArgs& args) {
   // The storage for |args| is owned by the missed begin frame task. Therefore
   // save |args| before cancelling the task either here or in the deadline.
   BeginFrameArgs adjusted_args = args;
   // Cancel the missed begin frame task in case the BFS sends a begin frame
   // before the missed frame task runs.
   missed_begin_frame_task_.Cancel();
 
   // Discard missed begin frames if they are too late.
   if (adjusted_args.type == BeginFrameArgs::MISSED &&
       Now() > adjusted_args.deadline) {

[brianderson, 2016/10/12 20:38:17]

Would it work to store this Now() and use it in both places?

[enne (OOO), 2016/10/13 17:09:44]

Done.

[enne (OOO), 2016/10/13 17:09:44]

Done.

     begin_frame_source_->DidFinishFrame(this, 0);
     return;
   }
 
   // Run the previous deadline if any.
   if (state_machine_.begin_impl_frame_state() ==
       SchedulerStateMachine::BEGIN_IMPL_FRAME_STATE_INSIDE_BEGIN_FRAME) {
     OnBeginImplFrameDeadline();
     // We may not need begin frames any longer.
     if (!observing_begin_frame_source_) {
@@ skipping 402 matching lines @@
   // latency if both the main and impl threads can run serially before the
   // deadline.
   return can_activate_before_deadline;
 }
 
 bool Scheduler::CanBeginMainFrameAndActivateBeforeDeadline(
     const BeginFrameArgs& args,
     base::TimeDelta bmf_to_activate_estimate) const {
   // Check if the main thread computation and commit can be finished before the
   // impl thread's deadline.
-  base::TimeTicks estimated_draw_time =
-      args.frame_time + bmf_to_activate_estimate;
+  base::TimeTicks estimated_draw_time = Now() + bmf_to_activate_estimate;
 
   return estimated_draw_time < args.deadline;
 }
 
 bool Scheduler::IsBeginMainFrameSentOrStarted() const {
   return (state_machine_.begin_main_frame_state() ==
               SchedulerStateMachine::BEGIN_MAIN_FRAME_STATE_SENT ||
           state_machine_.begin_main_frame_state() ==
               SchedulerStateMachine::BEGIN_MAIN_FRAME_STATE_STARTED);
 }
 
 }  // namespace cc