Add trace events to measure GPU process startup timing and time to first present.

content/common/gpu/gpu_channel.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.
 
 #if defined(OS_WIN)
 #include <windows.h>
 #endif
 
 #include "content/common/gpu/gpu_channel.h"
 
@@ skipping 161 matching lines @@
       FROM_HERE,
       base::Bind(&GpuChannel::HandleMessage, weak_factory_.GetWeakPtr()));
   handle_messages_scheduled_ = true;
 }
 
 void GpuChannel::CreateViewCommandBuffer(
     const gfx::GLSurfaceHandle& window,
     int32 surface_id,
     const GPUCreateCommandBufferConfig& init_params,
     int32* route_id) {
+  TRACE_EVENT1("gpu",
+               "GpuChannel::CreateViewCommandBuffer",
+               "surface_id",
+               surface_id);
+
   *route_id = MSG_ROUTING_NONE;
   content::GetContentClient()->SetActiveURL(init_params.active_url);
 
 #if defined(ENABLE_GPU)
   WillCreateCommandBuffer(init_params.gpu_preference);
 
   GpuCommandBufferStub* share_group = stubs_.Lookup(init_params.share_group_id);
 
   *route_id = GenerateRouteID();
   scoped_ptr<GpuCommandBufferStub> stub(new GpuCommandBufferStub(
@@ skipping 137 matching lines @@
                    weak_factory_.GetWeakPtr(),
                    stub->route_id()),
         base::TimeDelta::FromMilliseconds(delay));
   }
 }
 
 void GpuChannel::OnCreateOffscreenCommandBuffer(
     const gfx::Size& size,
     const GPUCreateCommandBufferConfig& init_params,
     IPC::Message* reply_message) {
+  TRACE_EVENT0("gpu", "GpuChannel::OnCreateOffscreenCommandBuffer");
+
   int32 route_id = MSG_ROUTING_NONE;
 
   content::GetContentClient()->SetActiveURL(init_params.active_url);
 #if defined(ENABLE_GPU)
   WillCreateCommandBuffer(init_params.gpu_preference);
 
   GpuCommandBufferStub* share_group = stubs_.Lookup(init_params.share_group_id);
 
   route_id = GenerateRouteID();
 
@@ skipping 17 matching lines @@
 #endif
 
   GpuChannelMsg_CreateOffscreenCommandBuffer::WriteReplyParams(
       reply_message,
       route_id);
   Send(reply_message);
 }
 
 void GpuChannel::OnDestroyCommandBuffer(int32 route_id,
                                         IPC::Message* reply_message) {
-#if defined(ENABLE_GPU)
   TRACE_EVENT1("gpu", "GpuChannel::OnDestroyCommandBuffer",
                "route_id", route_id);
+
+#if defined(ENABLE_GPU)

[piman, 2012/05/17 22:30:03]

nit: Just remove #ifdef altogether, since we don't support enable_gpu=0 any
more?

   if (router_.ResolveRoute(route_id)) {
     GpuCommandBufferStub* stub = stubs_.Lookup(route_id);
     bool need_reschedule = (stub && !stub->IsScheduled());
     gfx::GpuPreference gpu_preference =
         stub ? stub->gpu_preference() : gfx::PreferIntegratedGpu;
     router_.RemoveRoute(route_id);
     stubs_.Remove(route_id);
     // In case the renderer is currently blocked waiting for a sync reply from
     // the stub, we need to make sure to reschedule the GpuChannel here.
     if (need_reschedule)
@@ skipping 37 matching lines @@
 void GpuChannel::WillCreateCommandBuffer(gfx::GpuPreference gpu_preference) {
   if (gpu_preference == gfx::PreferDiscreteGpu)
     ++num_contexts_preferring_discrete_gpu_;
 }
 
 void GpuChannel::DidDestroyCommandBuffer(gfx::GpuPreference gpu_preference) {
   if (gpu_preference == gfx::PreferDiscreteGpu)
     --num_contexts_preferring_discrete_gpu_;
   DCHECK_GE(num_contexts_preferring_discrete_gpu_, 0);
 }