Extra logging for NinePatch layers in layerTreeAsText()

cc/nine_patch_layer_impl.cc

 // Copyright 2012 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 "config.h"
 
 #include "nine_patch_layer_impl.h"
 
+#include "base/stringprintf.h"
 #include "cc/quad_sink.h"
 #include "cc/texture_draw_quad.h"
 #include "ui/gfx/rect_f.h"
 
 namespace cc {
 
 NinePatchLayerImpl::NinePatchLayerImpl(int id)
     : LayerImpl(id)
     , m_resourceId(0)
 {
 }
 
 NinePatchLayerImpl::~NinePatchLayerImpl()
 {
 }
 
 ResourceProvider::ResourceId NinePatchLayerImpl::contentsResourceId() const
 {
     return 0;
 }
 
-void NinePatchLayerImpl::dumpLayerProperties(std::string* str, int indent) const
-{
-    LayerImpl::dumpLayerProperties(str, indent);
-}
-
-
 void NinePatchLayerImpl::willDraw(ResourceProvider* resourceProvider)
 {
 }
 
 static gfx::RectF normalizedRect(float x, float y, float width, float height, float totalWidth, float totalHeight)
 {
     return gfx::RectF(x / totalWidth, y / totalHeight, width / totalWidth, height / totalHeight);
 }
 
 void NinePatchLayerImpl::setLayout(const gfx::Size& imageBounds, const gfx::Rect& aperture)
@@ skipping 61 matching lines @@
 void NinePatchLayerImpl::didLoseContext()
 {
     m_resourceId = 0;
 }
 
 const char* NinePatchLayerImpl::layerTypeAsString() const
 {
     return "NinePatchLayer";
 }
 
+void NinePatchLayerImpl::dumpLayerProperties(std::string* str, int indent) const
+{
+    str->append(indentString(indent));
+    base::StringAppendF(str, "m_imageAperture: %s\n", m_imageAperture.ToString().c_str());

[danakj, 2012/11/07 21:31:35]

nit: drop the m_ it's cleaner.

maybe consider putting the call to the parent first, so the layerId stays at the
top.

+    LayerImpl::dumpLayerProperties(str, indent);
 }
+
+}

[danakj, 2012/11/07 19:00:01]

}  // namespace cc