Remove NoExceptionStateAssertionChecker

third_party/WebKit/Source/bindings/core/v8/ExceptionState.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 138 matching lines @@
   // The exception is empty when it was thrown through
   // DummyExceptionStateForTesting.
   ScopedPersistent<v8::Value> m_exception;
   v8::Isolate* m_isolate;
 };
 
 // NonThrowableExceptionState never allow call sites to throw an exception.
 // Should be used if an exception must not be thrown.
 class CORE_EXPORT NonThrowableExceptionState final : public ExceptionState {
  public:
-  NonThrowableExceptionState()
-      : ExceptionState(nullptr,
-                       ExceptionState::UnknownContext,
-                       nullptr,
-                       nullptr) {}
+  NonThrowableExceptionState();
+  NonThrowableExceptionState(const char*, int);
 
   void throwDOMException(ExceptionCode, const String& message) override;
   void throwTypeError(const String& message) override;
   void throwSecurityError(const String& sanitizedMessage,
                           const String& unsanitizedMessage) override;
   void throwRangeError(const String& message) override;
   void rethrowV8Exception(v8::Local<v8::Value>) override;
+  ExceptionState& returnThis() { return *this; }
+
+ private:
+  const char* m_file;
+  const int m_line;
 };
 
+// Syntax sugar for NonThrowableExceptionState.
+// This can be used as a default value of an ExceptionState parameter like this:
+//
+//   Node* removeChild(Node* child, ExceptionState& = ASSERT_NO_EXCEPTION)
+#if ENABLE(ASSERT)
+#define ASSERT_NO_EXCEPTION \
+  (::blink::NonThrowableExceptionState(__FILE__, __LINE__).returnThis())
+#else
+#define ASSERT_NO_EXCEPTION \
+  (::blink::DummyExceptionStateForTesting().returnThis())
+#endif
+
 // DummyExceptionStateForTesting ignores all thrown exceptions. You should not
 // use DummyExceptionStateForTesting in production code, where you need to
 // handle all exceptions properly. If you really need to ignore exceptions in
 // production code for some special reason, explicitly call clearException().
 class CORE_EXPORT DummyExceptionStateForTesting final : public ExceptionState {
  public:
   DummyExceptionStateForTesting()
       : ExceptionState(nullptr,
                        ExceptionState::UnknownContext,
                        nullptr,
                        nullptr) {}
   ~DummyExceptionStateForTesting() {
     // Prevent the base class throw an exception.
     if (hadException()) {
       clearException();
     }
   }
   void throwDOMException(ExceptionCode, const String& message) override;
   void throwTypeError(const String& message) override;
   void throwSecurityError(const String& sanitizedMessage,
                           const String& unsanitizedMessage) override;
   void throwRangeError(const String& message) override;
   void rethrowV8Exception(v8::Local<v8::Value>) override;
   ExceptionState& returnThis() { return *this; }
 };
 
+// Syntax sugar for DummyExceptionStateForTesting.
+// This can be used as a default value of an ExceptionState parameter like this:
+//
+//   Node* removeChild(Node* child, ExceptionState& = IGNORE_EXCEPTION)
+#define IGNORE_EXCEPTION (::blink::DummyExceptionStateForTesting().returnThis())

[Yuki, 2017/01/06 06:34:07]

IIRC, we don't want IGNORE_EXCEPTION in our codebase, right?
If so, please mark this as DEPRECATED or OBSOLETE.

+
 }  // namespace blink
 
 #endif  // ExceptionState_h