Renamed GpuWatchdogThread::OnExit to make it clear it is crashing deliberatel...

chrome/gpu/gpu_watchdog_thread.cc

Index: chrome/gpu/gpu_watchdog_thread.cc
===================================================================
--- chrome/gpu/gpu_watchdog_thread.cc	(revision 75655)
+++ chrome/gpu/gpu_watchdog_thread.cc	(working copy)
@@ -118,7 +118,7 @@
   if (!armed_)
     return;
 
-  // Revoke any pending OnExit.
+  // Revoke any pending hang termination.
   method_factory_->RevokeAll();
   armed_ = false;
 
@@ -188,12 +188,15 @@
   // not respond in time.
   message_loop()->PostDelayedTask(
       FROM_HERE,
-      method_factory_->NewRunnableMethod(&GpuWatchdogThread::OnExit),
+      method_factory_->NewRunnableMethod(
+          &GpuWatchdogThread::
+              DeliberatelyCrashingToRecoverFromHangNotNecessarilyABug),
       timeout_);
 }
 
 // Use the --disable-gpu-watchdog command line switch to disable this.
-void GpuWatchdogThread::OnExit() {
+void GpuWatchdogThread::
+    DeliberatelyCrashingToRecoverFromHangNotNecessarilyABug() {

[Avi (use Gerrit), 2011/02/24 19:15:43]

Awesome name!

[Scott Hess - ex-Googler, 2011/02/24 19:27:41]

+1.  Hope it doesn't break the crash server!

 #if defined(OS_WIN)
   // Defer termination until a certain amount of CPU time has elapsed on the
   // watched thread.
@@ -201,7 +204,9 @@
   if (time_since_arm < timeout_) {
     message_loop()->PostDelayedTask(
         FROM_HERE,
-        method_factory_->NewRunnableMethod(&GpuWatchdogThread::OnExit),
+        method_factory_->NewRunnableMethod(
+            &GpuWatchdogThread::
+                DeliberatelyCrashingToRecoverFromHangNotNecessarilyABug),
         timeout_ - time_since_arm);
     return;
   }