ThreadWatcher - give ShutdownWatchDogThread::Alarm a more

unique function body.

Changed ShutdownWatchDogThread::Alarm's to call
ShutdownCrash(), so it does not get merged by the link time 
optimization.

R=jar@chromium.org, eroman@chromium.org
BUG=232080
Review URL: https://codereview.chromium.org/14250003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@194524 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/metrics/thread_watcher.cc

@@ -51,6 +51,10 @@ void NullPointerCrash(int line_number) {
 #endif
 }
 
+NOINLINE void ShutdownCrash() {
+  NullPointerCrash(__LINE__);
+}
+
 NOINLINE void ThreadUnresponsive_UI() {
   NullPointerCrash(__LINE__);
 }
@@ -873,7 +877,7 @@ class ShutdownWatchDogThread : public base::Watchdog {
   // Alarm is called if the time expires after an Arm() without someone calling
   // Disarm(). We crash the browser if this method is called.
   virtual void Alarm() OVERRIDE {
-    CHECK(false);
+    ShutdownCrash();
   }
 
   DISALLOW_COPY_AND_ASSIGN(ShutdownWatchDogThread);