ReadableStreamSource was added as a way to define a simple interface for native sources. This interface is not well suited to JS sources since the code needed to handle JS sources is scattered between ReadableJSStream and ReadableJSStream::Source. It may be best to remove ReadableStreamSource from ReadableStream and ReadableJSStream. We would then reintroduce it when supporting native streams as a member of a ReadableNativeStream. ReadableNativeStream would be the equivalent of ReadableJSStream for native sources.
Created attachment 254164 [details] Patch
Comment on attachment 254164 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=254164&action=review > Source/WebCore/Modules/streams/ReadableStream.h:87 > + ReadableStream(ScriptExecutionContext&); Should add explicit here.
Created attachment 254255 [details] Patch for landing
Comment on attachment 254255 [details] Patch for landing Clearing flags on attachment: 254255 Committed r185196: <http://trac.webkit.org/changeset/185196>
All reviewed patches have been landed. Closing bug.