cc: Make the ScopedPtrVector and ScopedPtrDeque containers act like STL vector and deque.

cc/scoped_ptr_deque.h

 // Copyright 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 CC_SCOPED_PTR_DEQUE_H_
 #define CC_SCOPED_PTR_DEQUE_H_
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 13 matching lines @@
   typedef typename std::deque<T*>::const_reverse_iterator const_reverse_iterator;
 
   ScopedPtrDeque() {}
 
   ~ScopedPtrDeque() { clear(); }
 
   size_t size() const {
     return data_.size();
   }
 
-  T* Peek(size_t index) const {
+  T* at(size_t index) const {
     DCHECK(index < size());
     return data_[index];
   }
 
   T* operator[](size_t index) const {
-    return Peek(index);
+    return at(index);
   }
 
-  T* first() const {
-    DCHECK(!isEmpty());
-    return Peek(0);
+  T* front() const {
+    DCHECK(!empty());
+    return at(0);
   }
 
-  T* last() const {
-    DCHECK(!isEmpty());
-    return Peek(size() - 1);
+  T* back() const {
+    DCHECK(!empty());
+    return at(size() - 1);
   }
 
-  bool isEmpty() const {
+  bool empty() const {
     return size() == 0;
   }
 
-  scoped_ptr<T> takeFirst() {
-    scoped_ptr<T> ret(first());
+  scoped_ptr<T> take_front() {

[enne (OOO), 2012/11/21 04:12:57]

I know take is convenient, but it's also inconsistent with non-scoped_ptr
containers.  What about changing take_front calls to front()/pop_front()?

[danakj, 2012/11/21 04:23:21]

Oh, take_front is removing the element so it could make sense to rename it
here..

+    scoped_ptr<T> ret(front());
     data_.pop_front();
     return ret.Pass();
   }
 
-  scoped_ptr<T> takeLast() {
-    scoped_ptr<T> ret(last());
+  scoped_ptr<T> take_back() {
+    scoped_ptr<T> ret(back());
     data_.pop_back();
     return ret.Pass();
   }
 
   void clear() {
     STLDeleteElements(&data_);
   }
 
-  void append(scoped_ptr<T> item) {
+  void push_front(scoped_ptr<T> item) {
+    data_.push_front(item.release());
+  }
+
+  void push_back(scoped_ptr<T> item) {
     data_.push_back(item.release());
   }
 
-  void insert(size_t index, scoped_ptr<T> item) {
-    DCHECK(index < size());
-    data_.insert(data_.begin() + index, item.release());
+  void insert(iterator position, scoped_ptr<T> item) {
+    data_.insert(position, item.release());
   }
 
   iterator begin() { return data_.begin(); }
   const_iterator begin() const { return data_.begin(); }
   iterator end() { return data_.end(); }
   const_iterator end() const { return data_.end(); }
 
   reverse_iterator rbegin() { return data_.rbegin(); }
   const_reverse_iterator rbegin() const { return data_.rbegin(); }
   reverse_iterator rend() { return data_.rend(); }
   const_reverse_iterator rend() const { return data_.rend(); }
 
  private:
   std::deque<T*> data_;
 
   DISALLOW_COPY_AND_ASSIGN(ScopedPtrDeque);
 };
 
 }  // namespace cc
 
 #endif  // CC_SCOPED_PTR_DEQUE_H_