Remove NotEnumerable from Transform Streams WebIDL

According to whatwg/streams#1035, all methods and accessors are now enumerable, per Web IDL defaults, instead of non-enumerable, per ECMAScript defaults. Hence, 'NotEnumerable' can be removed from the Streams WebIDL files. This CL specifically removes them from transform streams. Bug: 1093862 Change-Id: I509b470722039ebad20b8caeaa7f34189c24b1ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2486006 Commit-Queue: Nidhi Jaju <nidhijaju@google.com> Reviewed-by: Adam Rice <ricea@chromium.org> Cr-Commit-Position: refs/heads/master@{#819230}
Mu-L · Oct 21, 2020 · b52b97c · b52b97c
1 parent 200a9f5
commit b52b97c
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 18 deletions.
diff --git a/third_party/blink/renderer/core/streams/transform_stream.idl b/third_party/blink/renderer/core/streams/transform_stream.idl
@@ -9,6 +9,6 @@
     [CallWith=ScriptState, RaisesException] constructor(optional any transformer,
                 optional any writableStrategy,
                 optional any readableStrategy);
-    [NotEnumerable] readonly attribute ReadableStream readable;
-    [NotEnumerable] readonly attribute WritableStream writable;
+    readonly attribute ReadableStream readable;
+    readonly attribute WritableStream writable;
 };
