cc: Use BeginFrameArgs

cc/output/begin_frame_args.cc

Index: cc/output/begin_frame_args.cc
diff --git a/cc/output/begin_frame_args.cc b/cc/output/begin_frame_args.cc
index cefb22bdc1382aa20f6217a5f077fad39d5daa98..6edf41e1e38ea0b633cfd4381ec33f441f892733 100644
--- a/cc/output/begin_frame_args.cc
+++ b/cc/output/begin_frame_args.cc
@@ -49,9 +49,7 @@ BeginFrameArgs BeginFrameArgs::CreateExpiredForTesting() {
 }
 
 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() {
@@ -62,5 +60,12 @@ 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