cc: Use BeginFrameArgs

cc/output/begin_frame_args.cc

 // Copyright 2013 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/output/begin_frame_args.h"
 
 namespace cc {
 
 BeginFrameArgs::BeginFrameArgs()
   : frame_time(base::TimeTicks()),
@@ skipping 31 matching lines @@
 }
 
 BeginFrameArgs BeginFrameArgs::CreateExpiredForTesting() {
   base::TimeTicks now = base::TimeTicks::Now();
   return BeginFrameArgs(now,
                         now - DefaultInterval(),
                         DefaultInterval());
 }
 
 base::TimeDelta BeginFrameArgs::DefaultDeadlineAdjustment() {
-  // Using a large deadline adjustment will effectively revert BeginFrame
-  // scheduling to the hard vsync scheduling we used to have.
-  return base::TimeDelta::FromSeconds(-1);
+  return base::TimeDelta::FromMicroseconds(-16666 / 3);

[danakj, 2013/08/19 21:54:41]

can you comment explaining this constant?

 }
 
 base::TimeDelta BeginFrameArgs::DefaultInterval() {
   return base::TimeDelta::FromMicroseconds(16666);
 }
 
 base::TimeDelta BeginFrameArgs::DefaultRetroactiveBeginFramePeriod() {
   return base::TimeDelta::FromMicroseconds(4444);
 }
 
+void BeginFrameArgs::AdjustDeadline(base::TimeDelta delta) {
+#if defined(OS_ANDROID)

[danakj, 2013/08/19 21:54:41]

ifdefs make unittests hard to write. i guess you have no unit tests that call
this method in any way then? is that missing test coverage?

it'd be nicer if this decision was made somewhere that can see
LayerTreeSettings, and given to this class in the constructor.

+  deadline += delta;
+#else
+  deadline = base::TimeTicks();
+#endif
+}
 
 }  // namespace cc