From 04529c04d8c3a3f947386b271013390102dfb055 Mon Sep 17 00:00:00 2001 From: Nidhi Jaju Date: Wed, 21 Oct 2020 16:08:53 +0000 Subject: [PATCH] Reland "Remove NotEnumerable from Transform Streams WebIDL" This is a reland of b52b97cb36de5ae9a366d76548b6f9aa32e234d7 Original change's description: > Remove NotEnumerable from Transform Streams WebIDL > > According to https://github.com/whatwg/streams/pull/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 > Reviewed-by: Adam Rice > Cr-Commit-Position: refs/heads/master@{#819230} Bug: 1093862 Change-Id: Ic3d119e1c5d2192a3159fb672a5782fa2af8aedc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2488946 Commit-Queue: Adam Rice Reviewed-by: Adam Rice Cr-Commit-Position: refs/heads/master@{#819380} --- .../blink/renderer/core/streams/transform_stream.idl | 4 ++-- .../core/streams/transform_stream_default_controller.idl | 8 ++++---- .../external/wpt/streams/idlharness.any-expected.txt | 6 +++--- .../wpt/streams/idlharness.any.serviceworker-expected.txt | 6 +++--- .../wpt/streams/idlharness.any.sharedworker-expected.txt | 6 +++--- .../wpt/streams/idlharness.any.worker-expected.txt | 6 +++--- 6 files changed, 18 insertions(+), 18 deletions(-) diff --git a/third_party/blink/renderer/core/streams/transform_stream.idl b/third_party/blink/renderer/core/streams/transform_stream.idl index ad26ffa35da509..bb3e38648c2682 100644 --- 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 index 02fc1802f9c07f..68f466afe32cf7 100644 --- 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 index 542c62cc839e5d..89b1069949c7fe 100644 --- 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; 128 PASS, 96 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 index 542c62cc839e5d..89b1069949c7fe 100644 --- 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; 128 PASS, 96 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 index 542c62cc839e5d..89b1069949c7fe 100644 --- 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; 128 PASS, 96 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 index 542c62cc839e5d..89b1069949c7fe 100644 --- 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; 128 PASS, 96 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