List all child surfaces (including occluded) in CompositorFrame

cc/trees/layer_tree_impl.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_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <set>
 
@@ skipping 1256 matching lines @@
 }
 
 void LayerTreeImpl::AddLayerWithCopyOutputRequest(LayerImpl* layer) {
   // Only the active tree needs to know about layers with copy requests, as
   // they are aborted if not serviced during draw.
   DCHECK(IsActiveTree());
 
   // DCHECK(std::find(layers_with_copy_output_request_.begin(),
   //                 layers_with_copy_output_request_.end(),
   //                 layer) == layers_with_copy_output_request_.end());
   // TODO(danakj): Remove this once crash is found crbug.com/309777

[danakj, 2015/11/16 21:45:46]

Oh, I should do this :)

Can you add a similar DCHECK in your Add method?

   for (size_t i = 0; i < layers_with_copy_output_request_.size(); ++i) {
     CHECK(layers_with_copy_output_request_[i] != layer)
         << i << " of " << layers_with_copy_output_request_.size();
   }
   layers_with_copy_output_request_.push_back(layer);
 }
 
 void LayerTreeImpl::RemoveLayerWithCopyOutputRequest(LayerImpl* layer) {
   // Only the active tree needs to know about layers with copy requests, as
   // they are aborted if not serviced during draw.
   DCHECK(IsActiveTree());
 
   std::vector<LayerImpl*>::iterator it = std::find(
       layers_with_copy_output_request_.begin(),
       layers_with_copy_output_request_.end(),
       layer);
   DCHECK(it != layers_with_copy_output_request_.end());
   layers_with_copy_output_request_.erase(it);
 
   // TODO(danakj): Remove this once crash is found crbug.com/309777
   for (size_t i = 0; i < layers_with_copy_output_request_.size(); ++i) {
     CHECK(layers_with_copy_output_request_[i] != layer)
         << i << " of " << layers_with_copy_output_request_.size();
   }
 }
 
+void LayerTreeImpl::AddSurfaceLayer(LayerImpl* layer) {
+  surface_layers_.push_back(layer);
+}
+
+void LayerTreeImpl::RemoveSurfaceLayer(LayerImpl* layer) {
+  std::vector<LayerImpl*>::iterator it =
+      std::find(surface_layers_.begin(), surface_layers_.end(), layer);
+  DCHECK(it != surface_layers_.end());
+  surface_layers_.erase(it);
+}
+
 const std::vector<LayerImpl*>& LayerTreeImpl::LayersWithCopyOutputRequest()
     const {
   // Only the active tree needs to know about layers with copy requests, as
   // they are aborted if not serviced during draw.
   DCHECK(IsActiveTree());
 
   return layers_with_copy_output_request_;
 }
 
 template <typename LayerType>
@@ skipping 613 matching lines @@
                                                    const gfx::BoxF& box,
                                                    gfx::BoxF* bounds) const {
   *bounds = gfx::BoxF();
   return layer_tree_host_impl_->animation_host()
              ? layer_tree_host_impl_->animation_host()
                    ->TransformAnimationBoundsForBox(layer->id(), box, bounds)
              : true;
 }
 
 }  // namespace cc