cc: Don't upload UI resources twice after eviction

cc/trees/layer_tree_host.cc

 // Copyright 2011 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.
 
 #include "cc/trees/layer_tree_host.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 1172 matching lines @@
   ui_resource_client_map_.erase(iter);
 }
 
 void LayerTreeHost::RecreateUIResources() {
   for (UIResourceClientMap::iterator iter = ui_resource_client_map_.begin();
        iter != ui_resource_client_map_.end();
        ++iter) {
     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 to find the == in it all.

auto it = std::find_if(....);
if (it == ..end()) {
}

would be better IMO

[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);
+    }
   }
 }
 
 // Returns the size of a resource given its id.
 gfx::Size LayerTreeHost::GetUIResourceSize(UIResourceId uid) const {
   UIResourceClientMap::const_iterator iter = ui_resource_client_map_.find(uid);
   if (iter == ui_resource_client_map_.end())
     return gfx::Size();
 
   const UIResourceClientData& data = iter->second;
@@ skipping 465 matching lines @@
 
   surface_client_id_ = proto.surface_client_id();
   next_surface_sequence_ = proto.next_surface_sequence();
 }
 
 AnimationHost* LayerTreeHost::animation_host() const {
   return layer_tree_.animation_host();
 }
 
 }  // namespace cc