searching for reviewer(Yoric)
abb551350a788459b414369130102cfd01f0c1ad: Bug 1659828 - Get rid of `virtualMemorySize` in ProcInfo r=Yoric
Nicklas Boman <smurfd@gmail.com> - Wed, 02 Sep 2020 12:56:46 +0000 - rev 547453
Push 37750 by nerli@mozilla.com at Wed, 02 Sep 2020 21:57:21 +0000
Bug 1659828 - Get rid of `virtualMemorySize` in ProcInfo r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D89013
f173105b595bb495372cbcb5c1f219d6885e8b42: Bug 1652013 - Reorder columns in about:processes. r=Yoric,fluent-reviewers,flod,florian
Sonia Singla <soniasingla.1812@gmail.com> - Sat, 22 Aug 2020 18:01:33 +0000 - rev 545729
Push 37722 by ccoroiu@mozilla.com at Sun, 23 Aug 2020 09:31:12 +0000
Bug 1652013 - Reorder columns in about:processes. r=Yoric,fluent-reviewers,flod,florian Differential Revision: https://phabricator.services.mozilla.com/D86348
891217d9ac8037fd815a20d37c1515fc9c1fd956: Bug 1659904 - Define UnixFile.open as a variadic function. r=Yoric
Mike Hommey <mh+mozilla@glandium.org> - Sat, 22 Aug 2020 21:51:04 +0000 - rev 545728
Push 37722 by ccoroiu@mozilla.com at Sun, 23 Aug 2020 09:31:12 +0000
Bug 1659904 - Define UnixFile.open as a variadic function. r=Yoric That is how it is actually defined in fcntl.h, and thus how C code calls it. It so happens that in many cases, it just works out fine because variadic arguments end up at the same place as if they weren't variadic in registers/stack, but that's not actually true on all platforms. Most notably, that's not true on Apple Silicon. Differential Revision: https://phabricator.services.mozilla.com/D87558
b73b2f9e7f1d530a01c5db138041bc48b8b6a823: Bug 1652013 - Reorder columns in about:processes. r=Yoric,fluent-reviewers,flod,florian
Sonia Singla <soniasingla.1812@gmail.com> - Sat, 22 Aug 2020 16:13:19 +0000 - rev 545720
Push 37721 by apavel@mozilla.com at Sat, 22 Aug 2020 21:38:08 +0000
Bug 1652013 - Reorder columns in about:processes. r=Yoric,fluent-reviewers,flod,florian Differential Revision: https://phabricator.services.mozilla.com/D86348
2cb7fc096723dd93c5fd00574b4e53e68c6a8a28: Bug 1659904 - Define UnixFile.open as a variadic function. r=Yoric
Mike Hommey <mh+mozilla@glandium.org> - Wed, 19 Aug 2020 08:16:42 +0000 - rev 545693
Push 37719 by btara@mozilla.com at Sat, 22 Aug 2020 09:46:59 +0000
Bug 1659904 - Define UnixFile.open as a variadic function. r=Yoric That is how it is actually defined in fcntl.h, and thus how C code calls it. It so happens that in many cases, it just works out fine because variadic arguments end up at the same place as if they weren't variadic in registers/stack, but that's not actually true on all platforms. Most notably, that's not true on Apple Silicon. Differential Revision: https://phabricator.services.mozilla.com/D87558
c1527aeaa9d15aed092f97de1c683fd9a768837f: Bug 1652013 - Reorder columns in about:processes. r=Yoric,fluent-reviewers,flod,florian
Sonia Singla <soniasingla.1812@gmail.com> - Thu, 20 Aug 2020 11:18:11 +0000 - rev 545482
Push 37715 by apavel@mozilla.com at Thu, 20 Aug 2020 21:21:07 +0000
Bug 1652013 - Reorder columns in about:processes. r=Yoric,fluent-reviewers,flod,florian Differential Revision: https://phabricator.services.mozilla.com/D86348
04321853216c6511f573a10893bb3187197a194a: Bug 1659077 - Don't use -suffixed symbols on arm64 mac. r=Yoric
Mike Hommey <mh+mozilla@glandium.org> - Fri, 14 Aug 2020 11:30:46 +0000 - rev 544657
Push 37700 by dluca@mozilla.com at Sat, 15 Aug 2020 09:31:17 +0000
Bug 1659077 - Don't use -suffixed symbols on arm64 mac. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D87060
f56158e0fd578f54c4a7307eda87c1c350032136: Bug 1658621 - Remove JS_WEB_PARSER_COMPILE_LAZY_AFTER_MS/JS_PRIVILEGED_PARSER_COMPILE_LAZY_AFTER_MS probes. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Tue, 11 Aug 2020 20:19:51 +0000 - rev 544312
Push 37692 by cbrindusan@mozilla.com at Wed, 12 Aug 2020 03:44:18 +0000
Bug 1658621 - Remove JS_WEB_PARSER_COMPILE_LAZY_AFTER_MS/JS_PRIVILEGED_PARSER_COMPILE_LAZY_AFTER_MS probes. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D86751
e1013eb60a84d0cc2d931fbc0bb8b27684c22db5: Bug 1649686 - Test whether webCOOP+COEP is a prefix before testing whether web is a prefix;r=Yoric
Bruno Bartolomasi <brunobrt@hotmail.com> - Wed, 22 Jul 2020 09:38:49 +0000 - rev 541560
Push 37627 by ncsoregi@mozilla.com at Wed, 22 Jul 2020 15:50:17 +0000
Bug 1649686 - Test whether webCOOP+COEP is a prefix before testing whether web is a prefix;r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D83386
47bf6da3e0081e80e9638a34fcd771560ec51b41: Bug 1648378 - Removed Virtual Memory from about:processes r=Yoric,fluent-reviewers,flod
Sonia Singla <soniasingla.1812@gmail.com> - Tue, 21 Jul 2020 20:47:07 +0000 - rev 541490
Push 37625 by csabou@mozilla.com at Wed, 22 Jul 2020 04:32:41 +0000
Bug 1648378 - Removed Virtual Memory from about:processes r=Yoric,fluent-reviewers,flod Differential Revision: https://phabricator.services.mozilla.com/D84093
6ae542c7cd014b8f1f5e8c2c55630f0d72a154b7: Bug 1653177: Fix minGW build failure due to use of missing identifiers r=Yoric,barret
Keefer Rourke <krourke@mozilla.com> - Thu, 16 Jul 2020 21:46:38 +0000 - rev 540816
Push 37609 by rmaries@mozilla.com at Fri, 17 Jul 2020 03:27:56 +0000
Bug 1653177: Fix minGW build failure due to use of missing identifiers r=Yoric,barret This patch attempts to fix the permanent win mingw build failure caused by use of the undeclared identifiers 'S_IFLNK' and 'S_IFSOCK' in dom/system/OSFileConstants.cpp These identifiers are only ever declared on Unix systems (Linux, macOS, Android, etc.), so the `ifdef` guard around the use of these constants has been updated appropriately. Differential Revision: https://phabricator.services.mozilla.com/D83844
e82376ae2519850aad415741e1862124dd46d13a: Bug 1633379 - Part 2: Add support for in-process JSWindowActors, r=kmag,Yoric
Nika Layzell <nika@thelayzells.com> - Thu, 25 Jun 2020 20:35:18 +0000 - rev 537516
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1633379 - Part 2: Add support for in-process JSWindowActors, r=kmag,Yoric This switches the `nsIContent{Parent,Child}` interface to be `nsIDOMProcess{Parent,Child}`, and also implements it on `InProcess{Parent,Child}`, along with the `ProcessActor` interface. Differential Revision: https://phabricator.services.mozilla.com/D80582
96ddb1b29a80f938b033724ae85781aa82c27ed5: Bug 1633379 - Part 1: Move PInProcess into dom/ipc, r=kmag,Yoric
Nika Layzell <nika@thelayzells.com> - Thu, 25 Jun 2020 17:50:51 +0000 - rev 537515
Push 37543 by abutkovits@mozilla.com at Fri, 26 Jun 2020 03:24:35 +0000
Bug 1633379 - Part 1: Move PInProcess into dom/ipc, r=kmag,Yoric This moves it near the cross-process `PContent` actor, and makes it more clear that this actor is only intended to be used for DOM things. Differential Revision: https://phabricator.services.mozilla.com/D80581
a26037f3225b5167f5431643a6a3d35ea4ad24ff: Bug 1633379 - Part 2: Add support for in-process JSWindowActors, r=kmag,Yoric
Nika Layzell <nika@thelayzells.com> - Thu, 25 Jun 2020 16:28:11 +0000 - rev 537438
Push 37542 by abutkovits@mozilla.com at Thu, 25 Jun 2020 21:45:10 +0000
Bug 1633379 - Part 2: Add support for in-process JSWindowActors, r=kmag,Yoric This switches the `nsIContent{Parent,Child}` interface to be `nsIDOMProcess{Parent,Child}`, and also implements it on `InProcess{Parent,Child}`, along with the `ProcessActor` interface. Differential Revision: https://phabricator.services.mozilla.com/D80582
efef0b59bcd814c0d94325f43a04909a86fd297b: Bug 1633379 - Part 1: Move PInProcess into dom/ipc, r=kmag,Yoric
Nika Layzell <nika@thelayzells.com> - Thu, 25 Jun 2020 15:49:15 +0000 - rev 537437
Push 37542 by abutkovits@mozilla.com at Thu, 25 Jun 2020 21:45:10 +0000
Bug 1633379 - Part 1: Move PInProcess into dom/ipc, r=kmag,Yoric This moves it near the cross-process `PContent` actor, and makes it more clear that this actor is only intended to be used for DOM things. Differential Revision: https://phabricator.services.mozilla.com/D80581
ce6192a7bdb5f0d8f5ace2c68253c876390a34f3: Bug 1640131 - When getting the error string for OSError, handle non-utf-8 character sets. r=Yoric
Mark Banner <standard8@mozilla.com> - Mon, 15 Jun 2020 10:58:32 +0000 - rev 535646
Push 37508 by btara@mozilla.com at Mon, 15 Jun 2020 21:48:38 +0000
Bug 1640131 - When getting the error string for OSError, handle non-utf-8 character sets. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D79664
f787820067a1d43ca185d3bf0d11652e1f45519d: Bug 1643175 - update documentation about JSWindowActor's observers list, r=Yoric
Gijs Kruitbosch <gijskruitbosch@gmail.com> - Thu, 11 Jun 2020 09:42:32 +0000 - rev 535056
Push 37498 by apavel@mozilla.com at Fri, 12 Jun 2020 03:05:25 +0000
Bug 1643175 - update documentation about JSWindowActor's observers list, r=Yoric DONTBUILD Differential Revision: https://phabricator.services.mozilla.com/D78566
02c0de0edc3cfd86e0a96930ed6ed0df0f7d8069: Bug 1638333 - BrowserTestUtilsChild.jsm: handleEvent - don't assume the load event target is from a document. r=Yoric
Magnus Melin <mkmelin+mozilla@iki.fi> - Wed, 27 May 2020 11:06:12 +0000 - rev 532356
Push 37455 by apavel@mozilla.com at Wed, 27 May 2020 21:33:21 +0000
Bug 1638333 - BrowserTestUtilsChild.jsm: handleEvent - don't assume the load event target is from a document. r=Yoric In the case of a <body> location won't be set. Differential Revision: https://phabricator.services.mozilla.com/D76137
25f2f567dad6e2ef468be49e8eb76c9c0c6f783e: Bug 1636204 - Return size in bytes instead of page count for RSS in ProcInfo. r=Yoric
Ian Moody <moz-ian@perix.co.uk> - Mon, 11 May 2020 16:09:20 +0000 - rev 529178
Push 37405 by csabou@mozilla.com at Tue, 12 May 2020 04:04:10 +0000
Bug 1636204 - Return size in bytes instead of page count for RSS in ProcInfo. r=Yoric As noted in the comment /proc/[pid]/stat is returning the number of pages, but this value is being used elsewhere as a bytes figure, so we need to multiply by the page size to get that. Differential Revision: https://phabricator.services.mozilla.com/D74350
c7713eb6813c249de6971ecdb116b51ed174d45b: Bug 1636204 - Use correct indexes for parsing /proc/[pid]/stat. r=Yoric
Ian Moody <moz-ian@perix.co.uk> - Mon, 11 May 2020 12:04:39 +0000 - rev 529177
Push 37405 by csabou@mozilla.com at Tue, 12 May 2020 04:04:10 +0000
Bug 1636204 - Use correct indexes for parsing /proc/[pid]/stat. r=Yoric All the indexes were off by two, so the values were for completely different measurements, and thus nonsense. Resident Set Size still doesn't seem to be handled correctly, perhaps because it's being treated as a bytes count rather than the page count it is somewhere, but with this change the Virtual Memory figures match what system monitor displays. Differential Revision: https://phabricator.services.mozilla.com/D74349
6fc9d1fa3e7bfd300599a328dace3aeba01528d2: Bug 1636203 - Include pid in ProcInfo on Linux. r=Yoric
Ian Moody <moz-ian@perix.co.uk> - Mon, 11 May 2020 09:58:40 +0000 - rev 529176
Push 37405 by csabou@mozilla.com at Tue, 12 May 2020 04:04:10 +0000
Bug 1636203 - Include pid in ProcInfo on Linux. r=Yoric The pid wasn't being included in the ProcInfo returned from GetProcInfo unlike on macOS, so all child processes were ending up with the default pid of 0 Differential Revision: https://phabricator.services.mozilla.com/D74348
c1a36d34c2cc3b8d42e615a4891335123af48007: Bug 1017237 - Ensure that Sqlite's AsyncShutdown phase displays the name of opened databases in case of crash r=Yoric
aarushivij <aarushivij@gmail.com> - Mon, 27 Apr 2020 15:29:31 +0000 - rev 526251
Push 37353 by shindli@mozilla.com at Mon, 27 Apr 2020 21:46:18 +0000
Bug 1017237 - Ensure that Sqlite's AsyncShutdown phase displays the name of opened databases in case of crash r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D71398
31ed49b0ce7746cbf275300aa98f44732fb29f16: Bug 1628784 - Pass TOPSRCDIR environment variable to jsapi-tests, in order to read test files. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Mon, 27 Apr 2020 11:37:13 +0000 - rev 526193
Push 37353 by shindli@mozilla.com at Mon, 27 Apr 2020 21:46:18 +0000
Bug 1628784 - Pass TOPSRCDIR environment variable to jsapi-tests, in order to read test files. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D71255
717435e372817457a2dd2befa5a8bf1a461283e4: Bug 1624622 - Allow for promise await inside promise worker r=Yoric
Scott <scott.downe@gmail.com> - Tue, 24 Mar 2020 15:50:32 +0000 - rev 520239
Push 37246 by opoprus@mozilla.com at Wed, 25 Mar 2020 03:40:33 +0000
Bug 1624622 - Allow for promise await inside promise worker r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D68043
db04841c16dcd58e0f7d09a47d48f24da3b31a2f: Bug 1620630 - The OS.File worker should record markers indicating which operation it is performing on which file, r=Yoric.
Florian Quèze <florian@queze.net> - Wed, 11 Mar 2020 14:22:06 +0000 - rev 518218
Push 37206 by aciure@mozilla.com at Thu, 12 Mar 2020 03:57:49 +0000
Bug 1620630 - The OS.File worker should record markers indicating which operation it is performing on which file, r=Yoric. Differential Revision: https://phabricator.services.mozilla.com/D65747
6aaa835ae0eb2f4d7b0d89d243f702df2143c690: Bug 1620630 - The OS.File worker should record markers indicating which operation it is performing on which file, r=Yoric.
Florian Quèze <florian@queze.net> - Tue, 10 Mar 2020 17:04:10 +0000 - rev 517887
Push 37203 by ccoroiu@mozilla.com at Wed, 11 Mar 2020 04:11:49 +0000
Bug 1620630 - The OS.File worker should record markers indicating which operation it is performing on which file, r=Yoric. Differential Revision: https://phabricator.services.mozilla.com/D65747
bb07f7e084a1b8066ab1c5700dae3e8f1d7c1494: Bug 1620613 - improve checks for the webrender + fission infobar, r=Yoric
Gijs Kruitbosch <gijskruitbosch@gmail.com> - Tue, 10 Mar 2020 09:08:09 +0000 - rev 517745
Push 37201 by csabou@mozilla.com at Tue, 10 Mar 2020 17:04:55 +0000
Bug 1620613 - improve checks for the webrender + fission infobar, r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D66123
c1b10a848d6431f8916a59f4f32f8c9d97169355: Bug 1620630 - The OS.File worker should record markers indicating which operation it is performing on which file, r=Yoric.
Florian Quèze <florian@queze.net> - Mon, 09 Mar 2020 14:00:04 +0000 - rev 517544
Push 37198 by opoprus@mozilla.com at Mon, 09 Mar 2020 21:52:54 +0000
Bug 1620630 - The OS.File worker should record markers indicating which operation it is performing on which file, r=Yoric. Differential Revision: https://phabricator.services.mozilla.com/D65747
326c5a753027f27f1540325b072a1937983b1f12: Bug 1619068 - Do not use `using js::frontend::*` in js/src/shell/js.cpp. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Sun, 01 Mar 2020 11:40:28 +0000 - rev 516282
Push 37171 by ncsoregi@mozilla.com at Sun, 01 Mar 2020 21:39:24 +0000
Bug 1619068 - Do not use `using js::frontend::*` in js/src/shell/js.cpp. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D64865
e7fe3c8d1e746d301e45d058a4323343486ab739: Bug 1616221 - Align some BinAST classes to pointer. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Tue, 18 Feb 2020 17:58:11 +0000 - rev 514716
Push 37141 by malexandru@mozilla.com at Thu, 20 Feb 2020 16:56:52 +0000
Bug 1616221 - Align some BinAST classes to pointer. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D63180
66fe6c785157c77c142d82424f4607394cf652e9: Bug 1609110 - Follow up fixes on async process creation. r=Yoric,asuth
Luca Greco <lgreco@mozilla.com> - Fri, 17 Jan 2020 21:00:44 +0000 - rev 510629
Push 37028 by btara@mozilla.com at Sat, 18 Jan 2020 04:04:50 +0000
Bug 1609110 - Follow up fixes on async process creation. r=Yoric,asuth Differential Revision: https://phabricator.services.mozilla.com/D59854
c883249bab17b93315a682ce490df4264e63959b: Bug 1597152 - Part 2: Support context parser in lazy parsing. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:19:44 +0000 - rev 503377
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1597152 - Part 2: Support context parser in lazy parsing. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D53369
24087ff1dfa9c14fc5f665eab2d06a606989dc30: Bug 1597152 - Part 1: Set lazy function position after reading tree. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:19:29 +0000 - rev 503376
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1597152 - Part 1: Set lazy function position after reading tree. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D53368
da0c2ed5c0e3326277f21d7a6fc1c95382c92bd0: Bug 1597153 - Part 3: Add --binast-format parameter to JS shell. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:19:11 +0000 - rev 503375
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1597153 - Part 3: Add --binast-format parameter to JS shell. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D53367
613769723c834446d37ef4b01620a7bc19b2ebe9: Bug 1597153 - Part 2: Use context parser when JS::BinASTFormat::Context is passed. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:18:59 +0000 - rev 503374
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1597153 - Part 2: Use context parser when JS::BinASTFormat::Context is passed. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D53366
7f5fc05b10a10b3d897319ba47f0467c8ef4449d: Bug 1597153 - Part 1: Add JS::BinASTFormat enum and use it in API. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:18:41 +0000 - rev 503373
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1597153 - Part 1: Add JS::BinASTFormat enum and use it in API. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D53365
141bcc14dacfe14db8314be9fe3ccdcd83085e1d: Bug 1555387 - Part 6: Support XDR in BinASTSourceMetadataContext. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:54:41 +0000 - rev 503372
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1555387 - Part 6: Support XDR in BinASTSourceMetadataContext. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D52959
cb0d6a715e14580314f9f1ed3d63a133e2ff8db4: Bug 1555387 - Part 5: Add BinASTSourceMetadataMultipart::create variant for XDR. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:18:29 +0000 - rev 503371
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1555387 - Part 5: Add BinASTSourceMetadataMultipart::create variant for XDR. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D53949
6463b420ecb391559e635ab574d553ce96309a02: Bug 1555387 - Part 4: Use calloc in BinASTSourceMetadata{Multipart,Context}::create. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:18:27 +0000 - rev 503370
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1555387 - Part 4: Use calloc in BinASTSourceMetadata{Multipart,Context}::create. r=Yoric Differential Revision: https://phabricator.services.mozilla.com/D53948
47efbdaab5bf9bcd07c6a3114713146debb9ef0e: Bug 1555387 - Part 3: Use HuffmanDictionaryForMetadata. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:48:03 +0000 - rev 503369
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1555387 - Part 3: Use HuffmanDictionaryForMetadata. r=Yoric After reading prelude, create HuffmanDictionaryForMetadata from HuffmanDictionary and TemporaryStorage, and store it in metadata, and use it while reading tree. Differential Revision: https://phabricator.services.mozilla.com/D52957
32906323feef46004cbe87c420a56578e5190798: Bug 1555387 - Part 2: Add HuffmanDictionaryForMetadata. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:40:37 +0000 - rev 503368
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1555387 - Part 2: Add HuffmanDictionaryForMetadata. r=Yoric HuffmanDictionaryForMetadata behaves similar to HuffmanDictionary. HuffmanDictionary is used while reading prelude, to store intermediate state. HuffmanDictionaryForMetadata is used after reading prelude, to store the complete state of the dictionary, with single chunk of memory. HuffmanDictionaryForMetadata has payload to store items that is linked from tables. Differential Revision: https://phabricator.services.mozilla.com/D52956
0983a81c46c52a2681112fedb286b739c92f482b: Bug 1555387 - Part 1: Move TableIdentity out of HuffmanDictionary. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:40:02 +0000 - rev 503367
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1555387 - Part 1: Move TableIdentity out of HuffmanDictionary. r=Yoric TableIdentity is going to be used from multiple classes. Differential Revision: https://phabricator.services.mozilla.com/D52955
05b9bea262ae7af3789859785b362f5f7e9facbf: Bug 1587271 - Add temporary storage for HuffmanEntry, InternalIndex, SingleLookupHuffmanTable, TwoLookupsHuffmanTable with less alloc/free. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:40:25 +0000 - rev 503366
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1587271 - Add temporary storage for HuffmanEntry, InternalIndex, SingleLookupHuffmanTable, TwoLookupsHuffmanTable with less alloc/free. r=Yoric TemporaryStorage is used while reading prelude, to store instances of HuffmanEntry etc, and tables store mozilla::Span to point items there. Each instances are allocated inside TemporaryStorageItem, that manages a linked list of item chunks, to reduce the amount of alloc/free, to avoid unnecessary mutex lock inside allocator. After reading prelude, items inside TemporaryStorage are supposed to be moved to metadata payload, and Spans in tables are supposed to be updated to point corresponding range. Differential Revision: https://phabricator.services.mozilla.com/D52954
bfb371be51b33a6dd6aea0bc4b7ef8ac6b090937: Bug 1596259 - Add BinASTSourceMetadata{Multipart,Context}. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:39:09 +0000 - rev 503365
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1596259 - Add BinASTSourceMetadata{Multipart,Context}. r=Yoric This adds BinASTSourceMetadata{Multipart,Context} as subclasses of BinASTSourceMetadata, with different fields for storing metadata. Basically this is specialization of Variant, but given metadata has payload after the class body itself, it doesn't work well with Variant. Differential Revision: https://phabricator.services.mozilla.com/D52953
ba7c82709e9f28a58a21cbc699cd89beeb804d28: Bug 1593922 - Add HuffmanDictionary.tableIndices_ to populate HuffmanDictionary.tables_ from the beginning. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:38:51 +0000 - rev 503364
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1593922 - Add HuffmanDictionary.tableIndices_ to populate HuffmanDictionary.tables_ from the beginning. r=Yoric This is preparation for bug 1555387, to store initialized GenericHuffmanTable instances in contiguous space. Differential Revision: https://phabricator.services.mozilla.com/D52952
fd82074eab1d8d4642d816436c166267350ecf6f: Bug 1594322 - Part 4: Add better accessors to HuffmanDictionary. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:38:03 +0000 - rev 503363
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1594322 - Part 4: Add better accessors to HuffmanDictionary. r=Yoric Hide the internal of status and table creation/access from consumers. Depends on D52032 Differential Revision: https://phabricator.services.mozilla.com/D52034
f395a7657d621a2d9463efca340a4b319da59eb4: Bug 1594322 - Part 3: Use HuffmanDictionary::TableIdentity in HuffmanPreludeReader::readTable. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:33:52 +0000 - rev 503362
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1594322 - Part 3: Use HuffmanDictionary::TableIdentity in HuffmanPreludeReader::readTable. r=Yoric Another preparation for Part 4. Depends on D52031 Differential Revision: https://phabricator.services.mozilla.com/D52032
4bd936bc2bb0f186e998472d5f0b30e220faf293: Bug 1594322 - Part 2: Use HuffmanDictionary::TableIdentity in public methods. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:30:16 +0000 - rev 503361
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1594322 - Part 2: Use HuffmanDictionary::TableIdentity in public methods. r=Yoric Preparation for Part 4. Depends on D52030 Differential Revision: https://phabricator.services.mozilla.com/D52031
27430d687488aeed79b0342cd0b459233c6a675e: Bug 1594322 - Part 1: Add HuffmanDictionary::TableIdentity to unify NormalizedInterfaceAndField and BinASTList. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:30:12 +0000 - rev 503360
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1594322 - Part 1: Add HuffmanDictionary::TableIdentity to unify NormalizedInterfaceAndField and BinASTList. r=Yoric For bug 1593922, merged tables/status for field and list length. Depends on D52029 Differential Revision: https://phabricator.services.mozilla.com/D52030
297273a3b34ab363bc0617d1f6953e1094c3b343: Bug 1593923 - Store the index into BinASTSourceMetadata::getAtom in BinASTSymbol instead of JSAtom*. r=Yoric
Tooru Fujisawa <arai_a@mac.com> - Fri, 22 Nov 2019 14:30:10 +0000 - rev 503359
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1593923 - Store the index into BinASTSourceMetadata::getAtom in BinASTSymbol instead of JSAtom*. r=Yoric To store dictionary to BinASTSourceMetadata, JSAtom* needs to be stored in the single place. Depends on D52028 Differential Revision: https://phabricator.services.mozilla.com/D52029