Report texture upload time in renderingStats.

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 382 matching lines @@
     IPC_MESSAGE_HANDLER_DELAY_REPLY(GpuChannelMsg_CreateOffscreenCommandBuffer,
                                     OnCreateOffscreenCommandBuffer)
     IPC_MESSAGE_HANDLER_DELAY_REPLY(GpuChannelMsg_DestroyCommandBuffer,
                                     OnDestroyCommandBuffer)
 #if defined(OS_ANDROID)
     IPC_MESSAGE_HANDLER(GpuChannelMsg_RegisterStreamTextureProxy,
                         OnRegisterStreamTextureProxy)
     IPC_MESSAGE_HANDLER(GpuChannelMsg_EstablishStreamTexture,
                         OnEstablishStreamTexture)
 #endif
+    IPC_MESSAGE_HANDLER_DELAY_REPLY(GpuChannelMsg_CollectRenderingStats,
+                                    OnCollectRenderingStats)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   DCHECK(handled) << msg.type();
   return handled;
 }
 
 void GpuChannel::HandleMessage() {
   handle_messages_scheduled_ = false;
 
   if (!deferred_messages_.empty()) {
@@ skipping 117 matching lines @@
 }
 
 void GpuChannel::OnEstablishStreamTexture(
     int32 stream_id, content::SurfaceTexturePeer::SurfaceTextureTarget type,
     int32 primary_id, int32 secondary_id) {
   stream_texture_manager_->EstablishStreamTexture(
       stream_id, type, primary_id, secondary_id);
 }
 #endif
 
+void GpuChannel::OnCollectRenderingStats(
+    int32 surface_id, IPC::Message* reply_message) {
+  content::GpuRenderingStats stats;
+
+#if defined(ENABLE_GPU)

[apatrick_chromium, 2012/08/28 20:07:06]

you don't need to check ENABLE_GPU is defined any more. we're in the process of
getting rid of it. it's always defined.

+  for (StubMap::Iterator<GpuCommandBufferStub> it(&stubs_);
+       !it.IsAtEnd(); it.Advance()) {
+    int textureUploadCount =

[apatrick_chromium, 2012/08/28 20:07:06]

Google style is texture_upload_count. ditto all the other camel case variables.

+        it.GetCurrentValue()->decoder()->GetTextureUploadCount();
+    double totalTextureUploadTimeInSeconds =
+        it.GetCurrentValue()->decoder()->GetTotalTextureUploadTimeInSeconds();
+
+    stats.globalTextureUploadCount += textureUploadCount;
+    stats.globalTotalTextureUploadTimeInSeconds +=
+        totalTextureUploadTimeInSeconds;
+    if (it.GetCurrentValue()->surface_id() == surface_id) {
+      stats.textureUploadCount += textureUploadCount;
+      stats.totalTextureUploadTimeInSeconds +=
+          totalTextureUploadTimeInSeconds;
+    }
+  }
+#endif  // defined (ENABLE_GPU)
+
+  GpuChannelMsg_CollectRenderingStats::WriteReplyParams(
+      reply_message,
+      stats);
+  Send(reply_message);
+}