Histogram times surrounding render crashes

content/public/browser/render_process_host.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_RENDER_PROCESS_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_RENDER_PROCESS_HOST_H_
 
 #include "base/basictypes.h"
 #include "base/id_map.h"
 #include "base/process.h"
@@ skipping 22 matching lines @@
 // communication channel. There will generally be one RenderProcessHost per
 // renderer process.
 class CONTENT_EXPORT RenderProcessHost : public IPC::Message::Sender,
                                          public IPC::Channel::Listener {
  public:
   typedef IDMap<RenderProcessHost>::iterator iterator;
   typedef IDMap<RenderWidgetHost>::const_iterator RenderWidgetHostsIterator;
 
   // Details for RENDERER_PROCESS_CLOSED notifications.
   struct RendererClosedDetails {
-    RendererClosedDetails(base::ProcessHandle handle,
-                          base::TerminationStatus status,
-                          int exit_code,
-                          bool was_alive) {
+    RendererClosedDetails(base::ProcessHandle handle) {
       this->handle = handle;
-      this->status = status;
-      this->exit_code = exit_code;
-      this->was_alive = was_alive;
+      // default values should be updated by caller.
+      status = base::TERMINATION_STATUS_NORMAL_TERMINATION;
+      exit_code = 0;
+      was_alive = false;
+
+#if defined(OS_WIN)
+      have_process_times = false;
+      FILETIME win_creation_time;
+      FILETIME win_exit_time;
+      FILETIME win_kernel_time;
+      FILETIME win_user_time;
+      if (!GetProcessTimes(handle, &win_creation_time, &win_exit_time,
+                               &win_kernel_time, &win_user_time)) {

[rvargas (doing something else), 2012/03/22 22:16:30]

nit: indent under handle

[jar (doing other things), 2012/03/22 22:42:58]

Done.

+        DWORD error = GetLastError();
+        DLOG(ERROR) << "Error getting process data" << error;
+        return;
+      }
+      user_duration = base::Time::FromFileTime(win_user_time) -
+          base::Time::Time();
+      kernel_duration = base::Time::FromFileTime(win_kernel_time) -
+          base::Time::Time();
+      run_duration = base::Time::FromFileTime(win_exit_time) -
+          base::Time::FromFileTime(win_creation_time);
+      have_process_times = true;
+#endif   // OS_WIN
     }
+
+#if defined(OS_WIN)
+    base::TimeDelta kernel_duration;
+    base::TimeDelta user_duration;
+    base::TimeDelta run_duration;
+    bool have_process_times;
+#endif   // OS_WIN
+
     base::ProcessHandle handle;
     base::TerminationStatus status;
     int exit_code;
     bool was_alive;
   };
 
   virtual ~RenderProcessHost() {}
 
   // Initialize the new renderer process, returning true on success. This must
   // be called once before the object can be used, but can be called after
@@ skipping 174 matching lines @@
 
   // Returns the current max number of renderer processes used by the content
   // module.
   static size_t GetMaxRendererProcessCount();
 };
 
 }  // namespace content.
 
 #endif  // CONTENT_PUBLIC_BROWSER_RENDER_PROCESS_HOST_H_