cc: Move WebCompositorOutputSurface and related classes into cc/

cc/output_surface_client.h

Index: cc/output_surface_client.h
diff --git a/cc/output_surface_client.h b/cc/output_surface_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..4267eec4262b6277a74f7ba4626db78aa7f04838
--- /dev/null
+++ b/cc/output_surface_client.h
@@ -0,0 +1,20 @@
+ // Copyright 2012 The Chromium Authors. All rights reserved.

[jamesr, 2012/12/04 07:06:51]

leading space here for some reason

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CC_OUTPUT_SURFACE_CLIENT_H_
+#define CC_OUTPUT_SURFACE_CLIENT_H_
+
+namespace cc {
+
+class CompositorFrameAck;
+
+class OutputSurfaceClient {
+ public:
+  virtual void OnVSyncParametersChanged(double monotonicTimebase, double intervalInSeconds) = 0;

[jamesr, 2012/12/04 07:06:51]

the types on this should be base::TimeTicks and base::TimeDelta. they're only
double in WebKit because it doesn't have proper time types

80col

[danakj, 2012/12/04 18:15:34]

Done.

+  virtual void OnSendFrameToParentCompositorAck(const CompositorFrameAck&) {}

[jamesr, 2012/12/04 07:06:51]

do we need a default impl of this or can it be pure virtual? it's useful to have
default impls on logically pure virtual interfaces in WebKit API since it's
annoying to update impls at the same time, but since this is all chromium you
should be able to find + update all OutputSurfaceClient implementations

[danakj, 2012/12/04 18:15:34]

Sure, I'll have to add the method to LayerTreeHost now then.

+};

[jamesr, 2012/12/04 07:06:51]

can you add an empty virtual dtor (probably protected) ?

[danakj, 2012/12/04 18:15:34]

Done.

+
+}  // namespace cc
+
+#endif  // CC_OUTPUT_SURFACE_CLIENT_H_