Trace FrameTreeNode Snapshots

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 4aaa5921a54aa6fb38a1095cd27e1381b37af9d1..46a9b3b9912f549784a1a81e4f70006640d114e4 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -1041,6 +1041,13 @@ RenderFrameImpl::RenderFrameImpl(const CreateParams& params)
 #endif
 
   manifest_manager_ = new ManifestManager(this);
+
+  TRACE_EVENT_OBJECT_CREATED_WITH_ID(

[Sami, 2016/03/02 00:29:01]

Once my part lands we don't need the changes in this file, right?

[benjhayden, 2016/03/02 18:53:03]

Right :-)
The changes in this file are just so that I can test the idRefs from
FrameTreeNode to RenderFrame, and to compare to your RenderFrame snapshots.
Both of those are done so I'll undo these changes.

+      "navigation", "RenderFrame",
+      TRACE_ID_WITH_SCOPE("RenderFrame", routing_id_));
+  TRACE_EVENT_OBJECT_SNAPSHOT_WITH_ID(
+      "navigation", "RenderFrame",
+      TRACE_ID_WITH_SCOPE("RenderFrame", routing_id_), "");
 }
 
 RenderFrameImpl::~RenderFrameImpl() {
@@ -1076,6 +1083,10 @@ RenderFrameImpl::~RenderFrameImpl() {
   render_view_->UnregisterRenderFrame(this);
   g_routing_id_frame_map.Get().erase(routing_id_);
   RenderThread::Get()->RemoveRoute(routing_id_);
+
+  TRACE_EVENT_OBJECT_DELETED_WITH_ID(
+      "navigation", "RenderFrame",
+      TRACE_ID_WITH_SCOPE("RenderFrame", routing_id_));
 }
 
 void RenderFrameImpl::BindToWebFrame(blink::WebLocalFrame* web_frame) {