cc: Report only on active tiles in tracing.

cc/trees/layer_tree_host_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_host_impl.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/basictypes.h"
@@ skipping 3201 matching lines @@
     FrameData* frame,
     base::debug::TracedValue* state) const {
   if (this->pending_tree_) {
     state->BeginDictionary("activation_state");
     ActivationStateAsValueInto(state);
     state->EndDictionary();
   }
   state->BeginDictionary("device_viewport_size");
   MathUtil::AddToTracedValue(device_viewport_size_, state);
   state->EndDictionary();
+
+  std::set<const Tile*> tiles;
+  active_tree_->AllTilesForTracingInto(&tiles);
+  if (pending_tree_)
+    pending_tree_->AllTilesForTracingInto(&tiles);
+
   if (tile_manager_) {
     state->BeginArray("tiles");
-    tile_manager_->AllTilesAsValueInto(state);
+    tile_manager_->AllTilesAsValueInto(state, tiles);

[enne (OOO), 2014/08/20 17:37:34]

Something seems redundant here.  Why do you need to get tiles from both the
trees as well as the tile manager? It seems like you'd want either or?  Am I
misunderstanding what lives in TileManager's tile map these days?

[vmpstr, 2014/08/20 17:41:11]

TileManager's tiles include recycled tiles (in fact, those are literally all the
tiles that have been created). Active/pending tree tiles are the ones that are
"in play" this frame. An alternative would be to plumb some sort of a
"IsTileOnActiveOrPendingTree" to tile manager via client which would search for
the tile in the trees, but this seems a more direct-ish approach.

[enne (OOO), 2014/08/20 18:05:42]

Should the TileManager::AllTilesAsValueInto get removed, then?

[vmpstr, 2014/08/20 18:18:58]

Hmm. Well, I don't really know. I feel like it's a good thing to log all the
tiles, even the recycle ones? I don't really have a justification for that
though. Nat, do you think these recycle tiles will ever be useful in trace
viewer? If not, I'll just remove the tile manager tile dump

     state->EndArray();
 
     state->BeginDictionary("tile_manager_basic_state");
     tile_manager_->BasicStateAsValueInto(state);
     state->EndDictionary();
   }
   state->BeginDictionary("active_tree");
   active_tree_->AsValueInto(state);
   state->EndDictionary();
   if (pending_tree_) {
@@ skipping 177 matching lines @@
 }
 
 void LayerTreeHostImpl::UnregisterPictureLayerImpl(PictureLayerImpl* layer) {
   std::vector<PictureLayerImpl*>::iterator it =
       std::find(picture_layers_.begin(), picture_layers_.end(), layer);
   DCHECK(it != picture_layers_.end());
   picture_layers_.erase(it);
 }
 
 }  // namespace cc