Have ScriptPromiseResolver on the Oilpan heap always.

Source/core/streams/ReadableStreamImpl.h

Index: Source/core/streams/ReadableStreamImpl.h
diff --git a/Source/core/streams/ReadableStreamImpl.h b/Source/core/streams/ReadableStreamImpl.h
index 719fe8e040a940fe2c8cb9456434f1bfc9b3cee0..fc3d3f5e8d1ba69a473a1c9a330b22e1feec75ca 100644
--- a/Source/core/streams/ReadableStreamImpl.h
+++ b/Source/core/streams/ReadableStreamImpl.h
@@ -118,18 +118,12 @@ public:
     DEFINE_INLINE_VIRTUAL_TRACE()
     {
         visitor->trace(m_strategy);
-#if ENABLE(OILPAN)
         visitor->trace(m_pendingReads);
-#endif
         ReadableStream::trace(visitor);
     }
 
 private:
-#if ENABLE(OILPAN)
     using PendingReads = HeapDeque<Member<ScriptPromiseResolver>>;
-#else
-    using PendingReads = Deque<RefPtr<ScriptPromiseResolver>>;
-#endif
 
     // ReadableStream methods
     bool isQueueEmpty() const override { return m_queue.isEmpty(); }
@@ -180,7 +174,7 @@ bool ReadableStreamImpl<ChunkTypeTraits>::enqueue(typename ChunkTypeTraits::Pass
         return enqueuePostAction();
     }
 
-    RefPtrWillBeRawPtr<ScriptPromiseResolver> resolver = m_pendingReads.takeFirst();
+    ScriptPromiseResolver* resolver = m_pendingReads.takeFirst();
     ScriptState* scriptState = resolver->scriptState();

[haraken, 2015/07/17 01:40:35]

Not related to this CL, we should have:

  if (!scriptState->contextIsValid())
    return false;

[sof, 2015/07/17 09:43:26]

Done, here & in resolveAllPendingReadsAsDone()

yhirano@: ptal ?

[yhirano, 2015/07/17 13:53:41]

haraken@: Is it guaranteed that the document is stopped when
scriptState->contextIsValid() returns false? Otherwise an assertion in the
ScriptPromiseResolver fails and we should modify there as well.

[haraken, 2015/07/18 07:10:27]

It is guaranteed for a document that is associated with a frame. It is not
guaranteed for some documents that don't have a frame, but that's an edge case
(e.g., XSLT documents).

     ScriptState::Scope scope(scriptState);
     resolver->resolve(v8IteratorResult(scriptState, chunk));