Unified BeginFrame scheduling

cc/output/begin_frame_args.h

Index: cc/output/begin_frame_args.h
diff --git a/cc/output/begin_frame_args.h b/cc/output/begin_frame_args.h
index b23a81e183a2180a90684cf92736d59ba3ba5804..40d33229d180289d741bf55b18b845850992c773 100644
--- a/cc/output/begin_frame_args.h
+++ b/cc/output/begin_frame_args.h
@@ -50,6 +50,18 @@ struct CC_EXPORT BeginFrameArgs {
   scoped_refptr<base::debug::ConvertableToTraceFormat> AsValue() const;
   void AsValueInto(base::debug::TracedValue* dict) const;
 
+  bool operator==(const BeginFrameArgs& other) const {

[mithro-old, 2014/09/09 03:22:37]

We have an operator== used just for testing in cc/test/begin_frame_args_test.h

Could you please remove that copy of the function?

It would probably be worth doing a minor refactoring the tests in
cc/output/begin_frame_args_unittest.cc to pull the equality tests out into their
own function.

+    return (frame_time == other.frame_time &

[brianderson, 2014/09/04 18:45:45]

& -> &&

+            deadline == other.deadline &&
+            interval == other.interval);
+  }
+
+  bool operator!=(const BeginFrameArgs& other) const {
+    return (frame_time != other.frame_time ||
+            deadline != other.deadline ||
+            interval != other.interval);
+  }
+
   base::TimeTicks frame_time;
   base::TimeTicks deadline;
   base::TimeDelta interval;