cc: Implement unified BeginFrame on aura

cc/output/begin_frame_args.h

 // 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.
 
 #ifndef CC_OUTPUT_BEGIN_FRAME_ARGS_H_
 #define CC_OUTPUT_BEGIN_FRAME_ARGS_H_
 
 #include "base/location.h"
 #include "base/memory/ref_counted.h"
 #include "base/time/time.h"
@@ skipping 65 matching lines @@
   // send a BeginFrame that had been skipped. This only has an effect if the
   // deadline has passed, since the deadline is also used to trigger BeginFrame
   // retroactively.
   static base::TimeDelta DefaultRetroactiveBeginFramePeriod();
 
   bool IsValid() const { return interval >= base::TimeDelta(); }
 
   scoped_refptr<base::debug::ConvertableToTraceFormat> AsValue() const;
   void AsValueInto(base::debug::TracedValue* dict) const;
 
+  bool operator==(const BeginFrameArgs& other) const {

[mithro-old, 2014/12/16 10:50:57]

Why have these been defined? We have explicitly kept these operators only
available in testing so people use them in production.

[simonhong, 2014/12/16 15:13:30]

In ui::Compositor, I need to compare two BeginFrameArgs.
So, I moved this from testing code to here.

+    return (frame_time == other.frame_time && deadline == other.deadline &&
+            interval == other.interval && type == other.type);
+  }
+  bool operator!=(const BeginFrameArgs& other) const {
+    return (frame_time != other.frame_time || deadline != other.deadline ||
+            interval != other.interval || type != other.type);
+  }
+
   base::TimeTicks frame_time;
   base::TimeTicks deadline;
   base::TimeDelta interval;
   BeginFrameArgsType type;
 
  private:
   BeginFrameArgs(base::TimeTicks frame_time,
                  base::TimeTicks deadline,
                  base::TimeDelta interval,
                  BeginFrameArgsType type);
 };
 
 }  // namespace cc
 
 #endif  // CC_OUTPUT_BEGIN_FRAME_ARGS_H_