Index: cc/scoped_ptr_deque.h |
diff --git a/cc/scoped_ptr_deque.h b/cc/scoped_ptr_deque.h |
index 64f09a8a8ec11f2a6f66ea5f9df41017ef21296d..f5eddea00fc2537e9d25187e2484921755d96563 100644 |
--- a/cc/scoped_ptr_deque.h |
+++ b/cc/scoped_ptr_deque.h |
@@ -18,10 +18,24 @@ namespace cc { |
template <typename T> |
class ScopedPtrDeque { |
public: |
- typedef typename std::deque<T*>::iterator iterator; |
typedef typename std::deque<T*>::const_iterator const_iterator; |
typedef typename std::deque<T*>::reverse_iterator reverse_iterator; |
- typedef typename std::deque<T*>::const_reverse_iterator const_reverse_iterator; |
+ typedef typename std::deque<T*>::const_reverse_iterator |
+ const_reverse_iterator; |
+ |
+#if defined(OS_ANDROID) |
+ // On Android the iterator is not a class, so we can't block assignment. |
+ typedef typename std::deque<T*>::iterator iterator; |
+#else |
+ // Ban setting values on the iterator directly. New pointers must be passed |
+ // to methods on the ScopedPtrDeque class to appear in the deque. |
+ class iterator : public std::deque<T*>::iterator { |
+ public: |
+ iterator(const typename std::deque<T*>::iterator& other) |
+ : std::deque<T*>::iterator(other) {} |
+ T* const& operator*() { return std::deque<T*>::iterator::operator*(); } |
+ }; |
+#endif |
ScopedPtrDeque() {} |
@@ -31,37 +45,37 @@ class ScopedPtrDeque { |
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 { |
- return size() == 0; |
+ bool empty() const { |
+ return data_.empty(); |
} |
- scoped_ptr<T> takeFirst() { |
- scoped_ptr<T> ret(first()); |
+ scoped_ptr<T> take_front() { |
+ 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(); |
} |
@@ -70,18 +84,32 @@ class ScopedPtrDeque { |
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) { |
+ DCHECK(position <= end()); |
+ data_.insert(position, item.release()); |
+ } |
+ |
+ void swap(iterator a, iterator b) { |
+ DCHECK(a < end()); |
+ DCHECK(b < end()); |
+ if (a == end() || b == end() || a == b) |
+ return; |
+ typename std::deque<T*>::iterator writable_a = a; |
+ typename std::deque<T*>::iterator writable_b = b; |
+ std::swap(*writable_a, *writable_b); |
} |
- iterator begin() { return data_.begin(); } |
+ iterator begin() { return static_cast<iterator>(data_.begin()); } |
const_iterator begin() const { return data_.begin(); } |
- iterator end() { return data_.end(); } |
+ iterator end() { return static_cast<iterator>(data_.end()); } |
const_iterator end() const { return data_.end(); } |
reverse_iterator rbegin() { return data_.rbegin(); } |