Stop calling [de]ref in ScriptPromiseResolverWithContext destructor.

Source/bindings/v8/ScriptPromiseResolverWithContext.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "bindings/v8/ScriptPromiseResolverWithContext.h"
 
 #include "bindings/v8/V8RecursionScope.h"
 
 namespace WebCore {
 
 ScriptPromiseResolverWithContext::ScriptPromiseResolverWithContext(ScriptState* scriptState)
     : ActiveDOMObject(scriptState->executionContext())
     , m_state(Pending)
     , m_scriptState(scriptState)
     , m_mode(Default)
     , m_timer(this, &ScriptPromiseResolverWithContext::onTimerFired)
     , m_resolver(ScriptPromiseResolver::create(m_scriptState.get()))
+#if ASSERTION_ENABLED
+    , m_isPromiseCalled(false)
+#endif
 {
     if (executionContext()->activeDOMObjectsAreStopped())
         m_state = ResolvedOrRejected;
 }
 
-ScriptPromiseResolverWithContext::~ScriptPromiseResolverWithContext()
-{
-    if (m_state != ResolvedOrRejected) {
-        ScriptState::Scope scope(m_scriptState.get());
-        reject(v8::Exception::Error(v8::String::NewFromUtf8(m_scriptState->isolate(),
-            "ScriptPromiseResolverWithContext is destructed without resolve / reject")));
-    }
-}
-
 void ScriptPromiseResolverWithContext::suspend()
 {
     m_timer.stop();
 }
 
 void ScriptPromiseResolverWithContext::resume()
 {
     if (m_state == Resolving || m_state == Rejecting)
         m_timer.startOneShot(0, FROM_HERE);
 }
@@ skipping 42 matching lines @@
 
 void ScriptPromiseResolverWithContext::clear()
 {
     if (m_state == ResolvedOrRejected)
         return;
     ResolutionState state = m_state;
     m_state = ResolvedOrRejected;
     m_resolver.clear();
     m_value.clear();
     if (m_mode == KeepAliveWhilePending) {
-        // |ref| was called in the constructor.
+        // |ref| was called in |keepAliveWhilePending|.
         deref();
     }
     // |this| may be deleted here, but it is safe to check |state| because
     // it doesn't depend on |this|. When |this| is deleted, |state| can't be
     // |Resolving| nor |Rejecting| and hence |this->deref()| can't be executed.
     if (state == Resolving || state == Rejecting) {
         // |ref| was called in |resolveOrReject|.
         deref();
     }
 }
 
 } // namespace WebCore