Marco Simonelli authored 2 years ago and Pier Angelo Vendrame committed 1 year ago

- fixes required to build third_party/sipcc

Marco Simonelli authored 2 years ago and Pier Angelo Vendrame committed 1 year ago

- fixes required to build third_party/libwebrtc

Bug 1857742 - patch 1 - Vendor the oxilangtag crate into third_party/rust. r=supply-chain-reviewers,dholbert a=RyanVM

Differential Revision: https://phabricator.services.mozilla.com/D195822

Differential Revision: https://phabricator.services.mozilla.com/D187876

Differential Revision: https://phabricator.services.mozilla.com/D187872

Bug 1841585 - Generate webrtc moz.build snippets for mipsel and mips64el (moz.build refresh). r=webrtc-reviewers,mjf a=RyanVM

Bug 1841577 - Don't set WEBRTC_ENABLE_AVX2 on platforms that don't have AVX2 (moz.build refresh). r=mjf,webrtc-reviewers a=RyanVM

Bug 1841577 - Don't set WEBRTC_ENABLE_AVX2 on platforms that don't have AVX2. r=mjf,webrtc-reviewers a=RyanVM

Differential Revision: https://phabricator.services.mozilla.com/D182695

Bug 1830461 - Allow to build webrtc on Linux/riscv64 - moz.build file updates. r=webrtc-reviewers,ng a=RyanVM

Depends on D176752

Differential Revision: https://phabricator.services.mozilla.com/D178854

Differential Revision: https://phabricator.services.mozilla.com/D179727

Differential Revision: https://phabricator.services.mozilla.com/D179727

There is a corner case that triggers an assert in wpf-gpu-raster if an empty sub-path
starts off a path. Just avoid the problem by eliding the sub-path, but at least handle
close correctly as per the Canvas2D closePath spec:

"The closePath() method, when invoked, must do nothing if the object's path has no subpaths.
Otherwise, it must mark the last subpath as closed, create a new subpath whose first point
is the same as the previous subpath's first point, and finally add this new subpath to the path."

Differential Revision: https://phabricator.services.mozilla.com/D181543

Bug 1839070 - Emit a start point if close is called immediately after move_to. r=jrmuizel, a=dmeehan


wpf-gpu-raster would blindly modify the last point type in a path when call was closed, even
if move_to preceded it directly and outputted no actual point type. In this case, we need to
ensure that we emit a start point if close is called immediately after move_to.

Differential Revision: https://phabricator.services.mozilla.com/D181358

Bug 1839070 - Emit a start point if close is called immediately after move_to. r=jrmuizel, a=dmeehan

wpf-gpu-raster would blindly modify the last point type in a path when call was closed, even
if move_to preceded it directly and outputted no actual point type. In this case, we need to
ensure that we emit a start point if close is called immediately after move_to.

Differential Revision: https://phabricator.services.mozilla.com/D181358

Differential Revision: https://phabricator.services.mozilla.com/D179727

Bug 1836670 - Use NonNull in cssparser::CowRcStr. r=layout-reviewers,glandium,dlrobertson, a=dmeehan

This prevents the compiler from misoptimizing this code due to undefined
behavior.

Upstream PR: https://github.com/servo/rust-cssparser/pull/345

Differential Revision: https://phabricator.services.mozilla.com/D179921

Differential Revision: https://phabricator.services.mozilla.com/D179727

Differential Revision: https://phabricator.services.mozilla.com/D179687

Differential Revision: https://phabricator.services.mozilla.com/D179754

It's only used by glean in code that is not actually used. It allows to
remove the only use of dashmap, which is a rather complex crate.

Differential Revision: https://phabricator.services.mozilla.com/D179747

Bug 1836402 - Disable clap default features in uniffi-bindgen-gecko-js. r=bdk,supply-chain-reviewers

The tool is not directly user visible, and it's not super useful for
features like color to be enabled.

Differential Revision: https://phabricator.services.mozilla.com/D179742

This patch allows the use of a JS shell binary, compiled to
`wasm32-wasi` (WASI ABI on WebAssembly), to make use of the Wizer
snapshotting tool to embed JavaScript source with SpiderMonkey to run in
a standalone Wasm module.

