[Streams API] Shield streams against user replacing the Promise constructor
[WebKit-https.git] / Source / WebCore / ChangeLog
index bbbf0fe742c1d13e3afc94e1053ecdf0d7385b27..0c47e0baac491af5d0e8bb1c6fa99f93ca7754c4 100644 (file)
@@ -1,3 +1,39 @@
+2015-11-04  Xabier Rodriguez Calvar  <calvaris@igalia.com>
+
+        [Streams API] Shield streams against user replacing the Promise constructor
+        https://bugs.webkit.org/show_bug.cgi?id=150887
+
+        Reviewed by Youenn Fablet.
+
+        With this rework, we shield the Streams implementation against the user doing something like "Promise =
+        function() { /* do garbage */ };".
+
+        Test: streams/streams-promises.html.
+
+        * Modules/streams/ReadableStream.js:
+        (initializeReadableStream):
+        (cancel):
+        * Modules/streams/ReadableStreamInternals.js:
+        (privateInitializeReadableStreamReader):
+        (cancelReadableStream):
+        (readFromReadableStreamReader):
+        * Modules/streams/ReadableStreamReader.js:
+        (cancel):
+        (read):
+        (closed):
+        * Modules/streams/StreamInternals.js:
+        (promiseInvokeOrNoop):
+        (promiseInvokeOrFallbackOrNoop):
+        * Modules/streams/WritableStream.js:
+        (initializeWritableStream):
+        (abort):
+        (close):
+        (write):
+        (closed):
+        (ready):
+        * Modules/streams/WritableStreamInternals.js:
+        (syncWritableStreamStateWithQueue):
+
 2015-11-04  Antoine Quint  <graouts@apple.com>
 
         SVG: hit testing region for <text> elements is incorrect