Process names, and predictable thread and process ordering in about:tracing

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 54e4e4c682313bafdfc285e4dbe57d29e63f864e..15f2d6a7ccf267d7497e84138afa73ff9f7fe9ed 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -968,6 +968,9 @@ RenderViewImpl::~RenderViewImpl() {
   if (decrement_shared_popup_at_destruction_)
     shared_popup_counter_->data--;
 
+  base::debug::TraceLog::GetInstance()->RemoveProcessLabel(
+      reinterpret_cast<int>(this));

[piman, 2013/07/03 19:35:55]

nit: Is the label id just an arbitrary unique, per-process id?
Should we use the routind_id which should have this property? Feels a bit less
of a hack.

+
   // If file chooser is still waiting for answer, dispatch empty answer.
   while (!file_chooser_completions_.empty()) {
     if (file_chooser_completions_.front()->completion) {
@@ -1954,6 +1957,9 @@ void RenderViewImpl::UpdateTitle(WebFrame* frame,
   if (frame->parent())
     return;
 
+  base::debug::TraceLog::GetInstance()->UpdateProcessLabel(
+      reinterpret_cast<int>(this), UTF16ToUTF8(title));
+
   string16 shortened_title = title.substr(0, kMaxTitleChars);
   Send(new ViewHostMsg_UpdateTitle(routing_id_, page_id_, shortened_title,
                                    title_direction));