Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1213)

Unified Diff: Source/core/streams/README.txt

Issue 1167343002: Add methods for creating V8 extras-based ReadableStreams from C++ (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: That todo is not necessary according to yhirano@ Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/streams/CountQueuingStrategy.js ('k') | Source/core/streams/ReadableStream2.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/streams/README.txt
diff --git a/Source/core/streams/README.txt b/Source/core/streams/README.txt
new file mode 100644
index 0000000000000000000000000000000000000000..08b1f350d23e1d81980929bbe4a77577b5e1bf4b
--- /dev/null
+++ b/Source/core/streams/README.txt
@@ -0,0 +1,43 @@
+This folder contains files from several different implementations and
+implementation strategies:
+
+## Traditional ReadableStream Implementation
+
+- ReadableByteStream.{cpp,h,idl}
+- ReadableByteStreamReader.{cpp,h,idl}
+- ReadableStream.{cpp,h,idl}
+- ReadableStreamImpl.h
+- ReadableStreamReader.{cpp,h,idl}
+- ReadableStreamReaderTest.cpp
+- ReadableStreamTest.cpp
+- UnderlyingSource.h
+
+These files implement the current streams spec, plus the more speculative
+ReadableByteStream, to the extent necessary to support Fetch response bodies.
+They do not support author-constructed readable streams. They use the normal
+approach for implementing web-exposed classes, i.e. IDL bindings with C++
+implementation. They are currently shipping in Chrome.
+
+## V8 Extras ReadableStream Implementation
+
+- ByteLengthQueuingStrategy.js
+- CountQueuingStrategy.js
+- QueuingStrategyBase.{cpp,h,idl}
+- ReadableStream2.{cpp,h,js}
+- ReadableStreamController.{cpp,h}
+- UnderlyingSourceBase.{cpp,h,idl}
+
+These files are an in-progress implementation of the current streams spec,
+using V8 extras [1]. They allow author construction. We hope eventually to
+have these supercede the traditional ReadableStream implementation, but the
+work to build Fetch response body streams on top of this is ongoing.
+
+[1]: https://docs.google.com/document/d/1AT5-T0aHGp7Lt29vPWFr2-qG8r3l9CByyvKwEuA8Ec0
+
+## Old streams
+
+- Stream.{cpp,h,idl}
+
+These files support an old streams spec. They should eventually be removed, but
+right now XMLHttpRequest and Media Streams Extension code in Blink still
+depends on them.
« no previous file with comments | « Source/core/streams/CountQueuingStrategy.js ('k') | Source/core/streams/ReadableStream2.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698