Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1342)

Unified Diff: cc/base/list_container.cc

Issue 1226503006: cc: More consistent reasoning about display list memory usage. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: capacity unit test Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/base/list_container.cc
diff --git a/cc/base/list_container.cc b/cc/base/list_container.cc
index 1d54a6d038fdb8eebe6a272c16438192316c5cd4..d5cb4f032605656d43ad072866ce27b3cacd6172 100644
--- a/cc/base/list_container.cc
+++ b/cc/base/list_container.cc
@@ -120,10 +120,8 @@ class ListContainerBase::ListContainerCharAllocator {
size_t Capacity() const {
size_t capacity_sum = 0;
- for (ScopedPtrVector<InnerList>::const_iterator iter = storage_.begin();
- iter != storage_.end(); ++iter) {
- capacity_sum += (*iter)->capacity;
- }
+ for (const auto& inner_list : storage_)
+ capacity_sum += inner_list->capacity;
return capacity_sum;
}
@@ -425,6 +423,10 @@ size_t ListContainerBase::MaxSizeForDerivedClass() const {
return data_->element_size();
}
+size_t ListContainerBase::GetCapacityInBytes() const {
+ return data_->Capacity() * data_->element_size();
+}
+
void ListContainerBase::clear() {
data_->Clear();
}

Powered by Google App Engine
This is Rietveld 408576698