Introducing SurfaceReferenceFactory

cc/surfaces/surface_reference_base.h

Index: cc/surfaces/surface_reference_base.h
diff --git a/cc/surfaces/surface_reference_base.h b/cc/surfaces/surface_reference_base.h
new file mode 100644
index 0000000000000000000000000000000000000000..d92651bd41eb591e5e0fa5f24d63ed234c900e94
--- /dev/null
+++ b/cc/surfaces/surface_reference_base.h
@@ -0,0 +1,46 @@
+// Copyright 2016 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_SURFACES_SURFACE_REFERENCE_BASE_H_
+#define CC_SURFACES_SURFACE_REFERENCE_BASE_H_
+
+#include <memory>
+
+#include "base/memory/ptr_util.h"
+#include "cc/surfaces/surface_reference_factory.h"
+#include "cc/surfaces/surface_sequence.h"
+
+namespace cc {
+
+class CompositorFrameMetadata;
+class SurfaceReferenceFactory;
+
+// The base class for the references returned by SurfaceReferenceFactory and
+// its subclasses.
+// The objects of this class hold on to their corresponding surface reference
+// until destroy is called or until the object goes out of scope.
+// To keep things as lightweight as possible, the base class only keeps a
+// pointer to the factory and it's up to the subclasses to decide what other
+// information they need to keep.
+class SurfaceReferenceBase {
+ public:
+  explicit SurfaceReferenceBase(
+      scoped_refptr<const SurfaceReferenceFactory> factory);
+  virtual ~SurfaceReferenceBase();
+
+  void Destroy();

[Fady Samuel, 2016/12/13 22:55:05]

Can you make this private?

[Saman Sami, 2016/12/14 17:50:01]

Moved the implementation to the destructor

+  void Destroy(CompositorFrameMetadata* metadata);

[Fady Samuel, 2016/12/13 22:55:05]

// TODO(fsamuel, samans, kylechar): Remove this.

[Saman Sami, 2016/12/14 17:50:01]

Done.

+  bool IsDestroyed() const;

[Fady Samuel, 2016/12/13 22:55:05]

Remove this if it's not used.

[Saman Sami, 2016/12/14 17:50:01]

Done.

+
+  const SurfaceReferenceFactory* factory() const { return factory_.get(); }

[Fady Samuel, 2016/12/13 22:55:05]

Do we need this?

[Saman Sami, 2016/12/14 17:50:02]

Nope. Removed.

+
+ private:
+  scoped_refptr<const SurfaceReferenceFactory> factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(SurfaceReferenceBase);
+};
+
+}  // namespace cc
+
+#endif  // CC_SURFACES_SURFACE_REFERENCE_BASE_H_