Revert of Oilpan: Prepare to make ExecutionContext GarbageCollectedMixin....

Revert of Oilpan: Prepare to make ExecutionContext GarbageCollectedMixin. (https://codereview.chromium.org/318023002/)

Reason for revert:
plugins/document-open.html is crashing on debug layout bots

Original issue's description:
> Oilpan: Prepare to make ExecutionContext GarbageCollectedMixin.
> 
> We creates closures with ExecutionContexts in LocalFileSystem. The
> ExecutionContext needs to be a CrossThreadPersistent data member of
> closure. This CL adds ParamStorageTraits specialized to garbage-collected
> classes.
> 
> This is a preparation to remove RefCounted from Node.
> 
> BUG=357163
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175734

TBR=oilpan-reviews@chromium.org,haraken@chromium.org,tkent@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=357163

Review URL: https://codereview.chromium.org/319333002

git-svn-id: svn://svn.chromium.org/blink/trunk@175741 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/Source/bindings/v8/CustomElementConstructorBuilder.cpp

@@ -156,6 +156,8 @@ PassRefPtr<CustomElementLifecycleCallbacks> CustomElementConstructorBuilder::cre
 {
     ASSERT(!m_prototype.IsEmpty());
 
+    RefPtr<ExecutionContext> executionContext = m_scriptState->executionContext();
+
     v8::TryCatch exceptionCatcher;
     exceptionCatcher.SetVerbose(true);
 

third_party/WebKit/Source/core/dom/Document.h

@@ -1132,10 +1132,8 @@ private:
     virtual PassRefPtrWillBeRawPtr<Node> cloneNode(bool deep = true) OVERRIDE FINAL;
     void cloneDataFromDocument(const Document&);
 
-#if !ENABLE(OILPAN)
     virtual void refExecutionContext() OVERRIDE FINAL { ref(); }
     virtual void derefExecutionContext() OVERRIDE FINAL { deref(); }
-#endif
 
     virtual const KURL& virtualURL() const OVERRIDE FINAL; // Same as url(), but needed for ExecutionContext to implement it without a performance loss for direct calls.
     virtual KURL virtualCompleteURL(const String&) const OVERRIDE FINAL; // Same as completeURL() for the same reason as above.

third_party/WebKit/Source/core/dom/ExecutionContext.h

@@ -38,7 +38,6 @@
 #include "core/frame/DOMTimer.h"
 #include "platform/LifecycleContext.h"
 #include "platform/Supplementable.h"
-#include "platform/heap/Handle.h"
 #include "platform/weborigin/KURL.h"
 #include "wtf/Functional.h"
 #include "wtf/OwnPtr.h"
@@ -61,10 +60,7 @@ class PublicURLManager;
 class SecurityOrigin;
 class ScriptCallStack;
 
-class ExecutionContext
-    : public WillBeGarbageCollectedMixin
-    , public LifecycleContext<ExecutionContext>
-    , public Supplementable<ExecutionContext> {
+class ExecutionContext : public LifecycleContext<ExecutionContext>, public Supplementable<ExecutionContext> {
 public:
     ExecutionContext();
     virtual ~ExecutionContext();
@@ -111,10 +107,9 @@ public:
 
     // Called after the construction of an ActiveDOMObject to synchronize suspend state.
     void suspendActiveDOMObjectIfNeeded(ActiveDOMObject*);
-#if !ENABLE(OILPAN)
+
     void ref() { refExecutionContext(); }
     void deref() { derefExecutionContext(); }
-#endif
 
     // Gets the next id in a circular sequence from 1 to 2^31-1.
     int circularSequentialID();
@@ -138,10 +133,8 @@ private:
 
     bool dispatchErrorEvent(PassRefPtrWillBeRawPtr<ErrorEvent>, AccessControlStatus);
 
-#if !ENABLE(OILPAN)
     virtual void refExecutionContext() = 0;
     virtual void derefExecutionContext() = 0;
-#endif
     // LifecycleContext implementation.
 
     // Implementation details for DOMTimer. No other classes should call these functions.

third_party/WebKit/Source/core/dom/MainThreadTaskRunnerTest.cpp

@@ -47,17 +47,13 @@ public:
     virtual void close() OVERRIDE { }
 };
 
-class NullExecutionContext : public RefCountedWillBeGarbageCollectedFinalized<NullExecutionContext>, public ExecutionContext {
-    WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(NullExecutionContext);
+class NullExecutionContext : public ExecutionContext, public RefCounted<NullExecutionContext> {
 public:
-    void trace(Visitor* visitor) { ExecutionContext::trace(visitor); }
-#if !ENABLE(OILPAN)
     using RefCounted<NullExecutionContext>::ref;
     using RefCounted<NullExecutionContext>::deref;
 
     virtual void refExecutionContext() OVERRIDE { ref(); }
     virtual void derefExecutionContext() OVERRIDE { deref(); }
-#endif
     virtual EventQueue* eventQueue() const OVERRIDE { return m_queue.get(); }
     virtual bool tasksNeedSuspension() { return m_tasksNeedSuspension; }
 
@@ -99,7 +95,7 @@ private:
 
 TEST(MainThreadTaskRunnerTest, PostTask)
 {
-    RefPtrWillBeRawPtr<NullExecutionContext> context = adoptRefWillBeNoop(new NullExecutionContext());
+    RefPtr<NullExecutionContext> context = adoptRef(new NullExecutionContext());
     OwnPtr<MainThreadTaskRunner> runner = MainThreadTaskRunner::create(context.get());
     bool isMarked = false;
 
@@ -111,7 +107,7 @@ TEST(MainThreadTaskRunnerTest, PostTask)
 
 TEST(MainThreadTaskRunnerTest, SuspendTask)
 {
-    RefPtrWillBeRawPtr<NullExecutionContext> context = adoptRefWillBeNoop(new NullExecutionContext());
+    RefPtr<NullExecutionContext> context = adoptRef(new NullExecutionContext());
     OwnPtr<MainThreadTaskRunner> runner = MainThreadTaskRunner::create(context.get());
     bool isMarked = false;
 
@@ -129,7 +125,7 @@ TEST(MainThreadTaskRunnerTest, SuspendTask)
 
 TEST(MainThreadTaskRunnerTest, RemoveRunner)
 {
-    RefPtrWillBeRawPtr<NullExecutionContext> context = adoptRefWillBeNoop(new NullExecutionContext());
+    RefPtr<NullExecutionContext> context = adoptRef(new NullExecutionContext());
     OwnPtr<MainThreadTaskRunner> runner = MainThreadTaskRunner::create(context.get());
     bool isMarked = false;
 

third_party/WebKit/Source/core/workers/WorkerGlobalScope.h

@@ -148,10 +148,8 @@ namespace WebCore {
         void addMessageToWorkerConsole(MessageSource, MessageLevel, const String& message, const String& sourceURL, unsigned lineNumber, PassRefPtr<ScriptCallStack>, ScriptState*);
 
     private:
-#if !ENABLE(OILPAN)
         virtual void refExecutionContext() OVERRIDE FINAL { ref(); }
         virtual void derefExecutionContext() OVERRIDE FINAL { deref(); }
-#endif
 
         virtual const KURL& virtualURL() const OVERRIDE FINAL;
         virtual KURL virtualCompleteURL(const String&) const OVERRIDE FINAL;

third_party/WebKit/Source/core/workers/WorkerMessagingProxy.h

@@ -86,7 +86,7 @@ namespace WebCore {
     private:
         static void workerObjectDestroyedInternal(ExecutionContext*, WorkerMessagingProxy*);
 
-        RefPtrWillBePersistent<ExecutionContext> m_executionContext;
+        RefPtr<ExecutionContext> m_executionContext;
         OwnPtr<WorkerObjectProxy> m_workerObjectProxy;
         Worker* m_workerObject;
         bool m_mayBeDestroyed;

third_party/WebKit/Source/modules/filesystem/FileSystemCallbacks.h

@@ -74,7 +74,7 @@ protected:
 
     OwnPtr<ErrorCallback> m_errorCallback;
     RefPtrWillBePersistent<DOMFileSystemBase> m_fileSystem;
-    RefPtrWillBePersistent<ExecutionContext> m_executionContext;
+    RefPtr<ExecutionContext> m_executionContext;
 };
 
 // Subclasses ----------------------------------------------------------------

third_party/WebKit/Source/modules/filesystem/LocalFileSystem.cpp

@@ -88,12 +88,12 @@ void LocalFileSystem::resolveURL(ExecutionContext* context, const KURL& fileSyst
     RefPtr<CallbackWrapper> wrapper = adoptRef(new CallbackWrapper(callbacks));
     requestFileSystemAccessInternal(context,
         bind(&LocalFileSystem::resolveURLInternal, this, fileSystemURL, wrapper),
-        bind(&LocalFileSystem::fileSystemNotAllowedInternal, this, PassRefPtrWillBeRawPtr<ExecutionContext>(context), wrapper));
+        bind(&LocalFileSystem::fileSystemNotAllowedInternal, this, PassRefPtr<ExecutionContext>(context), wrapper));
 }
 
 void LocalFileSystem::requestFileSystem(ExecutionContext* context, FileSystemType type, long long size, PassOwnPtr<AsyncFileSystemCallbacks> callbacks)
 {
-    RefPtrWillBeRawPtr<ExecutionContext> contextPtr(context);
+    RefPtr<ExecutionContext> contextPtr(context);
     RefPtr<CallbackWrapper> wrapper = adoptRef(new CallbackWrapper(callbacks));
     requestFileSystemAccessInternal(context,
         bind(&LocalFileSystem::fileSystemAllowedInternal, this, contextPtr, type, wrapper),
@@ -102,7 +102,7 @@ void LocalFileSystem::requestFileSystem(ExecutionContext* context, FileSystemTyp
 
 void LocalFileSystem::deleteFileSystem(ExecutionContext* context, FileSystemType type, PassOwnPtr<AsyncFileSystemCallbacks> callbacks)
 {
-    RefPtrWillBeRawPtr<ExecutionContext> contextPtr(context);
+    RefPtr<ExecutionContext> contextPtr(context);
     ASSERT(context);
     ASSERT_WITH_SECURITY_IMPLICATION(context->isDocument());
 
@@ -130,14 +130,14 @@ void LocalFileSystem::requestFileSystemAccessInternal(ExecutionContext* context,
 }
 
 void LocalFileSystem::fileSystemNotAllowedInternal(
-    PassRefPtrWillBeRawPtr<ExecutionContext> context,
+    PassRefPtr<ExecutionContext> context,
     PassRefPtr<CallbackWrapper> callbacks)
 {
     context->postTask(createCallbackTask(&fileSystemNotAllowed, callbacks->release()));
 }
 
 void LocalFileSystem::fileSystemAllowedInternal(
-    PassRefPtrWillBeRawPtr<ExecutionContext> context,
+    PassRefPtr<ExecutionContext> context,
     FileSystemType type,
     PassRefPtr<CallbackWrapper> callbacks)
 {
@@ -153,7 +153,7 @@ void LocalFileSystem::resolveURLInternal(
 }
 
 void LocalFileSystem::deleteFileSystemInternal(
-    PassRefPtrWillBeRawPtr<ExecutionContext> context,
+    PassRefPtr<ExecutionContext> context,
     FileSystemType type,
     PassRefPtr<CallbackWrapper> callbacks)
 {

third_party/WebKit/Source/modules/filesystem/LocalFileSystem.h

@@ -71,10 +71,10 @@ protected:
 
 private:
     void requestFileSystemAccessInternal(ExecutionContext*, const Closure& allowed, const Closure& denied);
-    void fileSystemNotAllowedInternal(PassRefPtrWillBeRawPtr<ExecutionContext>, PassRefPtr<CallbackWrapper>);
-    void fileSystemAllowedInternal(PassRefPtrWillBeRawPtr<ExecutionContext>, FileSystemType, PassRefPtr<CallbackWrapper>);
+    void fileSystemNotAllowedInternal(PassRefPtr<ExecutionContext>, PassRefPtr<CallbackWrapper>);
+    void fileSystemAllowedInternal(PassRefPtr<ExecutionContext>, FileSystemType, PassRefPtr<CallbackWrapper>);
     void resolveURLInternal(const KURL&, PassRefPtr<CallbackWrapper>);
-    void deleteFileSystemInternal(PassRefPtrWillBeRawPtr<ExecutionContext>, FileSystemType, PassRefPtr<CallbackWrapper>);
+    void deleteFileSystemInternal(PassRefPtr<ExecutionContext>, FileSystemType, PassRefPtr<CallbackWrapper>);
     OwnPtr<FileSystemClient> m_client;
 };
 

third_party/WebKit/Source/modules/indexeddb/IDBRequestTest.cpp

@@ -53,17 +53,13 @@ public:
     virtual void close() OVERRIDE { }
 };
 
-class NullExecutionContext FINAL : public RefCountedWillBeGarbageCollectedFinalized<NullExecutionContext>, public ExecutionContext {
-    WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(NullExecutionContext);
+class NullExecutionContext FINAL : public ExecutionContext, public RefCounted<NullExecutionContext> {
 public:
-    void trace(Visitor* visitor) { ExecutionContext::trace(visitor); }
-#if !ENABLE(OILPAN)
     using RefCounted<NullExecutionContext>::ref;
     using RefCounted<NullExecutionContext>::deref;
 
     virtual void refExecutionContext() OVERRIDE { ref(); }
     virtual void derefExecutionContext() OVERRIDE { deref(); }
-#endif
     virtual EventQueue* eventQueue() const OVERRIDE { return m_queue.get(); }
 
     NullExecutionContext();
@@ -80,7 +76,7 @@ class IDBRequestTest : public testing::Test {
 public:
     IDBRequestTest()
         : m_scope(v8::Isolate::GetCurrent())
-        , m_executionContext(adoptRefWillBeNoop(new NullExecutionContext()))
+        , m_executionContext(adoptRef(new NullExecutionContext()))
     {
         m_scope.scriptState()->setExecutionContext(m_executionContext.get());
     }
@@ -96,7 +92,7 @@ public:
 
 private:
     V8TestingScope m_scope;
-    RefPtrWillBePersistent<ExecutionContext> m_executionContext;
+    RefPtr<ExecutionContext> m_executionContext;
 };
 
 TEST_F(IDBRequestTest, EventsAfterStopping)

third_party/WebKit/Source/modules/indexeddb/IDBTransactionTest.cpp

@@ -67,7 +67,7 @@ public:
 
 private:
     V8TestingScope m_scope;
-    RefPtrWillBePersistent<ExecutionContext> m_executionContext;
+    RefPtr<ExecutionContext> m_executionContext;
 };
 
 class FakeWebIDBDatabase FINAL : public blink::WebIDBDatabase {

third_party/WebKit/Source/modules/webdatabase/SQLCallbackWrapper.h

@@ -56,7 +56,8 @@ public:
         clear();
     }
 
-    void trace(Visitor* visitor) { visitor->trace(m_executionContext); }
+    // FIXME: Oilpan: Trace m_executionContext.
+    void trace(Visitor* visitor) { }
 
     void clear()
     {
@@ -129,7 +130,7 @@ private:
 
     Mutex m_mutex;
     OwnPtr<T> m_callback;
-    RefPtrWillBeMember<ExecutionContext> m_executionContext;
+    RefPtr<ExecutionContext> m_executionContext;
 };
 
 } // namespace WebCore

third_party/WebKit/Source/platform/heap/Handle.h

@@ -34,7 +34,6 @@
 #include "platform/heap/Heap.h"
 #include "platform/heap/ThreadState.h"
 #include "platform/heap/Visitor.h"
-#include "wtf/Functional.h"
 #include "wtf/HashFunctions.h"
 #include "wtf/Locker.h"
 #include "wtf/RawPtr.h"
@@ -1153,36 +1152,6 @@ struct NeedsTracing<ListHashSetNode<T, WebCore::HeapListHashSetAllocator<T, inli
     static const bool value = true;
 };
 
-// For wtf/Functional.h
-template<typename T, bool isGarbageCollected> struct PointerParamStorageTraits;
-
-template<typename T> struct PointerParamStorageTraits<T*, false> {
-    typedef T* StorageType;
-
-    static StorageType wrap(T* value) { return value; }
-    static T* unwrap(const StorageType& value) { return value; }
-};
-
-template<typename T> struct PointerParamStorageTraits<T*, true> {
-    typedef WebCore::CrossThreadPersistent<T> StorageType;
-
-    static StorageType wrap(T* value) { return value; }
-    static T* unwrap(const StorageType& value) { return value.get(); }
-};
-
-// FIXME: This doesn't support collections and const types. See
-// COMPILE_ASSERT_IS_GARBAGE_COLLECTED.
-template<typename T> struct ParamStorageTraits<T*> : public PointerParamStorageTraits<T*, WTF::IsSubclassOfTemplate<T, WebCore::GarbageCollected>::value || WebCore::IsGarbageCollectedMixin<T>::value> {
-};
-
-// We assume RawPtr<T> is used only for garbage-collected types.
-template<typename T> struct ParamStorageTraits<RawPtr<T> > {
-    typedef WebCore::CrossThreadPersistent<T> StorageType;
-
-    static StorageType wrap(RawPtr<T> value) { return value.get(); }
-    static T* unwrap(const StorageType& value) { return value.get(); }
-};
-
 } // namespace WTF
 
 #endif

third_party/WebKit/Source/platform/heap/HeapTest.cpp

@@ -3978,24 +3978,4 @@ TEST(HeapTest, NeedsAdjustAndMark)
     EXPECT_FALSE(NeedsAdjustAndMark<const UseMixin>::value);
 }
 
-TEST(HeapTest, Bind)
-{
-    Closure closure = bind(&Bar::trace, Bar::create(), static_cast<Visitor*>(0));
-    Heap::collectGarbage(ThreadState::NoHeapPointersOnStack);
-    // The closure should have a persistent handle to the Bar.
-    EXPECT_EQ(1u, Bar::s_live);
-
-    Closure closure2 = bind(&Bar::trace, RawPtr<Bar>(Bar::create()), static_cast<Visitor*>(0));
-    Heap::collectGarbage(ThreadState::NoHeapPointersOnStack);
-    // The closure should have a persistent handle to the Bar.
-    EXPECT_EQ(2u, Bar::s_live);
-
-    UseMixin::s_traceCount = 0;
-    Mixin* mixin = UseMixin::create();
-    Closure mixinClosure = bind(&Mixin::trace, mixin, static_cast<Visitor*>(0));
-    Heap::collectGarbage(ThreadState::NoHeapPointersOnStack);
-    // The closure should have a persistent handle to the mixin.
-    EXPECT_EQ(1, UseMixin::s_traceCount);
-}
-
 } // WebCore namespace

third_party/WebKit/Source/web/WebSharedWorkerImpl.h

@@ -131,7 +131,7 @@ private:
     void workerGlobalScopeDestroyedOnMainThread();
 
     // 'shadow page' - created to proxy loading requests from the worker.
-    RefPtrWillBePersistent<WebCore::ExecutionContext> m_loadingDocument;
+    RefPtr<WebCore::ExecutionContext> m_loadingDocument;
     WebView* m_webView;
     WebFrame* m_mainFrame;
     bool m_askedToTerminate;