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

Unified Diff: third_party/WebKit/Source/core/streams/README.md

Issue 1404523005: Implement author-constructible ReadableStream using V8 extras (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Minor test tweaks Created 5 years, 1 month 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
Index: third_party/WebKit/Source/core/streams/README.md
diff --git a/third_party/WebKit/Source/core/streams/README.md b/third_party/WebKit/Source/core/streams/README.md
new file mode 100644
index 0000000000000000000000000000000000000000..9de53e041478294e5a1a54a8ce264277acf30f04
--- /dev/null
+++ b/third_party/WebKit/Source/core/streams/README.md
@@ -0,0 +1,41 @@
+This directory contains files from several different implementations and
+implementation strategies:
+
+## Traditional ReadableStream Implementation
+
+- ReadableByteStream.{cpp,h,idl}
+- ReadableByteStreamReader.{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
+- ReadableStream.js
+- ReadableStreamTempStub.js
+
+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.

Powered by Google App Engine
This is Rietveld 408576698