Store gpu crashes in the systemprofileproto

content/browser/gpu/gpu_process_host_ui_shim.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_BROWSER_GPU_GPU_PROCESS_HOST_UI_SHIM_H_
 #define CONTENT_BROWSER_GPU_GPU_PROCESS_HOST_UI_SHIM_H_
 
 // This class lives on the UI thread and supports classes like the
 // BackingStoreProxy, which must live on the UI thread. The IO thread
 // portion of this class, the GpuProcessHost, is responsible for
 // shuttling messages between the browser and GPU processes.
 
 #include <string>
 
 #include "base/callback_forward.h"
 #include "base/compiler_specific.h"
 #include "base/memory/linked_ptr.h"
 #include "base/memory/ref_counted.h"
+#include "base/process/kill.h"
 #include "base/threading/non_thread_safe.h"
 #include "content/common/content_export.h"
 #include "content/common/message_router.h"
 #include "content/public/common/gpu_memory_stats.h"
 #include "gpu/config/gpu_info.h"
 #include "ipc/ipc_listener.h"
 #include "ipc/ipc_sender.h"
 
 struct GpuHostMsg_AcceleratedSurfaceBuffersSwapped_Params;
 struct GpuHostMsg_AcceleratedSurfacePostSubBuffer_Params;
@@ skipping 17 matching lines @@
 class GpuProcessHostUIShim : public IPC::Listener,
                              public IPC::Sender,
                              public base::NonThreadSafe {
  public:
   // Create a GpuProcessHostUIShim with the given ID.  The object can be found
   // using FromID with the same id.
   static GpuProcessHostUIShim* Create(int host_id);
 
   // Destroy the GpuProcessHostUIShim with the given host ID. This can only
   // be called on the UI thread. Only the GpuProcessHost should destroy the
-  // UI shim.
-  static void Destroy(int host_id, const std::string& message);
+  // UI shim. We also pass in the status of the GPU termination.
+  static void Destroy(int host_id, base::TerminationStatus status,

[Alexei Svitkine (slow), 2013/12/13 22:37:22]

Nit: 1 param per line and align the params.

[rkaplow, 2013/12/13 23:54:03]

done, aligned on another method as well.

+      const std::string& message);
 
   // Destroy all remaining GpuProcessHostUIShims.
   CONTENT_EXPORT static void DestroyAll();
 
   CONTENT_EXPORT static GpuProcessHostUIShim* FromID(int host_id);
 
   // Get a GpuProcessHostUIShim instance; it doesn't matter which one.
   // Return NULL if none has been created.
   CONTENT_EXPORT static GpuProcessHostUIShim* GetOneInstance();
 
@@ skipping 40 matching lines @@
                                base::TimeDelta interval);
   void OnFrameDrawn(const ui::LatencyInfo& latency_info);
 
   // The serial number of the GpuProcessHost / GpuProcessHostUIShim pair.
   int host_id_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_GPU_GPU_PROCESS_HOST_UI_SHIM_H_