cc : Store surface layer ids on LayerTreeHost and push them at commit

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 1074 matching lines @@
                                  bool* content_is_suitable_for_gpu) {
   base::AutoReset<bool> painting(&in_paint_layer_contents_, true);
   bool did_paint_content = false;
   for (const auto& layer : update_layer_list) {
     did_paint_content |= layer->Update();
     *content_is_suitable_for_gpu &= layer->IsSuitableForGpuRasterization();
   }
   return did_paint_content;
 }
 
+void LayerTreeHost::AddSurfaceLayerId(const SurfaceId& surface_id) {
+  auto pos = std::find(surface_layer_ids_.begin(), surface_layer_ids_.end(),
+                       surface_id);
+  if (pos == surface_layer_ids_.end()) {
+    surface_layer_ids_.push_back(surface_id);
+    needs_surface_ids_sync_ = true;
+  }
+}
+
+void LayerTreeHost::RemoveSurfaceLayerId(const SurfaceId& surface_id) {
+  auto pos = std::find(surface_layer_ids_.begin(), surface_layer_ids_.end(),

[enne (OOO), 2017/05/24 18:44:54]

What about flat_set over vector, so you don't have to find before insert/erase?

[jaydasika, 2017/05/24 21:39:54]

Done.

+                       surface_id);
+  if (pos != surface_layer_ids_.end()) {
+    surface_layer_ids_.erase(pos);
+    needs_surface_ids_sync_ = true;
+  }
+}
+
+const std::vector<SurfaceId>& LayerTreeHost::SurfaceLayerIds() const {
+  return surface_layer_ids_;
+}
+
 void LayerTreeHost::AddLayerShouldPushProperties(Layer* layer) {
   layers_that_should_push_properties_.insert(layer);
 }
 
 void LayerTreeHost::RemoveLayerShouldPushProperties(Layer* layer) {
   layers_that_should_push_properties_.erase(layer);
 }
 
 std::unordered_set<Layer*>& LayerTreeHost::LayersThatShouldPushProperties() {
   return layers_that_should_push_properties_;
@@ skipping 380 matching lines @@
 void LayerTreeHost::SetNeedsDisplayOnAllLayers() {
   for (auto* layer : *this)
     layer->SetNeedsDisplay();
 }
 
 void LayerTreeHost::SetHasCopyRequest(bool has_copy_request) {
   has_copy_request_ = has_copy_request;
 }
 
 }  // namespace cc