WTF: Remove NO_RETURN, NO_RETURN_DUE_TO_CRASH, and NO_RETURN_DUE_TO_ASSERT.

third_party/WebKit/Source/wtf/Compiler.h

Index: third_party/WebKit/Source/wtf/Compiler.h
diff --git a/third_party/WebKit/Source/wtf/Compiler.h b/third_party/WebKit/Source/wtf/Compiler.h
index 519c20522353036e9dfeb2b5cfbed90993ec98f0..108ebd55f671239b5d115fa70602f370723691f2 100644
--- a/third_party/WebKit/Source/wtf/Compiler.h
+++ b/third_party/WebKit/Source/wtf/Compiler.h
@@ -61,18 +61,6 @@
 
 /* ==== Compiler features ==== */
 
-/* NO_RETURN */
-
-#ifndef NO_RETURN

[Yuta Kitamura, 2016/11/21 08:55:21]

I was initially a bit hesitant to remove this, because noreturn has some
performance benefits (compilers are able to skip saving the register
values, etc.).

However, I start to think it's okay to remove this and the use sites in
Blink for now, because:
(1) C++11 has a standard way of writing this: [[noreturn]]
    http://en.cppreference.com/w/cpp/language/attributes
(2) All the use sites do not seem performance sensitive.

-#if COMPILER(GCC)
-#define NO_RETURN __attribute((__noreturn__))
-#elif COMPILER(MSVC)
-#define NO_RETURN __declspec(noreturn)
-#else
-#define NO_RETURN
-#endif
-#endif
-
 /* NEVER_INLINE */
 
 // TODO(palmer): Remove this and update callers to use NOINLINE from Chromium