If enabled with `--enable-js-shell-wizer`, the shell no longer provides
an interactive REPL; instead, when invoked during the "initialization"
phase of Wizer, it reads JS source on standard input, then
saves JS state (the global, the context, etc) for later. When the
execution is snapshotted and the snapshot is later executed as an
ordinary WASI command, the resume entry-point invokes a JS function
named `main` in the global scope, if any.

An example of how to use this follows:

```
$ echo "function main() { print('hello world'); }" | \
  wizer --allow-wasi -r _start=wizer.resume obj-release/dist/bin/js \
  -o snapshotted.wasm

$ wasmtime snapshotted.wasm
hello world
$
```

Differential Revision: https://phabricator.services.mozilla.com/D176979

And add some bindings support to it, which will be used by the next
patch.

Differential Revision: https://phabricator.services.mozilla.com/D178768

Depends on D177827

Differential Revision: https://phabricator.services.mozilla.com/D177829

Bug 18179974 - Update mp4parse-rust to revision cf8b0e04. r=media-playback-reviewers,glandium,kinetik

Differential Revision: https://phabricator.services.mozilla.com/D176050

Differential Revision: https://phabricator.services.mozilla.com/D176046

As far as I've understood, compare-locales was the last user of this.

Depends on D178299

Differential Revision: https://phabricator.services.mozilla.com/D178300

Bug 18125390 - Update to compare-locales 9.0.1, fluent.migrate 0.12.0, and fluent.syntax 0.19.0. r=flod,glandium

Differential Revision: https://phabricator.services.mozilla.com/D178299

This restores libwebrtc's DesktopCapturer to their upstream state as far as
the tab capturer is concerned.

Differential Revision: https://phabricator.services.mozilla.com/D174290

Differential Revision: https://phabricator.services.mozilla.com/D177919

Upstream commit: https://webrtc.googlesource.com/src/+/301e546a689020320f919a660591759e993ef051
    Remove SequenceCheckerImpl::valid_system_queue_

    As pointed out in issue webrtc:15146 this Mac/iOS specific variable,
    makes the SequenceChecker behave incorrectly on those platforms.

    The variable was introduced in a CL that merged the previous checker
    classes, ThreadChecker and SequencedTaskChecker, but curiously neither
    one of them had such a variable. So I'm not exactly sure what problem
    was being solved. Hence I'm wondering if we actually need it.

    Reference: https://webrtc-review.googlesource.com/c/src/+/129721

    Bug: webrtc:15146
    Change-Id: Ia7a9eb17b993c4f8a1e8204c658bf0b3dbdaa1e0
    Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/304401


Reviewed-by: Peter Hanspers <peterhanspers@webrtc.org>
    Commit-Queue: Tomas Gunnarsson <tommi@webrtc.org>
    Cr-Commit-Position: refs/heads/main@{#40019}

Differential Revision: https://phabricator.services.mozilla.com/D177875

Upstream commit: https://webrtc.googlesource.com/src/+/3da04a93cd18dc7b65c6756910cc8a9cbf20fb8c
    Allow SequenceChecker to be initialized detached.

    The motivation for this is to not have to implement this pattern:

    foo.h:

    class Foo {
     public:
      Foo();
     private:
      SequenceChecker checker_;
    };

    foo.cc:

    Foo::Foo() {
      checker_.Detach();
    }

    And instead be able to do this inline in the .h file:

    class Foo {
     public:
      Foo();
     private:
      SequenceChecker checker_{SequenceChecker::kDetached};
    };

    Bug: none
    Change-Id: Idd7ca82d15c2f77f3aaccf26f1943a49f4b40661
    Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/298445


Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
    Commit-Queue: Tomas Gunnarsson <tommi@webrtc.org>
    Cr-Commit-Position: refs/heads/main@{#39616}

Differential Revision: https://phabricator.services.mozilla.com/D177874

Bug 1815186 - Add ColorTo to InspectorUtils to convert color to other color spaces. r=emilio,layout-reviewers,supply-chain-reviewers

This will allow calling `InspectorUtils.ColorTo("red", "lab")` and
receive `{color: "lab(54.3 80.8 69.9)", components: [54.3, 80.8, 69.9, 1.0], adjusted: false}`.

The adjusted value is always "false", because gamut checking and mapping
is not supported at the moment. See:
https://bugzilla.mozilla.org/show_bug.cgi?id=1823363

Differential Revision: https://phabricator.services.mozilla.com/D169942

Differential Revision: https://phabricator.services.mozilla.com/D178591