- Nov 23, 2022
-
-
Tom Ritter authored
Differential Revision: https://phabricator.services.mozilla.com/D162756
-
Tom Ritter authored
Differential Revision: https://phabricator.services.mozilla.com/D162858
-
Emilio Cobos Álvarez authored
The early blank first-paint stuff[1] manually sets these attributes in lowercase in an about:blank window, marvelous o.O [1]: https://searchfox.org/mozilla-central/rev/650c19c96529eb28d081062c1ca274bc50ef3635/browser/components/BrowserGlue.jsm#1507-1523 Differential Revision: https://phabricator.services.mozilla.com/D162745
-
Ziran Sun authored
Differential Revision: https://phabricator.services.mozilla.com/D162853
-
Noemi Erli authored
Backed out 16 changesets (bug 1792775) for causing build bustage in TestDateIntervalFormat.cpp CLOSED TREE Backed out changeset 240681aef215 (bug 1792775) Backed out changeset e15aea6bb46b (bug 1792775) Backed out changeset ee9cd4c8d5cd (bug 1792775) Backed out changeset 0be61a39756f (bug 1792775) Backed out changeset 7db4fb376f48 (bug 1792775) Backed out changeset 7009b35a05a0 (bug 1792775) Backed out changeset 5abc2704004e (bug 1792775) Backed out changeset 6d76fe50cf35 (bug 1792775) Backed out changeset d6300cbbf628 (bug 1792775) Backed out changeset 8f5e0e60d195 (bug 1792775) Backed out changeset 0b9cf0354dc0 (bug 1792775) Backed out changeset 7136c85641b7 (bug 1792775) Backed out changeset 486778427d64 (bug 1792775) Backed out changeset 7acee89c2699 (bug 1792775) Backed out changeset 654c5f659fed (bug 1792775) Backed out changeset a60b1bc33f91 (bug 1792775)
-
Andi-Bogdan Postelnicu authored
Ported from clangd, this still can be improved over time, but it can be landed. This was based on the work from https://bit.ly/3TkV2N1 * The utility makes the assumption that all header are self contained! * It only checkes `Decls` from the main translation file, where SourceLocarion is the passed `cpp` file. * It builds a list with all of the includes from the translation unit. * It matches all of the `Decls` from the main translation units with definitions from the included header files and builds a list with used header files. * All of the includes that are not part of the matched used header files are considered to be unused. Of course this is correct if the first assumption if followed by the coding guide, where all of the header are self contained. Since the mozilla code base doesn't follow this approach false positives might appear where the is the following situation: FOO.cpp #include <A> #Include <B> If header `A` defines a symbol that is used by header `B` and `B` doesn't include `A` nor it has symbols defined that are used by `FOO.cpp` then `B` it will be marked as potentially to be removed by the tool. This is the limitation determined by header that are not self contained. The limitation presented above can be fixed in the future with extra work, but it's very time expensive during the runtime of the checker. Differential Revision: https://phabricator.services.mozilla.com/D161583
-
Jon Coppeard authored
Finally, this adds the trace method and uses it for Map/SetObject. OrderedHashTable calls a method on Ops to split the entry into key/key+value as appropriate. Depends on D162832 Differential Revision: https://phabricator.services.mozilla.com/D162833
-
Jon Coppeard authored
Depends on D162831 Differential Revision: https://phabricator.services.mozilla.com/D162832
-
Jon Coppeard authored
This removes the prebarrier from HashableValue and changes its uses to PreBarriered<HashableValue>. This lets us get rid of weird tricks to avoid the barrier in TraceKey. I also tidied up, removing the extra trace method and making the different equality tests clearer and more explicit. Depends on D162830 Differential Revision: https://phabricator.services.mozilla.com/D162831
-
Jon Coppeard authored
Bug 1802106 - Part 1: Update OrderedHashMap/Set interface to use Lookup type rather than Key where possible r=jandem Standard HashMap/Set tables support lookup by a type that's separate from the key and we can do that here too by using the Lookup type appopriately. This is useful for the folowing patch where we will split the prebarrier out of HashableValue enabling us to have Key as PreBarriered<HashableValue> and then lookup with just a HashableValue. Differential Revision: https://phabricator.services.mozilla.com/D162830
-
William Durand authored
Differential Revision: https://phabricator.services.mozilla.com/D162765
-
Jon Coppeard authored
This refactors the code to move the update into the GC. Depends on D162820 Differential Revision: https://phabricator.services.mozilla.com/D162821
-
Jon Coppeard authored
Currently we discard JIT code for RegExpShared by clearing fields during marking. The patch changes this to do this at the start of GC. This is a slight pessimisation as it happens for all RegExpShared objects including unreachable ones. I don't expect this to be noticable in practice. Differential Revision: https://phabricator.services.mozilla.com/D162820
-
Gabriele Svelto authored
Differential Revision: https://phabricator.services.mozilla.com/D162538
-
Marco Castelluccio authored
Differential Revision: https://phabricator.services.mozilla.com/D162413
-
Noemi Erli authored
-
Marco Castelluccio authored
# ignore-this-changeset Differential Revision: https://phabricator.services.mozilla.com/D162663
-
Nicolas Chevobbe authored
Bug 1801337 - Fix uninstall on sitepermission AddonCard created on the AOM onInstalled event. r=rpl. There are 2 folds to this patch. First, the `SitePermsAddonInstalling` was calling its parent constructor (`SitePermsAddonWrapper`), without the expected permissions parameter. This would then lead to the `uninstall` method to not do anything as there were no permissions. This is fixed by passing the permission for which we want to install the addon. Second, if multiple addons were installed for the same origin (e.g. "midi" and "midi-sysex") while about:addons was open, removing the addon would only revoke the "first" permission, as `SitePermsAddonInstalling` only holds one permission. To fix this, we define an `SitePermsAddonInstalling#uninstall` method that checks if we registered a `SitePermsAddonWrapper` instance for this origin, and in such case, uninstall the addon from this instance instead of the `SitePermsAddonInstalling`, one, as the `SitePermsAddonWrapper` instance has the whole set of permissions granted for a given origin. Test cases are added to ensure we don't regress this. Differential Revision: https://phabricator.services.mozilla.com/D162431
-
Marco Castelluccio authored
# ignore-this-changeset Differential Revision: https://phabricator.services.mozilla.com/D162645
-
Marco Castelluccio authored
# ignore-this-changeset Differential Revision: https://phabricator.services.mozilla.com/D162665
-
Andrew Osmond authored
Differential Revision: https://phabricator.services.mozilla.com/D162751
-
Marco Castelluccio authored
# ignore-this-changeset Differential Revision: https://phabricator.services.mozilla.com/D162662
-
Marco Castelluccio authored
# ignore-this-changeset Differential Revision: https://phabricator.services.mozilla.com/D162642
-
Jan Varga authored
Bug 1801281 - Make sure that FileSystemManagerParent::mDataManager is set to null after subprotocol destruction; r=smaug Calling AllowToClose in OnChannelClose/OnChannelError is tricky because we can only do it either before or after calling PFileSystemManagerParent::OnChannelClose/OnChannelError. When we call it before, it can happen that mDataManager is already null when a subprotocol tries to use in its ActorDestroy (destruction of subprotocols is triggered by calling PFileSystemManagerParent::OnChannelClose/OnChannelError). Calling AllowToClose after calling PFileSystemManagerParent::OnChannelClose/OnChannelError makes no sense because the object is deleted at that point. The solution is to override ActorDestroy instead of overriding OnChannelClose/OnChannelError. When ActorDestroy is called all subprotocols are already destroyed but the object itself hasn't been deleted yet. Differential Revision: https://phabricator.services.mozilla.com/D162825
-
Marco Castelluccio authored
# ignore-this-changeset Differential Revision: https://phabricator.services.mozilla.com/D162653
-
Marco Castelluccio authored
# ignore-this-changeset Differential Revision: https://phabricator.services.mozilla.com/D162650
-
Mark Banner authored
Manual fixing of tests. Differential Revision: https://phabricator.services.mozilla.com/D162696
-
Noemi Erli authored
Backed out changeset 41bebff09700 (bug 1800033) for causing failures in browser_utility_audioDecodeCrash.js CLOSED TREE
-
André Bargull authored
Temporarily disable some test262 tests which no longer produce the expected results when using CLDR 42 resp. Unicode 15. We can re-enable those tests when we next update test262 and the following PRs are all merged upstream: - https://github.com/tc39/test262/pull/3662 - https://github.com/tc39/test262/pull/3663 - https://github.com/tc39/test262/pull/3676 Differential Revision: https://phabricator.services.mozilla.com/D160192
-
André Bargull authored
See bug 1315397 for why a clobber is necessary. Differential Revision: https://phabricator.services.mozilla.com/D160191
-
André Bargull authored
Require at least ICU 72, because we're using ICU APIs which are only stable in ICU 72. Differential Revision: https://phabricator.services.mozilla.com/D160190
-
André Bargull authored
Bug 1792775 - Part 11.1: Update expected test results in non-SpiderMonkey files. r=platform-i18n-reviewers,gregtatum Differential Revision: https://phabricator.services.mozilla.com/D162707
-
André Bargull authored
CLDR 42 replaced plain ASCII space U+0020 with U+202F when formatting dates. This change requires to update the expected results for more tests than usual. Differential Revision: https://phabricator.services.mozilla.com/D160189
-
André Bargull authored
CLDR 42 replaced plain ASCII space U+0020 with U+202F when formatting dates. For backward-compatibility reasons we want to ensure that `Date.parse(date.toLocaleString())` works when the default locale is English, therefore we have to normalise U+202F to U+0020. Differential Revision: https://phabricator.services.mozilla.com/D160188
-
André Bargull authored
Bug 1792775 - Part 9: Handle new script codes in GetCommonFallbackFonts. r=platform-i18n-reviewers,jfkthame Use the fallback path for both new script codes. Differential Revision: https://phabricator.services.mozilla.com/D160187
-
André Bargull authored
Bug 1792775 - Part 8: Remove no longer needed ICU draft API guards. r=platform-i18n-reviewers,jfkthame `UDAT_STANDALONE_NARROW_QUARTERS` has been promoted from draft to stable. Differential Revision: https://phabricator.services.mozilla.com/D160186
-
André Bargull authored
Update Gecko to Unicode 15 by running `genUnicodePropertyData.pl` and `genSpecialCasingData.pl`. Differential Revision: https://phabricator.services.mozilla.com/D160185
-
André Bargull authored
Update SpiderMonkey to Unicode 15 by running `./make_unicode.py --version=UNIDATA`. Differential Revision: https://phabricator.services.mozilla.com/D160184
-
André Bargull authored
Update numbering systems by running `./make_intl_data.py numbering`. Differential Revision: https://phabricator.services.mozilla.com/D160183
-
André Bargull authored
Update measurement units by running `./make_intl_data.py units`. Differential Revision: https://phabricator.services.mozilla.com/D160182
-