cc: Heuristic for Renderer latency recovery

cc/test/begin_frame_args_test.cc

 // Copyright 2014 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/test/begin_frame_args_test.h"
 
 #include "base/time/time.h"
 #include "cc/output/begin_frame_args.h"
 #include "ui/gfx/frame_time.h"
 
 namespace cc {
 
 BeginFrameArgs CreateBeginFrameArgsForTesting(
     BeginFrameArgs::CreationLocation location) {
   return CreateBeginFrameArgsForTesting(location, gfx::FrameTime::Now());
 }
 
 BeginFrameArgs CreateBeginFrameArgsForTesting(
     BeginFrameArgs::CreationLocation location,
     base::TimeTicks frame_time) {
   return BeginFrameArgs::Create(
       location, frame_time,
-      frame_time + (BeginFrameArgs::DefaultInterval() / 2),
+      frame_time + BeginFrameArgs::DefaultInterval() -
+          BeginFrameArgs::DefaultEstimatedParentDrawTime(),

[mithro-old, 2015/05/27 04:40:15]

nit: I wonder if this should be in a separate CL?

[brianderson, 2015/05/27 23:22:32]

Will split it out.

       BeginFrameArgs::DefaultInterval(), BeginFrameArgs::NORMAL);
 }
 
 BeginFrameArgs CreateBeginFrameArgsForTesting(
     BeginFrameArgs::CreationLocation location,
     int64 frame_time,
     int64 deadline,
     int64 interval) {
   return BeginFrameArgs::Create(
       location, base::TimeTicks::FromInternalValue(frame_time),
@@ skipping 19 matching lines @@
   return BeginFrameArgs::Create(
       location, now, now - BeginFrameArgs::DefaultInterval(),
       BeginFrameArgs::DefaultInterval(), BeginFrameArgs::NORMAL);
 }
 
 BeginFrameArgs CreateBeginFrameArgsForTesting(
     BeginFrameArgs::CreationLocation location,
     scoped_refptr<TestNowSource> now_src) {
   base::TimeTicks now = now_src->Now();
   return BeginFrameArgs::Create(
-      location, now, now + (BeginFrameArgs::DefaultInterval() / 2),
+      location, now, now + BeginFrameArgs::DefaultInterval() -
+                         BeginFrameArgs::DefaultEstimatedParentDrawTime(),

[mithro-old, 2015/05/27 04:40:15]

nit: Same as above....

       BeginFrameArgs::DefaultInterval(), BeginFrameArgs::NORMAL);
 }
 
 BeginFrameArgs CreateExpiredBeginFrameArgsForTesting(
     BeginFrameArgs::CreationLocation location,
     scoped_refptr<TestNowSource> now_src) {
   base::TimeTicks now = now_src->Now();
   return BeginFrameArgs::Create(
       location, now, now - BeginFrameArgs::DefaultInterval(),
       BeginFrameArgs::DefaultInterval(), BeginFrameArgs::NORMAL);
@@ skipping 10 matching lines @@
 }
 
 void PrintTo(const BeginFrameArgs& args, ::std::ostream* os) {
   *os << "BeginFrameArgs(" << BeginFrameArgs::TypeToString(args.type) << ", "
       << args.frame_time.ToInternalValue() << ", "
       << args.deadline.ToInternalValue() << ", "
       << args.interval.InMicroseconds() << "us)";
 }
 
 }  // namespace cc