Reimplement PooledI420Buffer, without an extra indirection.

webrtc/base/refcount.h

 /*
  *  Copyright 2011 The WebRTC Project Authors. All rights reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 89 matching lines @@
   }
 
   virtual int Release() const {
     int count = AtomicOps::Decrement(&ref_count_);
     if (!count) {
       delete this;
     }
     return count;
   }
 
+  // This call returns the reference count, and performs the memory
+  // barrier needed for the owning thread to act on the object, in
+  // case the value of the reference count implies exclusive access to
+  // the object.
+  // TODO(nisse): Currently used only by
+  // I420BufferPool::PooledI420Buffer. Consider deleting once
+  // IsMutable is deleted, see
+  // https://bugs.chromium.org/p/webrtc/issues/detail?id=5921
+  virtual int GetRefCount() const {
+    return AtomicOps::AcquireLoad(&ref_count_);
+  }
   // Return whether the reference count is one. If the reference count is used
   // in the conventional way, a reference count of 1 implies that the current
   // thread owns the reference and no other thread shares it. This call
   // performs the test for a reference count of one, and performs the memory
   // barrier needed for the owning thread to act on the object, knowing that it
   // has exclusive access to the object.
-  virtual bool HasOneRef() const {
-    return AtomicOps::AcquireLoad(&ref_count_) == 1;
-  }
+  virtual bool HasOneRef() const { return GetRefCount() == 1; }
 
  protected:
   virtual ~RefCountedObject() {
   }
 
   mutable volatile int ref_count_;
 };
 
 }  // namespace rtc
 
 #endif  // WEBRTC_BASE_REFCOUNT_H_