1823619 - Implement Compression Streams

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, task)

Comment 1

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1823619 - Implement Compression Streams r=smaug

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e5ef47700fe9
Implement Compression Streams r=smaug

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/39194 for changes under testing/web-platform/tests

Comment 4

[Iulian Moraru]

Backed out for causing mochitest failures related to CompressionStream.

• Push with failures

• Failure log

• Failure log 2
  Later some build bustages started to occur: failure log for Bonf build bustages.

• Backout link

[task 2023-03-24T21:30:06.176Z] 21:30:06     INFO -  TEST-PASS | dom/tests/mochitest/general/test_interfaces.html | ClipboardEvent is exposed as an own property on 'window' but has no property descriptor in the global scope
[task 2023-03-24T21:30:06.176Z] 21:30:06     INFO -  Buffered messages finished
[task 2023-03-24T21:30:06.177Z] 21:30:06  WARNING -  TEST-UNEXPECTED-FAIL | dom/tests/mochitest/general/test_interfaces.html | If this is failing: DANGER, are you sure you want to expose the new interface DecompressionStream to all webpages as a property on 'window'? Do not make a change to this file without a  review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals)
[task 2023-03-24T21:30:06.177Z] 21:30:06     INFO -      SimpleTest.ok@SimpleTest/SimpleTest.js:421:16
[task 2023-03-24T21:30:06.177Z] 21:30:06     INFO -      runTest@dom/tests/mochitest/general/test_interfaces.js:1922:7
[task 2023-03-24T21:30:06.177Z] 21:30:06     INFO -      @dom/tests/mochitest/general/test_interfaces.js:1973:8
[task 2023-03-24T21:30:06.177Z] 21:30:06     INFO -  TEST-PASS | dom/tests/mochitest/general/test_interfaces.html | DecompressionStream is exposed as an own property on 'window' but tests false for "in" in the global scope

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 6

[Kagami Rosylight [:saschanaz] (they/them)]

Oh, sure. I hope there's some reminder bot for this.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a7b749260f11
Implement Compression Streams r=smaug

Comment 8

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1823619 - Add missing items to test_(service)worker_interfaces.js

Comment 9

[Kagami Rosylight [:saschanaz] (they/them)]

Attachment: Bug 1823619 - Include ContentChild.h in FuzzingFunctions.cpp

Comment 10

[Phabricator Automation]

Comment on attachment 9325023 [details]
Bug 1823619 - Include ContentChild.h in FuzzingFunctions.cpp

Revision D173623 was moved to bug 1824522. Setting attachment 9325023 [details] to obsolete.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7a2bc36f7838
Add missing items to test_(service)worker_interfaces.js. a=mass-failures-fix

Comment 12

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/a7b749260f11
https://hg.mozilla.org/mozilla-central/rev/7a2bc36f7838

Comment 13

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 14

[Jens Stutte [:jstutte]]

Is bug 1824565 really a regression of this one? Or is it just one way it can happen?

Comment 15

[Kagami Rosylight [:saschanaz] (they/them)]

Not a regression in a traditional meaning, it just logs the crash started when this was merged. But it's not like this patch was faulty.

Comment 16

[Brian Thomas Smith [bsmth]]

Documentation changes for these additions are being tracked in the following GitHub issue: https://github.com/mdn/content/issues/26158