ffe98e0281a1d509d56b539a1c9350f5152640fd: Tasks automatically selected. draft
hxu <hxu@mozilla.com> - Mon, 20 Sep 2021 19:58:36 +0200 - rev 3978398
Push 731731 by hxu@mozilla.com at Mon, 20 Sep 2021 17:59:04 +0000
Tasks automatically selected. Pushed via `mach try auto`
747d1454326d4c3760f7e01de3ac702def273386: Bug 1731263 - Add QM_WARNONLY_TRY to some IPC calls during shutdown of CacheAPI; r=#dom-storage-reviewers draft
hxu <hxu@mozilla.com> - Mon, 20 Sep 2021 13:12:44 +0200 - rev 3978397
Push 731731 by hxu@mozilla.com at Mon, 20 Sep 2021 17:59:04 +0000
Bug 1731263 - Add QM_WARNONLY_TRY to some IPC calls during shutdown of CacheAPI; r=#dom-storage-reviewers Differential Revision: https://phabricator.services.mozilla.com/D126092
da1559dbb390dbb977c4bcfc56707a325aea8777: try_task_config for https://phabricator.services.mozilla.com/D126098 draft
libmozevent <release-mgmt-analysis@mozilla.com> - Mon, 20 Sep 2021 17:57:53 +0000 - rev 3978396
Push 731730 by reviewbot at Mon, 20 Sep 2021 17:58:12 +0000
try_task_config for https://phabricator.services.mozilla.com/D126098 Differential Diff: PHID-DIFF-pnoym5nt4swjl7ovqb6t
951b6792ea7ca6897ec5ac4f3e0231fa4594e964: Bug 1693804: Re-enable these tests on linux. r?ng draft
Byron Campen <docfaraday@gmail.com> - Mon, 20 Sep 2021 17:57:48 +0000 - rev 3978395
Push 731730 by reviewbot at Mon, 20 Sep 2021 17:58:12 +0000
Bug 1693804: Re-enable these tests on linux. r?ng Summary: Test Plan: Reviewers: ng Subscribers: Bug #: 1693804 Differential Diff: PHID-DIFF-pnoym5nt4swjl7ovqb6t
cdc4cab9181038557153b463b81275b8ff214845: try_task_config for https://phabricator.services.mozilla.com/D124752 draft
libmozevent <release-mgmt-analysis@mozilla.com> - Mon, 20 Sep 2021 17:56:40 +0000 - rev 3978394
Push 731729 by reviewbot at Mon, 20 Sep 2021 17:57:00 +0000
try_task_config for https://phabricator.services.mozilla.com/D124752 Differential Diff: PHID-DIFF-ps2p34nozo2lzvnf7p6t
5b1e7a69a6d5d67c6868e8ed02ea9e5d882caae4: Bug 1729431 - Use an enum as the index of connection info's hash key, r=#necko draft
Kershaw Chang <kershaw@mozilla.com> - Mon, 20 Sep 2021 17:56:35 +0000 - rev 3978393
Push 731729 by reviewbot at Mon, 20 Sep 2021 17:57:00 +0000
Bug 1729431 - Use an enum as the index of connection info's hash key, r=#necko Summary: Differential Revision: https://phabricator.services.mozilla.com/D124752 Test Plan: Reviewers: #necko Subscribers: Bug #: 1729431 Differential Diff: PHID-DIFF-ps2p34nozo2lzvnf7p6t
f7fd51caabfd3301dcdb330b3d1f267f2e3abe5d: try_task_config for https://phabricator.services.mozilla.com/D124027 draft
libmozevent <release-mgmt-analysis@mozilla.com> - Mon, 20 Sep 2021 17:55:24 +0000 - rev 3978392
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
try_task_config for https://phabricator.services.mozilla.com/D124027 Differential Diff: PHID-DIFF-cdy3cpmt2uuibfipi44z
57b9757992965401be2b37ed4915276cdad75af8: Bug 1654413 - Deserialize and stream the markers and marker schemas r?emilio!,gerald! draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:55:19 +0000 - rev 3978391
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Deserialize and stream the markers and marker schemas r?emilio!,gerald! Summary: Now we can deserialize and stream everything to the JSON. Differential Revision: https://phabricator.services.mozilla.com/D124027 Depends on D124026 Test Plan: Reviewers: emilio!, gerald! Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-cdy3cpmt2uuibfipi44z
e697a6bf3acac679ba3551ad53b78532b641a961: Bug 1654413 - Add add_marker Rust profiler API and serialize the marker to the buffer r?emilio!,gerald! draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:55:14 +0000 - rev 3978390
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Add add_marker Rust profiler API and serialize the marker to the buffer r?emilio!,gerald! Summary: This only adds the API and then adds the profiler payload to the buffer. The deserialization and streaming will happen in the next patch. Differential Revision: https://phabricator.services.mozilla.com/D124026 Depends on D124025 Test Plan: Reviewers: emilio!, gerald! Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-aba5mbmaei2bg2qndnum
46e3adba30e54715c16ea1359f02ac3c5e0d51fb: Bug 1654413 - Add MarkerSchema struct and its enums for marker API r?emilio!,gerald! draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:55:09 +0000 - rev 3978389
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Add MarkerSchema struct and its enums for marker API r?emilio!,gerald! Summary: You can see the `mozilla::MarkerSchema` for the C++ counterpart. This Rust struct simply wraps the C++ object and keeps the reference of it as RAII. This heap allocates the inner C++ object but it's fine to do it here, because it's we only create a MarkerSchema object at the end of a profiling session and it happens once per marker type. It should be very rare. Differential Revision: https://phabricator.services.mozilla.com/D124025 Depends on D124024 Test Plan: Reviewers: emilio!, gerald! Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-bkkrvwcuy5jv4cgcln3n
8756658563c9a37a993d8e057aba3f37f7b379a6: Bug 1654413 - Add JSONWriter struct for marker schema r?emilio!,gerald! draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:55:04 +0000 - rev 3978388
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Add JSONWriter struct for marker schema r?emilio!,gerald! Summary: JSON writer will be used for the third and last marker API. This is needed because we need to describe a marker payload struct on how to serialize it. Differential Revision: https://phabricator.services.mozilla.com/D124024 Depends on D124023 Test Plan: Reviewers: emilio!, gerald! Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-j4d22vbedeo6ea3cijcx
8bbef2798d7f74e430adfd123f75fd9ad94bf76b: Bug 1654413 - Add add_text_marker Rust API r?emilio!,gerald! draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:54:59 +0000 - rev 3978387
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Add add_text_marker Rust API r?emilio!,gerald! Summary: This is the second API for the markers. This allows one payload as a text for more information. See the PROFILER_MARKER_TEXT macro for the C++ counterpart. Differential Revision: https://phabricator.services.mozilla.com/D124023 Depends on D124022 Test Plan: Reviewers: emilio!, gerald! Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-p7nwgcwjvtkqccwjmt32
8a65914359a62c67e6175c8462a4307aabfdba7b: Bug 1654413 - Add add_untyped_marker Rust API r?emilio!,gerald! draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:54:55 +0000 - rev 3978386
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Add add_untyped_marker Rust API r?emilio!,gerald! Summary: This is the first and simplest API for the markers. There will be two more APIs in the following patches (add_text_marker and add_marker). You can see the PROFILER_MARKER_UNTYPED macro for the C++ counterpart. Differential Revision: https://phabricator.services.mozilla.com/D124022 Depends on D124021 Test Plan: Reviewers: emilio!, gerald! Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-3srxsjxkff3eniziktvk
d5eaadc23f783c3d846a512da48ff8219cd4af31: Bug 1654413 - Add a MarkerStack Rust enum for the marker API r?emilio!,gerald! draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:54:52 +0000 - rev 3978385
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Add a MarkerStack Rust enum for the marker API r?emilio!,gerald! Summary: MarkerStack object is a bit more complex in the C++ side. But we don't need a complex object like that in here. A simple enum is enough to determine what type of marker stack we are capturing. C++ side will be handling the capturing of the stack. Differential Revision: https://phabricator.services.mozilla.com/D124021 Depends on D124020 Test Plan: Reviewers: emilio!, gerald! Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-wcnciyyeuko2esavo6m4
49677387bdbb2c36f962395a45a18e1746af5620: Bug 1654413 - Add a MarkerTiming and MarkerOptions Rust structs for the marker API r?emilio!,gerald! draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:54:49 +0000 - rev 3978384
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Add a MarkerTiming and MarkerOptions Rust structs for the marker API r?emilio!,gerald! Summary: These structs are needed for the marker APIs. We also have the same structs as the C++ classes. See `mozilla::MarkerTiming` and `mozilla::MarkerOptions`. Differential Revision: https://phabricator.services.mozilla.com/D124020 Depends on D124019 Test Plan: Reviewers: emilio!, gerald! Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-cjrhskvolo2ngoiqq354
5165b3e560fb37af134079818088f25f36fc8603: Bug 1654413 - Add a small gecko_profiler_category macro as a sugar for marker categories r?emilio! draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:54:44 +0000 - rev 3978383
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Add a small gecko_profiler_category macro as a sugar for marker categories r?emilio! Summary: This is a syntactical sugar to write the categories more easily. I prefer to have this macro because this is the same syntax as the categories in the label frames API. Differential Revision: https://phabricator.services.mozilla.com/D124019 Depends on D124018 Test Plan: Reviewers: emilio! Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-52tsyxd22hz5bmxuedzw
db271d05b4c65003895cfac1ac215767fbdd3e86: Bug 1654413 - Add ProfilerTime Rust struct for marker API r?emilio!,gerald! draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:54:40 +0000 - rev 3978382
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Add ProfilerTime Rust struct for marker API r?emilio!,gerald! Summary: This will be used to get the time from profiler, so marker API can use it to understand the start or end time. Differential Revision: https://phabricator.services.mozilla.com/D124018 Depends on D124017 Test Plan: Reviewers: emilio!, gerald! Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-oy4qjq5wfdtwz4e2dhku
c125c13efb03668e63e737f3c452a8227e92c650: Bug 1654413 - Rerun the profiler API build.rs when the included header files are changed r?emilio draft
Nazım Can Altınova <canaltinova@gmail.com> - Mon, 20 Sep 2021 17:54:36 +0000 - rev 3978381
Push 731728 by reviewbot at Mon, 20 Sep 2021 17:55:44 +0000
Bug 1654413 - Rerun the profiler API build.rs when the included header files are changed r?emilio Summary: For some reason, I thought that `parse_callbacks(Box::new(CargoCallbacks))` would also automatically adds the initial header files that are manually added. But apparently `parse_callbacks` is only executed when the inner included header files are found. Differential Revision: https://phabricator.services.mozilla.com/D124017 Test Plan: Reviewers: emilio Subscribers: Bug #: 1654413 Differential Diff: PHID-DIFF-pzvs56ahio63qe3cty2p
2a2c58541fd4c4d14959bdfc4d2bd519abfc5088: scriptworker tests: linux-signing-partial draft
trybld - Mon, 20 Sep 2021 17:52:55 +0000 - rev 3978380
Push 731727 by trybld-scriptworker at Mon, 20 Sep 2021 17:53:44 +0000
scriptworker tests: linux-signing-partial Pushed via `mach try scriptworker`
4a764dc751f1c81c6830e9f689271088c9d3d278: try_task_config for https://phabricator.services.mozilla.com/D124026 draft
libmozevent <release-mgmt-analysis@mozilla.com> - Mon, 20 Sep 2021 17:52:36 +0000 - rev 3978379
Push 731726 by reviewbot at Mon, 20 Sep 2021 17:52:58 +0000
try_task_config for https://phabricator.services.mozilla.com/D124026 Differential Diff: PHID-DIFF-aba5mbmaei2bg2qndnum
(0) -3000000 -1000000 -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip