cc: Add frame data to LTHI tracing

cc/quads/draw_quad.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 "cc/quads/draw_quad.h"
 
 #include "base/logging.h"
+#include "base/values.h"
+#include "cc/base/math_util.h"
+#include "cc/debug/traced_value.h"
 #include "cc/quads/checkerboard_draw_quad.h"
 #include "cc/quads/debug_border_draw_quad.h"
 #include "cc/quads/io_surface_draw_quad.h"
 #include "cc/quads/picture_draw_quad.h"
 #include "cc/quads/render_pass_draw_quad.h"
 #include "cc/quads/solid_color_draw_quad.h"
 #include "cc/quads/stream_video_draw_quad.h"
 #include "cc/quads/texture_draw_quad.h"
 #include "cc/quads/tile_draw_quad.h"
 #include "cc/quads/yuv_video_draw_quad.h"
+#include "ui/gfx/quad_f.h"
 
 namespace {
 template<typename T> T* TypedCopy(const cc::DrawQuad* other) {
   return new T(*T::MaterialCast(other));
 }
 }  // namespace
 
 namespace cc {
 
 DrawQuad::DrawQuad()
@@ skipping 55 matching lines @@
       break;
     case RENDER_PASS:  // RenderPass quads have their own copy() method.
     case INVALID:
       LOG(FATAL) << "Invalid DrawQuad material " << material;
       break;
   }
   copy_quad->shared_quad_state = copied_shared_quad_state;
   return copy_quad.Pass();
 }
 
+scoped_ptr<base::Value> DrawQuad::AsValue() const {
+  scoped_ptr<base::DictionaryValue> value(new base::DictionaryValue());
+  value->SetInteger("material", material);
+  value->Set("shared_state",
+             TracedValue::CreateIDRef(shared_quad_state).release());
+
+  value->Set("rect_in_content_space", MathUtil::AsValue(rect).release());

[vmpstr, 2013/07/31 16:23:25]

nit: A little note for the future (or maybe for here): if you name the variables
ending with _rect, then the tracing importer will do some work for us and
convert it from an array to a rect (and same for _quad). So, content_space_rect
would probably be a bit better, but trying to name every rect ending with _rect
might make naming awkward.

[piman, 2013/08/06 04:19:32]

It did it that way, I think the names work.

+  bool clipped;
+  gfx::QuadF rect_as_target_space_quad = MathUtil::MapQuad(
+      shared_quad_state->content_to_target_transform,
+      gfx::QuadF(rect),
+      &clipped);
+  value->Set("rect_as_target_space_quad",
+             MathUtil::AsValue(rect_as_target_space_quad).release());
+
+  value->Set("opaque_rect_in_content_space",
+             MathUtil::AsValue(opaque_rect).release());
+  gfx::QuadF opaque_rect_as_target_space_quad = MathUtil::MapQuad(
+      shared_quad_state->content_to_target_transform,
+      gfx::QuadF(opaque_rect),
+      &clipped);
+  value->Set("opaque_rect_as_target_space_quad",
+             MathUtil::AsValue(opaque_rect_as_target_space_quad).release());
+
+  value->Set("visible_rect_in_content_space",
+             MathUtil::AsValue(visible_rect).release());
+  gfx::QuadF visible_rect_as_target_space_quad = MathUtil::MapQuad(
+      shared_quad_state->content_to_target_transform,
+      gfx::QuadF(visible_rect),
+      &clipped);
+  value->Set("visible_rect_as_target_space_quad",
+             MathUtil::AsValue(visible_rect_as_target_space_quad).release());
+
+  value->SetBoolean("needs_blending", needs_blending);
+  value->SetBoolean("should_draw_with_blending", ShouldDrawWithBlending());
+  ExtendValue(value.get());
+  return value.PassAs<base::Value>();
+}
+
 }  // namespace cc