Have bindings layer assume and insist that all interface types are GCed.

third_party/WebKit/Source/bindings/core/v8/ScriptPromisePropertyTest.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 "bindings/core/v8/ScriptPromiseProperty.h"
 
 #include "bindings/core/v8/DOMWrapperWorld.h"
 #include "bindings/core/v8/ScriptFunction.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/core/v8/ScriptValue.h"
 #include "bindings/core/v8/V8Binding.h"
 #include "bindings/core/v8/V8BindingForTesting.h"
 #include "bindings/core/v8/V8GCController.h"
 #include "core/dom/Document.h"
 #include "core/testing/DummyPageHolder.h"
 #include "core/testing/GCObservation.h"
 #include "core/testing/GarbageCollectedScriptWrappable.h"
-#include "core/testing/RefCountedScriptWrappable.h"
 #include "platform/heap/Handle.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "wtf/OwnPtr.h"
 #include "wtf/PassOwnPtr.h"
 #include "wtf/PassRefPtr.h"
 #include "wtf/RefPtr.h"
 #include <v8.h>
 
 using namespace blink;
 
@@ skipping 62 matching lines @@
     DEFINE_INLINE_VIRTUAL_TRACE()
     {
         GarbageCollectedScriptWrappable::trace(visitor);
         visitor->trace(m_property);
     }
 
 private:
     Member<Property> m_property;
 };
 
-class RefCountedHolder : public RefCountedScriptWrappable {
-public:
-    // Do not resolve or reject the property with the holder itself. It leads
-    // to a leak.
-    typedef ScriptPromiseProperty<RefCountedScriptWrappable*, RefPtr<RefCountedScriptWrappable>, RefPtr<RefCountedScriptWrappable>> Property;
-    static PassRefPtr<RefCountedHolder> create(ExecutionContext* executionContext)
-    {
-        return adoptRef(new RefCountedHolder(executionContext));
-    }
-    Property* getProperty() { return m_property; }
-    RefCountedScriptWrappable* toRefCountedScriptWrappable() { return this; }
-
-private:
-    RefCountedHolder(ExecutionContext* executionContext)
-        : RefCountedScriptWrappable("holder")
-        , m_property(new Property(executionContext, toRefCountedScriptWrappable(), Property::Ready)) { }
-
-    Persistent<Property> m_property;
-};
-
 class ScriptPromisePropertyTestBase {
 public:
     ScriptPromisePropertyTestBase()
         : m_page(DummyPageHolder::create(IntSize(1, 1)))
     {
         v8::HandleScope handleScope(isolate());
         m_otherScriptState = ScriptStateForTesting::create(v8::Context::New(isolate()), DOMWrapperWorld::ensureIsolatedWorld(isolate(), 1, -1));
     }
 
     virtual ~ScriptPromisePropertyTestBase()
@@ skipping 275 matching lines @@
 
     v8::MicrotasksScope::PerformCheckpoint(isolate());
     EXPECT_EQ(1u, nOldResolveCalls);
     EXPECT_EQ(1u, nNewRejectCalls);
     EXPECT_NE(oldPromise, newPromise);
     EXPECT_EQ(wrap(mainWorld(), oldValue), oldActual);
     EXPECT_EQ(wrap(mainWorld(), newValue), newActual);
     EXPECT_NE(oldActual, newActual);
 }
 
-// Tests that ScriptPromiseProperty works with a ref-counted holder.
-class ScriptPromisePropertyRefCountedTest : public ScriptPromisePropertyTestBase, public ::testing::Test {
-public:
-    typedef RefCountedHolder::Property Property;
-
-    ScriptPromisePropertyRefCountedTest()
-        : m_holder(RefCountedHolder::create(&document()))
-    {
-    }
-
-    RefCountedHolder* holder() { return m_holder.get(); }
-    Property* getProperty() { return m_holder->getProperty(); }
-
-private:
-    RefPtr<RefCountedHolder> m_holder;
-};
-
-TEST_F(ScriptPromisePropertyRefCountedTest, Resolve)
-{
-    ScriptValue actual;
-    size_t nResolveCalls = 0;
-
-    {
-        ScriptState::Scope scope(mainScriptState());
-        getProperty()->promise(DOMWrapperWorld::mainWorld()).then(stub(currentScriptState(), actual, nResolveCalls), notReached(currentScriptState()));
-    }
-
-    RefPtr<RefCountedScriptWrappable> value = RefCountedScriptWrappable::create("value");
-    getProperty()->resolve(value.get());
-    EXPECT_EQ(Property::Resolved, getProperty()->getState());
-
-    v8::MicrotasksScope::PerformCheckpoint(isolate());
-    EXPECT_EQ(1u, nResolveCalls);
-    EXPECT_EQ(wrap(mainWorld(), value), actual);
-}
-
-TEST_F(ScriptPromisePropertyRefCountedTest, Reject)
-{
-    ScriptValue actual;
-    size_t nRejectCalls = 0;
-
-    {
-        ScriptState::Scope scope(mainScriptState());
-        getProperty()->promise(DOMWrapperWorld::mainWorld()).then(notReached(currentScriptState()), stub(currentScriptState(), actual, nRejectCalls));
-    }
-
-    RefPtr<RefCountedScriptWrappable> reason = RefCountedScriptWrappable::create("reason");
-    getProperty()->reject(reason);
-    EXPECT_EQ(Property::Rejected, getProperty()->getState());
-
-    v8::MicrotasksScope::PerformCheckpoint(isolate());
-    EXPECT_EQ(1u, nRejectCalls);
-    EXPECT_EQ(wrap(mainWorld(), reason), actual);
-}
-
-TEST_F(ScriptPromisePropertyRefCountedTest, ReSolveAndReset)
-{
-    RefPtr<RefCountedScriptWrappable> value = RefCountedScriptWrappable::create("value");
-
-    {
-        ScriptState::Scope scope(mainScriptState());
-        getProperty()->resolve(value);
-    }
-
-    EXPECT_EQ(2, value->refCount());
-    getProperty()->reset();
-    EXPECT_EQ(1, value->refCount());
-}
-
-TEST_F(ScriptPromisePropertyRefCountedTest, RejectAndReset)
-{
-    RefPtr<RefCountedScriptWrappable> value = RefCountedScriptWrappable::create("value");
-
-    {
-        ScriptState::Scope scope(mainScriptState());
-        getProperty()->reject(value.get());
-    }
-
-    EXPECT_EQ(2, value->refCount());
-    getProperty()->reset();
-    EXPECT_EQ(1, value->refCount());
-}
-
 // Tests that ScriptPromiseProperty works with a non ScriptWrappable resolution
 // target.
 class ScriptPromisePropertyNonScriptWrappableResolutionTargetTest : public ScriptPromisePropertyTestBase, public ::testing::Test {
 public:
     template <typename T>
     void test(const T& value, const char* expected, const char* file, size_t line)
     {
         typedef ScriptPromiseProperty<Member<GarbageCollectedScriptWrappable>, T, ToV8UndefinedGenerator> Property;
         Property* property = new Property(&document(), new GarbageCollectedScriptWrappable("holder"), Property::Ready);
         size_t nResolveCalls = 0;
@@ skipping 25 matching lines @@
 {
     test(String("hello"), "hello", __FILE__, __LINE__);
 }
 
 TEST_F(ScriptPromisePropertyNonScriptWrappableResolutionTargetTest, ResolveWithInteger)
 {
     test(-1, "-1", __FILE__, __LINE__);
 }
 
 } // namespace