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

content/renderer/renderer_main.cc

 // Copyright (c) 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 "base/base_switches.h"
 #include "base/command_line.h"
 #include "base/debug/debugger.h"
 #include "base/debug/stack_trace.h"
 #include "base/debug/trace_event.h"
 #include "base/hi_res_timer_manager.h"
@@ skipping 89 matching lines @@
   virtual void DidProcessTask(const base::PendingTask& pending_task) OVERRIDE {
     HISTOGRAM_MEMORY_KB("Memory.RendererUsed", webkit_glue::MemoryUsageKB());
   }
  private:
   DISALLOW_COPY_AND_ASSIGN(MemoryObserver);
 };
 
 // mainline routine for running as the Renderer process
 int RendererMain(const MainFunctionParams& parameters) {
   TRACE_EVENT_BEGIN_ETW("RendererMain", 0, "");
+  base::debug::TraceLog::GetInstance()->SetProcessSortIndex(1);
+  base::debug::TraceLog::GetInstance()->SetThreadSortIndex(
+      static_cast<int>(base::PlatformThread::CurrentId()), -1);

[dsinclair, 2013/06/20 15:01:47]

If we're always call SetThreadSortIndex from the thread, we could move the
base::PlatformTrhead::CurrentId() into the method itself, keeps the signature
the same as the SetProcessSortIndex method which is nice.

 
   const CommandLine& parsed_command_line = parameters.command_line;
 
 #if defined(OS_MACOSX)
   base::mac::ScopedNSAutoreleasePool* pool = parameters.autorelease_pool;
 #endif  // OS_MACOSX
 
 #if defined(OS_CHROMEOS)
   // As Zygote process starts up earlier than browser process gets its own
   // locale (at login time for Chrome OS), we have to set the ICU default
@@ skipping 116 matching lines @@
       base::MessageLoop::current()->Run();
       TRACE_EVENT_END_ETW("RendererMain.START_MSG_LOOP", 0, 0);
     }
   }
   platform.PlatformUninitialize();
   TRACE_EVENT_END_ETW("RendererMain", 0, "");
   return 0;
 }
 
 }  // namespace content