Add helpers for CGL types.

base/mac/scoped_cftyperef.h

 // Copyright (c) 2012 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 BASE_MAC_SCOPED_CFTYPEREF_H_
 #define BASE_MAC_SCOPED_CFTYPEREF_H_
 
 #include <CoreFoundation/CoreFoundation.h>
 
-#include "base/basictypes.h"
-#include "base/compiler_specific.h"
-#include "base/memory/scoped_policy.h"
+#include "base/mac/scoped_typeref.h"
 
 namespace base {
 
 // ScopedCFTypeRef<> is patterned after scoped_ptr<>, but maintains ownership
 // of a CoreFoundation object: any object that can be represented as a
 // CFTypeRef.  Style deviations here are solely for compatibility with
 // scoped_ptr<>'s interface, with which everyone is already familiar.
 //
 // By default, ScopedCFTypeRef<> takes ownership of an object (in the
 // constructor or in reset()) by taking over the caller's existing ownership
 // claim.  The caller must own the object it gives to ScopedCFTypeRef<>, and
 // relinquishes an ownership claim to that object.  ScopedCFTypeRef<> does not
 // call CFRetain(). This behavior is parameterized by the |OwnershipPolicy|
 // enum. If the value |RETAIN| is passed (in the constructor or in reset()),
 // then ScopedCFTypeRef<> will call CFRetain() on the object, and the initial
 // ownership is not changed.
 
 template<typename CFT>

[Mark Mentovai, 2014/02/07 19:53:40]

No need to templatize this at all, since you’re specifying it explicitly when
you instantiate ScopedTypeRef below.

You can put this struct into a sub-namespace like “internal” to make it obvious
that it’s an implementation detail of this file. (And I’m not an auto-complete
user, but it might also make auto-completers less likely to suggest
ScopedCFTypeRefTraits as an option to people who really just want
ScopedCFTypeRef.)

[ccameron, 2014/02/07 20:58:04]

Done.

-class ScopedCFTypeRef {
+struct ScopedCFTypeRefTraits {
+  static void Retain(CFT object) {
+    CFRetain(object);
+  }
+  static void Release(CFT object) {
+    CFRelease(object);
+  }
+};
+
+template<typename CFT>
+class ScopedCFTypeRef
+    : public ScopedTypeRef<CFT, ScopedCFTypeRefTraits<CFT> > {

[Mark Mentovai, 2014/02/07 19:53:40]

I think we only ever build with C++11 language support on Mac now, so if you
wanted to close your template with >>, that should be fine. Line 47 also.

[Avi (use Gerrit), 2014/02/07 19:59:23]

Does the compiler we use for iOS support C++11? This header file, via
ScopedCFTypeRef, is definitely used by them.

[ccameron, 2014/02/07 20:58:04]

Changed (a different instance) to >> -- if the iOS trybots are unhappy I'll
change it back.

  public:
   typedef CFT element_type;
 
   explicit ScopedCFTypeRef(
       CFT object = NULL,
       base::scoped_policy::OwnershipPolicy policy = base::scoped_policy::ASSUME)
-      : object_(object) {
-    if (object_ && policy == base::scoped_policy::RETAIN)
-      CFRetain(object_);
-  }
+      : ScopedTypeRef<CFT, ScopedCFTypeRefTraits<CFT> >(object, policy) {}
 
   ScopedCFTypeRef(const ScopedCFTypeRef<CFT>& that)
-      : object_(that.object_) {
-    if (object_)
-      CFRetain(object_);
-  }
+      : ScopedTypeRef<CFT, ScopedCFTypeRefTraits<CFT> >(that) {}
 
-  ~ScopedCFTypeRef() {
-    if (object_)
-      CFRelease(object_);
-  }
-
-  ScopedCFTypeRef& operator=(const ScopedCFTypeRef<CFT>& that) {
-    reset(that.get(), base::scoped_policy::RETAIN);
-    return *this;
-  }
-
-  void reset(CFT object = NULL,
-             base::scoped_policy::OwnershipPolicy policy =
-                base::scoped_policy::ASSUME) {
-    if (object && policy == base::scoped_policy::RETAIN)
-      CFRetain(object);
-    if (object_)
-      CFRelease(object_);
-    object_ = object;
-  }
-
-  bool operator==(CFT that) const {
-    return object_ == that;
-  }
-
-  bool operator!=(CFT that) const {
-    return object_ != that;
-  }
-
-  operator CFT() const {
-    return object_;
-  }
-
-  CFT get() const {
-    return object_;
-  }
-
-  void swap(ScopedCFTypeRef& that) {
-    CFT temp = that.object_;
-    that.object_ = object_;
-    object_ = temp;
-  }
-
-  // ScopedCFTypeRef<>::release() is like scoped_ptr<>::release.  It is NOT
-  // a wrapper for CFRelease().  To force a ScopedCFTypeRef<> object to call
-  // CFRelease(), use ScopedCFTypeRef<>::reset().
-  CFT release() WARN_UNUSED_RESULT {
-    CFT temp = object_;
-    object_ = NULL;
-    return temp;
-  }
-
- private:
-  CFT object_;
+  virtual ~ScopedCFTypeRef() {}

[Mark Mentovai, 2014/02/07 19:53:40]

Why is this virtual?

[ccameron, 2014/02/07 20:58:04]

Removed (was here for style, not correctness).

 };
 
 }  // namespace base
 
 #endif  // BASE_MAC_SCOPED_CFTYPEREF_H_