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

Unified Diff: cc/trees/layer_tree_host.cc

Issue 2247573002: cc: Don't upload UI resources twice after eviction (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: cc: Don't upload UI resources again after eviction Created 4 years, 4 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/trees/layer_tree_host.cc
diff --git a/cc/trees/layer_tree_host.cc b/cc/trees/layer_tree_host.cc
index 8e0493a8c1af61f5f87ab840f4ae5be1a468df18..eabac70e304fa6812552421c8ee2a931eba98733 100644
--- a/cc/trees/layer_tree_host.cc
+++ b/cc/trees/layer_tree_host.cc
@@ -1190,9 +1190,15 @@ void LayerTreeHost::RecreateUIResources() {
UIResourceId uid = iter->first;
const UIResourceClientData& data = iter->second;
bool resource_lost = true;
- UIResourceRequest request(UIResourceRequest::UI_RESOURCE_CREATE, uid,
- data.client->GetBitmap(uid, resource_lost));
- ui_resource_request_queue_.push_back(request);
+ if (std::find_if(ui_resource_request_queue_.begin(),
danakj 2016/08/15 22:52:11 Can you split this into a few lines cuz its hard t
no sievers 2016/08/26 18:38:18 Done.
+ ui_resource_request_queue_.end(),
+ [uid](const UIResourceRequest& request) {
+ return request.GetId() == uid;
+ }) == ui_resource_request_queue_.end()) {
+ UIResourceRequest request(UIResourceRequest::UI_RESOURCE_CREATE, uid,
+ data.client->GetBitmap(uid, resource_lost));
+ ui_resource_request_queue_.push_back(request);
+ }
}
}

Powered by Google App Engine
This is Rietveld 408576698