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

Unified Diff: cc/resources/resource_update_queue.cc

Issue 12962007: cc: Chromify resource_update_queue (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 9 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/resources/resource_update_queue.cc
diff --git a/cc/resources/resource_update_queue.cc b/cc/resources/resource_update_queue.cc
index e94f2187b6dcb3ba4b7924d782a6759ed8ce4c04..52c376f39565a862284fc36588464c74897df0fa 100644
--- a/cc/resources/resource_update_queue.cc
+++ b/cc/resources/resource_update_queue.cc
@@ -8,71 +8,60 @@
namespace cc {
-ResourceUpdateQueue::ResourceUpdateQueue()
-{
-}
+ResourceUpdateQueue::ResourceUpdateQueue() {}
-ResourceUpdateQueue::~ResourceUpdateQueue()
-{
-}
+ResourceUpdateQueue::~ResourceUpdateQueue() {}
-void ResourceUpdateQueue::appendFullUpload(const ResourceUpdate& upload)
-{
- m_fullEntries.push_back(upload);
+void ResourceUpdateQueue::AppendFullUpload(const ResourceUpdate& upload) {
+ full_entries_.push_back(upload);
}
-void ResourceUpdateQueue::appendPartialUpload(const ResourceUpdate& upload)
-{
- m_partialEntries.push_back(upload);
+void ResourceUpdateQueue::AppendPartialUpload(const ResourceUpdate& upload) {
+ partial_entries_.push_back(upload);
}
-void ResourceUpdateQueue::appendCopy(TextureCopier::Parameters copy)
-{
- m_copyEntries.push_back(copy);
+void ResourceUpdateQueue::AppendCopy(TextureCopier::Parameters copy) {
+ copy_entries_.push_back(copy);
}
-void ResourceUpdateQueue::clearUploadsToEvictedResources()
-{
- clearUploadsToEvictedResources(m_fullEntries);
- clearUploadsToEvictedResources(m_partialEntries);
+void ResourceUpdateQueue::ClearUploadsToEvictedResources() {
+ ClearUploadsToEvictedResources(&full_entries_);
+ ClearUploadsToEvictedResources(&partial_entries_);
}
-void ResourceUpdateQueue::clearUploadsToEvictedResources(std::deque<ResourceUpdate>& entryQueue)
-{
- std::deque<ResourceUpdate> temp;
- entryQueue.swap(temp);
- while (temp.size()) {
- ResourceUpdate upload = temp.front();
- temp.pop_front();
- if (!upload.texture->BackingResourceWasEvicted())
- entryQueue.push_back(upload);
- }
+void ResourceUpdateQueue::ClearUploadsToEvictedResources(
+ std::deque<ResourceUpdate>* entry_queue) {
+ std::deque<ResourceUpdate> temp;
+ entry_queue->swap(temp);
+ while (temp.size()) {
+ ResourceUpdate upload = temp.front();
+ temp.pop_front();
+ if (!upload.texture->BackingResourceWasEvicted())
+ entry_queue->push_back(upload);
+ }
}
-ResourceUpdate ResourceUpdateQueue::takeFirstFullUpload()
-{
- ResourceUpdate first = m_fullEntries.front();
- m_fullEntries.pop_front();
- return first;
+ResourceUpdate ResourceUpdateQueue::TakeFirstFullUpload() {
+ ResourceUpdate first = full_entries_.front();
+ full_entries_.pop_front();
+ return first;
}
-ResourceUpdate ResourceUpdateQueue::takeFirstPartialUpload()
-{
- ResourceUpdate first = m_partialEntries.front();
- m_partialEntries.pop_front();
- return first;
+ResourceUpdate ResourceUpdateQueue::TakeFirstPartialUpload() {
+ ResourceUpdate first = partial_entries_.front();
+ partial_entries_.pop_front();
+ return first;
}
-TextureCopier::Parameters ResourceUpdateQueue::takeFirstCopy()
-{
- TextureCopier::Parameters first = m_copyEntries.front();
- m_copyEntries.pop_front();
- return first;
+TextureCopier::Parameters ResourceUpdateQueue::TakeFirstCopy() {
+ TextureCopier::Parameters first = copy_entries_.front();
+ copy_entries_.pop_front();
+ return first;
}
-bool ResourceUpdateQueue::hasMoreUpdates() const
-{
- return m_fullEntries.size() || m_partialEntries.size() || m_copyEntries.size();
+bool ResourceUpdateQueue::HasMoreUpdates() const {
+ return full_entries_.size() || partial_entries_.size() ||
+ copy_entries_.size();
jamesr 2013/03/20 19:59:30 .empty() would be slightly better, I think
}
} // namespace cc

Powered by Google App Engine
This is Rietveld 408576698