diff --git a/third_party/blink/renderer/core/streams/transform_stream_default_controller.idl b/third_party/blink/renderer/core/streams/transform_stream_default_controller.idl
@@ -6,9 +6,9 @@
 
 // https://streams.spec.whatwg.org/#rs-default-controller-class-definition
 interface TransformStreamDefaultController {
-    [NotEnumerable] readonly attribute double? desiredSize;
-    [CallWith=ScriptState, NotEnumerable, RaisesException] void enqueue(
+    readonly attribute double? desiredSize;
+    [CallWith=ScriptState, RaisesException] void enqueue(
         optional any chunk);
-    [CallWith=ScriptState, NotEnumerable] void error(optional any reason);
-    [CallWith=ScriptState, NotEnumerable] void terminate();
+    [CallWith=ScriptState] void error(optional any reason);
+    [CallWith=ScriptState] void terminate();
 };
diff --git a/third_party/blink/web_tests/external/wpt/streams/idlharness.any-expected.txt b/third_party/blink/web_tests/external/wpt/streams/idlharness.any-expected.txt
@@ -1,5 +1,5 @@
 This is a testharness.js-based test.
-Found 224 tests; 126 PASS, 98 FAIL, 0 TIMEOUT, 0 NOTRUN.
+Found 224 tests; 138 PASS, 86 FAIL, 0 TIMEOUT, 0 NOTRUN.
 PASS idl_test setup
 PASS idl_test validation
 PASS ReadableStreamDefaultReader includes ReadableStreamGenericReader: member names are unique
@@ -176,8 +176,8 @@ PASS TransformStream interface object name
 PASS TransformStream interface: existence and properties of interface prototype object
 PASS TransformStream interface: existence and properties of interface prototype object's "constructor" property
 PASS TransformStream interface: existence and properties of interface prototype object's @@unscopables property
-FAIL TransformStream interface: attribute readable assert_true: property should be enumerable expected true got false
-FAIL TransformStream interface: attribute writable assert_true: property should be enumerable expected true got false
+PASS TransformStream interface: attribute readable
+PASS TransformStream interface: attribute writable
 PASS TransformStream must be primary interface of new TransformStream()
 PASS Stringification of new TransformStream()
 PASS TransformStream interface: new TransformStream() must inherit property "readable" with the proper type

diff --git a/third_party/blink/web_tests/external/wpt/streams/idlharness.any.serviceworker-expected.txt b/third_party/blink/web_tests/external/wpt/streams/idlharness.any.serviceworker-expected.txt
@@ -1,5 +1,5 @@
 This is a testharness.js-based test.
-Found 224 tests; 126 PASS, 98 FAIL, 0 TIMEOUT, 0 NOTRUN.
+Found 224 tests; 138 PASS, 86 FAIL, 0 TIMEOUT, 0 NOTRUN.
 PASS idl_test setup
 PASS idl_test validation
 PASS ReadableStreamDefaultReader includes ReadableStreamGenericReader: member names are unique
@@ -176,8 +176,8 @@ PASS TransformStream interface object name
 PASS TransformStream interface: existence and properties of interface prototype object
 PASS TransformStream interface: existence and properties of interface prototype object's "constructor" property
 PASS TransformStream interface: existence and properties of interface prototype object's @@unscopables property
-FAIL TransformStream interface: attribute readable assert_true: property should be enumerable expected true got false
-FAIL TransformStream interface: attribute writable assert_true: property should be enumerable expected true got false
+PASS TransformStream interface: attribute readable
+PASS TransformStream interface: attribute writable
 PASS TransformStream must be primary interface of new TransformStream()
 PASS Stringification of new TransformStream()
 PASS TransformStream interface: new TransformStream() must inherit property "readable" with the proper type

diff --git a/third_party/blink/web_tests/external/wpt/streams/idlharness.any.sharedworker-expected.txt b/third_party/blink/web_tests/external/wpt/streams/idlharness.any.sharedworker-expected.txt
@@ -1,5 +1,5 @@
 This is a testharness.js-based test.
-Found 224 tests; 126 PASS, 98 FAIL, 0 TIMEOUT, 0 NOTRUN.
+Found 224 tests; 138 PASS, 86 FAIL, 0 TIMEOUT, 0 NOTRUN.
 PASS idl_test setup
 PASS idl_test validation
 PASS ReadableStreamDefaultReader includes ReadableStreamGenericReader: member names are unique
@@ -176,8 +176,8 @@ PASS TransformStream interface object name
 PASS TransformStream interface: existence and properties of interface prototype object
 PASS TransformStream interface: existence and properties of interface prototype object's "constructor" property
 PASS TransformStream interface: existence and properties of interface prototype object's @@unscopables property
-FAIL TransformStream interface: attribute readable assert_true: property should be enumerable expected true got false
-FAIL TransformStream interface: attribute writable assert_true: property should be enumerable expected true got false
+PASS TransformStream interface: attribute readable
+PASS TransformStream interface: attribute writable
 PASS TransformStream must be primary interface of new TransformStream()
 PASS Stringification of new TransformStream()
 PASS TransformStream interface: new TransformStream() must inherit property "readable" with the proper type

diff --git a/third_party/blink/web_tests/external/wpt/streams/idlharness.any.worker-expected.txt b/third_party/blink/web_tests/external/wpt/streams/idlharness.any.worker-expected.txt
@@ -1,5 +1,5 @@
 This is a testharness.js-based test.
-Found 224 tests; 126 PASS, 98 FAIL, 0 TIMEOUT, 0 NOTRUN.
+Found 224 tests; 138 PASS, 86 FAIL, 0 TIMEOUT, 0 NOTRUN.
 PASS idl_test setup
 PASS idl_test validation
 PASS ReadableStreamDefaultReader includes ReadableStreamGenericReader: member names are unique
@@ -176,8 +176,8 @@ PASS TransformStream interface object name
 PASS TransformStream interface: existence and properties of interface prototype object
 PASS TransformStream interface: existence and properties of interface prototype object's "constructor" property
 PASS TransformStream interface: existence and properties of interface prototype object's @@unscopables property
-FAIL TransformStream interface: attribute readable assert_true: property should be enumerable expected true got false
-FAIL TransformStream interface: attribute writable assert_true: property should be enumerable expected true got false
+PASS TransformStream interface: attribute readable
+PASS TransformStream interface: attribute writable
 PASS TransformStream must be primary interface of new TransformStream()
 PASS Stringification of new TransformStream()
 PASS TransformStream interface: new TransformStream() must inherit property "readable" with the proper type