Add layer name into frame viewer

cc/layers/layer.cc

 // Copyright 2010 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/layers/layer.h"
 
 #include <algorithm>
 
+#include "base/debug/trace_event.h"
 #include "base/location.h"
 #include "base/metrics/histogram.h"
 #include "base/single_thread_task_runner.h"
 #include "cc/animation/animation.h"
 #include "cc/animation/animation_events.h"
 #include "cc/animation/layer_animation_controller.h"
 #include "cc/layers/layer_impl.h"
 #include "cc/output/copy_output_request.h"
 #include "cc/output/copy_output_result.h"
 #include "cc/trees/layer_tree_host.h"
@@ skipping 649 matching lines @@
                                                base::Passed(&result)));
 }
 
 void Layer::PushPropertiesTo(LayerImpl* layer) {
   layer->SetAnchorPoint(anchor_point_);
   layer->SetAnchorPointZ(anchor_point_z_);
   layer->SetBackgroundColor(background_color_);
   layer->SetBounds(paint_properties_.bounds);
   layer->SetContentBounds(content_bounds());
   layer->SetContentsScale(contents_scale_x(), contents_scale_y());
-  layer->SetDebugName(debug_name_);
+
+  bool pulling_debug_name;
+  TRACE_EVENT_CATEGORY_GROUP_ENABLED(TRACE_DISABLED_BY_DEFAULT("cc.debug"),

[enne (OOO), 2013/07/09 20:15:32]

Please remove the disabled by default here.

Also, as it turns out, this is calling into Blink code from the compositor
thread while the main thread is blocked.  I think this would be safer to update
the name on the layer during the Update call, which does happen on the main
thread.

[qiankun, 2013/07/10 04:53:07]

Done.
Pull name then call SetDebugName() in ContentLayer::Update()? If it's correct, I
can do this.

[enne (OOO), 2013/07/10 16:48:42]

Yeah, that's exactly it.  Rather than pulling the name in PushPropertiesTo, do
it in Update.

[qiankun, 2013/07/11 10:04:43]

Done.

+                                     &pulling_debug_name);
+  if (pulling_debug_name)
+    layer->SetDebugName(DebugName());
+  else
+    layer->SetDebugName(debug_name_);

[enne (OOO), 2013/07/09 20:15:32]

What about clearing the name if tracing is off?

[qiankun, 2013/07/10 04:53:07]

Ok.

+
   layer->SetCompositingReasons(compositing_reasons_);
   layer->SetDoubleSided(double_sided_);
   layer->SetDrawCheckerboardForMissingTiles(
       draw_checkerboard_for_missing_tiles_);
   layer->SetForceRenderSurface(force_render_surface_);
   layer->SetDrawsContent(DrawsContent());
   layer->SetFilters(filters());
   layer->SetFilter(filter());
   layer->SetBackgroundFilters(background_filters());
   layer->SetMasksToBounds(masks_to_bounds_);
@@ skipping 198 matching lines @@
 
 RenderingStatsInstrumentation* Layer::rendering_stats_instrumentation() const {
   return layer_tree_host_->rendering_stats_instrumentation();
 }
 
 bool Layer::SupportsLCDText() const {
   return false;
 }
 
 }  // namespace cc