binding: Retires ExceptionState::throwIfNeeded().

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 52 matching lines @@
         IndexedSetterContext,
         IndexedDeletionContext,
         UnknownContext, // FIXME: Remove this once we've flipped over to the new API.
     };
 
     ExceptionState(v8::Isolate* isolate, ContextType contextType, const char* interfaceName, const char* propertyName)
         : m_code(0)
         , m_context(contextType)
         , m_propertyName(propertyName)
         , m_interfaceName(interfaceName)
-        , m_isolate(isolate) { }
+        , m_isolate(isolate)
+    {
+        DCHECK(m_isolate);
+    }
 
     ExceptionState(v8::Isolate* isolate, ContextType contextType, const char* interfaceName)
         : ExceptionState(isolate, contextType, interfaceName, nullptr)
     {
 #if ENABLE(ASSERT)
         switch (m_context) {
         case ConstructionContext:
         case EnumerationContext:
         case IndexedGetterContext:
         case IndexedSetterContext:
         case IndexedDeletionContext:
             break;
         default:
             NOTREACHED();
         }
 #endif // ENABLE(ASSERT)
     }
 
     ExceptionState(ContextType context, const char* propertyName, const char* interfaceName, const v8::Local<v8::Object>& creationContext, v8::Isolate* isolate) // DEPRECATED
         : ExceptionState(isolate, context, interfaceName, propertyName) { }
 
     ExceptionState(ContextType context, const char* interfaceName, const v8::Local<v8::Object>& creationContext, v8::Isolate* isolate) // DEPRECATED
         : ExceptionState(isolate, context, interfaceName) { }
 
+    ~ExceptionState()
+    {
+        if (!m_exception.isEmpty()) {
+            V8ThrowException::throwException(m_isolate, m_exception.newLocal(m_isolate));
+        }
+    }
+
     virtual void throwDOMException(const ExceptionCode&, const String& message);
+    virtual void throwRangeError(const String& message);
+    virtual void throwSecurityError(const String& sanitizedMessage, const String& unsanitizedMessage = String());
     virtual void throwTypeError(const String& message);
-    virtual void throwSecurityError(const String& sanitizedMessage, const String& unsanitizedMessage = String());
-    virtual void throwRangeError(const String& message);
 
-    bool hadException() const { return !m_exception.isEmpty() || m_code; }
+    bool hadException() const { return !m_exception.isEmpty(); }
     void clearException();
 
     ExceptionCode code() const { return m_code; }
     const String& message() const { return m_message; }
-    v8::Local<v8::Value> getException() { DCHECK(!m_exception.isEmpty()); return m_exception.newLocal(m_isolate); }
-
-    bool throwIfNeeded()
+    v8::Local<v8::Value> getException()
     {
-        if (!hadException())
-            return false;
-        throwException();
-        return true;
+        DCHECK(!m_exception.isEmpty());
+        return m_exception.newLocal(m_isolate);
     }
 
     // This method clears out the exception which |this| has.
     ScriptPromise reject(ScriptState*);
 
     // This method clears out the exception which |this| has.
     void reject(ScriptPromiseResolver*);
 
-    ContextType context() const { return m_context; }
     const char* propertyName() const { return m_propertyName; }
     const char* interfaceName() const { return m_interfaceName; }
 
     void rethrowV8Exception(v8::Local<v8::Value> value)
     {
-        setException(value);
+        setException(0, String(), value);
     }
 
-    // Might return nullptr.
     v8::Isolate* isolate() const { return m_isolate; }
 
 #if ENABLE(ASSERT)
     OnStackObjectChecker& getOnStackObjectChecker() { return m_onStackObjectChecker; }
 #endif
 
 protected:
+    void setException(ExceptionCode, const String&, v8::Local<v8::Value>);
+
+private:
+    String addExceptionContext(const String&) const;
+
     ExceptionCode m_code;
     ContextType m_context;
     String m_message;
     const char* m_propertyName;
     const char* m_interfaceName;
-
-private:
-    void setException(v8::Local<v8::Value>);
-    void throwException();
-
-    String addExceptionContext(const String&) const;
-
     ScopedPersistent<v8::Value> m_exception;
     v8::Isolate* m_isolate;
 #if ENABLE(ASSERT)
     OnStackObjectChecker m_onStackObjectChecker;
 #endif
 };
 
 // Used if exceptions can/should not be directly thrown.
 class CORE_EXPORT NonThrowableExceptionState final : public ExceptionState {
-    WTF_MAKE_NONCOPYABLE(NonThrowableExceptionState);
 public:
-    NonThrowableExceptionState(): ExceptionState(ExceptionState::UnknownContext, 0, 0, v8::Local<v8::Object>(), v8::Isolate::GetCurrent()) { }
+    NonThrowableExceptionState()
+        : ExceptionState(v8::Isolate::GetCurrent(), ExceptionState::UnknownContext, nullptr, nullptr) { }
     void throwDOMException(const ExceptionCode&, const String& message) override;
     void throwTypeError(const String& message = String()) override;
     void throwSecurityError(const String& sanitizedMessage, const String& unsanitizedMessage = String()) override;
     void throwRangeError(const String& message) override;
 };
 
 // Used if any exceptions thrown are ignorable.
 class CORE_EXPORT TrackExceptionState final : public ExceptionState {
-    WTF_MAKE_NONCOPYABLE(TrackExceptionState);
 public:
-    TrackExceptionState(): ExceptionState(ExceptionState::UnknownContext, 0, 0, v8::Local<v8::Object>(), v8::Isolate::GetCurrent()) { }
+    TrackExceptionState()
+        : ExceptionState(v8::Isolate::GetCurrent(), ExceptionState::UnknownContext, nullptr, nullptr) { }
+    ~TrackExceptionState()
+    {
+        // Prevent the base class throw an exception.
+        if (hadException()) {
+            clearException();
+        }
+    }
+
     void throwDOMException(const ExceptionCode&, const String& message) override;
     void throwTypeError(const String& message = String()) override;
     void throwSecurityError(const String& sanitizedMessage, const String& unsanitizedMessage = String()) override;
     void throwRangeError(const String& message) override;
 };
 
 } // namespace blink
 
 #endif // ExceptionState_h