Stop using the TRACE_EVENT_*_ETW macros.

content/renderer/render_thread_impl.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 "content/renderer/render_thread_impl.h"
 
 #include <algorithm>
 #include <limits>
 #include <map>
 #include <vector>
@@ skipping 536 matching lines @@
     : ChildThreadImpl(Options::Builder()
                           .UseMojoChannel(ShouldUseMojoChannel())
                           .Build()),
       renderer_scheduler_(scheduler.Pass()),
       main_message_loop_(main_message_loop.Pass()),
       raster_worker_pool_(new RasterWorkerPool()) {
   Init();
 }
 
 void RenderThreadImpl::Init() {
-  TRACE_EVENT_BEGIN_ETW("RenderThreadImpl::Init", 0, "");
+  TRACE_EVENT0("startup", "RenderThreadImpl::Init");
 
   base::trace_event::TraceLog::GetInstance()->SetThreadSortIndex(
       base::PlatformThread::CurrentId(),
       kTraceEventRendererMainThreadSortIndex);
 
 #if defined(OS_MACOSX) || defined(OS_ANDROID)
   // On Mac and Android, the select popups are rendered by the browser.
   blink::WebView::setUseExternalPopupMenus(true);
 #endif
 
@@ skipping 186 matching lines @@
   // TODO(boliu): In single process, browser main loop should set up the
   // discardable memory manager, and should skip this if kSingleProcess.
   // See crbug.com/503724.
   base::DiscardableMemoryAllocator::SetInstance(
       ChildThreadImpl::discardable_shared_memory_manager());
 
   service_registry()->AddService<RenderFrameSetup>(
       base::Bind(CreateRenderFrameSetup));
   service_registry()->AddService<EmbeddedWorkerSetup>(
       base::Bind(CreateEmbeddedWorkerSetup));
-
-  TRACE_EVENT_END_ETW("RenderThreadImpl::Init", 0, "");

[brucedawson, 2015/10/07 19:04:23]

Why is this call being removed completely?

[fdoray, 2015/10/08 15:38:26]

TRACE_EVENT0 emits a synchronous event with a start and end time. The end time
is taken at the end of the scope that contains the TRACE_EVENT0 macro. We don't
need and end event here, because we will get the end time of "RenderThreadImpl"
from the event emitted by TRACE_EVENT0.

At other places in this CL, I didn't use TRACE_EVENT0 because the tracing
infrastructure doesn't expect synchronous events around the main loop of a
thread. Here, it's ok to use TRACE_EVENT0 because we are not around a message
loop.

 }
 
 RenderThreadImpl::~RenderThreadImpl() {
 }
 
 void RenderThreadImpl::Shutdown() {
   FOR_EACH_OBSERVER(
       RenderProcessObserver, observers_, OnRenderProcessShutdown());
 
   ChildThreadImpl::Shutdown();
@@ skipping 1215 matching lines @@
 }
 
 void RenderThreadImpl::PendingRenderFrameConnect::OnConnectionError() {
   size_t erased =
       RenderThreadImpl::current()->pending_render_frame_connects_.erase(
           routing_id_);
   DCHECK_EQ(1u, erased);
 }
 
 }  // namespace content