Make AboutCrash handler crash directly.

chrome/renderer/about_handler.cc

Index: chrome/renderer/about_handler.cc
diff --git a/chrome/renderer/about_handler.cc b/chrome/renderer/about_handler.cc
index de4b57aec325cee746287023d30194ab38c5bc45..ecfeeba9ab7e9d489d855fb2766967e76a105fc7 100644
--- a/chrome/renderer/about_handler.cc
+++ b/chrome/renderer/about_handler.cc
@@ -4,7 +4,6 @@
 
 #include "chrome/renderer/about_handler.h"
 
-#include "base/logging.h"
 #include "base/process_util.h"
 #include "base/threading/platform_thread.h"
 #include "chrome/common/about_handler.h"
@@ -43,7 +42,10 @@ bool AboutHandler::MaybeHandle(const GURL& url) {
 
 // static
 void AboutHandler::AboutCrash() {
-  CHECK(false);
+  // NOTE(shess): Crash directly rather than using NOTREACHED() so
+  // that the signature is easier to triage in crash reports.
+  int* zero = NULL;
+  *zero = 0;

[Scott Hess - ex-Googler, 2011/07/01 19:59:01]

In an email thread, msw said:
Would the use of volatile be to prevent the compiler from realizing that this is
undefined behavior?

Regardless, it seems reasonable to add a NOTREACHED() after this, just in case.

 }
 
 // static