Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 This directory contains files from several different implementations and | 1 This directory contains files from several different implementations and |
| 2 implementation strategies: | 2 implementation strategies: |
| 3 | 3 |
| 4 ## Traditional ReadableStream Implementation | 4 ## Traditional ReadableStream Implementation |
| 5 | 5 |
| 6 - ReadableByteStream.{cpp,h,idl} | 6 - ReadableByteStream.{cpp,h,idl} |
| 7 - ReadableByteStreamReader.{h,idl} | 7 - ReadableByteStreamReader.{h,idl} |
| 8 - ReadableStream.{cpp,h,idl} | 8 - ReadableStream.{cpp,h,idl} |
| 9 - ReadableStreamImpl.h | 9 - ReadableStreamImpl.h |
| 10 - ReadableStreamReader.{cpp,h,idl} | 10 - ReadableStreamReader.{cpp,h,idl} |
| 11 - ReadableStreamReaderTest.cpp | 11 - ReadableStreamReaderTest.cpp |
| 12 - ReadableStreamTest.cpp | 12 - ReadableStreamTest.cpp |
| 13 - UnderlyingSource.h | 13 - UnderlyingSource.h |
| 14 | 14 |
| 15 These files implement the current streams spec, plus the more speculative | 15 These files implement the current streams spec, plus the more speculative |
| 16 ReadableByteStream, to the extent necessary to support Fetch response bodies. | 16 ReadableByteStream, to the extent necessary to support Fetch response bodies. |
| 17 They do not support author-constructed readable streams. They use the normal | 17 They do not support author-constructed readable streams. They use the normal |
| 18 approach for implementing web-exposed classes, i.e. IDL bindings with C++ | 18 approach for implementing web-exposed classes, i.e. IDL bindings with C++ |
| 19 implementation. They are currently shipping in Chrome. | 19 implementation. They are currently shipping in Chrome. |
| 20 | 20 |
| 21 ## V8 Extras ReadableStream Implementation | 21 ## V8 Extras ReadableStream Implementation |
| 22 | 22 |
| 23 - ByteLengthQueuingStrategy.js | 23 - ByteLengthQueuingStrategy.js |
| 24 - CountQueuingStrategy.js | 24 - CountQueuingStrategy.js |
| 25 - ReadableStream.js | 25 - ReadableStream.js |
| 26 - ReadableStreamTempStub.js | |
| 27 - ReadableStreamController.h | 26 - ReadableStreamController.h |
| 28 - UnderlyingSourceBase.{cpp,h,idl} | 27 - UnderlyingSourceBase.{cpp,h,idl} |
| 29 - bindings/core/v8/ReadableStreamOperations.{cpp,h} | 28 - bindings/core/v8/ReadableStreamOperations.{cpp,h} |
| 30 | 29 |
| 31 These files are an in-progress implementation of the current streams spec, | 30 These files are an in-progress implementation of the current streams spec, |
| 32 using [V8 extras][1]. They allow author construction. We hope eventually to | 31 using [V8 extras][1]. They allow author construction. We hope eventually to |
| 33 have these supercede the traditional ReadableStream implementation, but the | 32 have these supercede the traditional ReadableStream implementation, but the |
| 34 work to build Fetch response body streams on top of this is ongoing. | 33 work to build Fetch response body streams on top of this is ongoing. |
|
tyoshino (SeeGerritForStatus)
2016/05/19 06:44:40
Remove?
yhirano
2016/05/19 07:14:57
Updated the description.
tyoshino (SeeGerritForStatus)
2016/05/19 07:55:08
LGTM
| |
| 35 | 34 |
| 36 [1]: https://docs.google.com/document/d/1AT5-T0aHGp7Lt29vPWFr2-qG8r3l9CByyvKwEuA 8Ec0 | 35 [1]: https://docs.google.com/document/d/1AT5-T0aHGp7Lt29vPWFr2-qG8r3l9CByyvKwEuA 8Ec0 |
| 37 | 36 |
| 38 ## Old streams | 37 ## Old streams |
| 39 | 38 |
| 40 - Stream.{cpp,h,idl} | 39 - Stream.{cpp,h,idl} |
| 41 | 40 |
| 42 These files support an old streams spec. They should eventually be removed, but | 41 These files support an old streams spec. They should eventually be removed, but |
| 43 right now XMLHttpRequest and Media Streams Extension code in Blink still | 42 right now XMLHttpRequest and Media Streams Extension code in Blink still |
| 44 depends on them. | 43 depends on them. |
| OLD | NEW |