bafc4ddc87f6369a5085b1e43867fda014dba654: Bug 1730117 - Part 2: Make EnterModalState suppress event handling for the nested in-process documents; r=smaug
Edgar Chen <echen@mozilla.com> - Tue, 21 Sep 2021 11:39:46 +0000 - rev 592632
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1730117 - Part 2: Make EnterModalState suppress event handling for the nested in-process documents; r=smaug This also makes nsIDOMWindowUtils::SuppressEventHandling work properly. Differential Revision: https://phabricator.services.mozilla.com/D125615
27a1d92e550caaba282a450a952434005634ca78: Bug 1730117 - Part 1: Make sync XHR suppress event handling for the nested in-process documents; r=smaug
Edgar Chen <echen@mozilla.com> - Tue, 21 Sep 2021 11:39:45 +0000 - rev 592631
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1730117 - Part 1: Make sync XHR suppress event handling for the nested in-process documents; r=smaug Differential Revision: https://phabricator.services.mozilla.com/D125187
07e90ae59d8c9f94f7c957e4d737874afb8c88bc: Bug 1731189 - Combine browsertime_script and browsertime_options into a single variable. r=perftest-reviewers,kimberlythegeek
Gregory Mierzwinski <gmierz2@outlook.com> - Tue, 21 Sep 2021 11:39:32 +0000 - rev 592630
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1731189 - Combine browsertime_script and browsertime_options into a single variable. r=perftest-reviewers,kimberlythegeek This patch combines the browsertime_script and browsertime_options into a single variable before we check for and replace them with options obtained from `browsertime_args` (if any exist). See bug 1731607 for more information about this issue. Differential Revision: https://phabricator.services.mozilla.com/D126099
8c51e486379da85065221221aca2823d7cb29264: Bug 1731706: Explicitely check mAbortPromise and mState during PaymentRequest::RespondAbortPayment r=edenchuang
Jens Stutte <jstutte@mozilla.com> - Tue, 21 Sep 2021 11:32:37 +0000 - rev 592629
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1731706: Explicitely check mAbortPromise and mState during PaymentRequest::RespondAbortPayment r=edenchuang Differential Revision: https://phabricator.services.mozilla.com/D126180
3aa4f09a896eacaf6a5cac1fb45f2f363ee67a69: Bug 1731138 - Invalidate WebRender mask data from css::ImageLoader. r=tnikkel
Emilio Cobos Álvarez <emilio@crisal.io> - Tue, 21 Sep 2021 11:18:58 +0000 - rev 592628
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1731138 - Invalidate WebRender mask data from css::ImageLoader. r=tnikkel Otherwise we don't properly repaint animated images. Differential Revision: https://phabricator.services.mozilla.com/D126093
f763919875172716c44f6e113dd38e4cd8654eff: Bug 1731218 - Part 6: Check that we don't increase the size of Rooted for GC thing types r=sfink
Jon Coppeard <jcoppeard@mozilla.com> - Tue, 21 Sep 2021 11:17:51 +0000 - rev 592627
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1731218 - Part 6: Check that we don't increase the size of Rooted for GC thing types r=sfink Differential Revision: https://phabricator.services.mozilla.com/D125955
a9372964d060eb2617b7a80e046111ebe3b69816: Bug 1731218 - Part 5: Update Rooted constructor test expectations r=sfink
Jon Coppeard <jcoppeard@mozilla.com> - Tue, 21 Sep 2021 11:17:50 +0000 - rev 592626
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1731218 - Part 5: Update Rooted constructor test expectations r=sfink I don't know exactly why this is happening but we're now directly constructing MyContainer and not copying it. I confirmed that this is calling the first Rooted consturctor so I updated the tests to expect this. Differential Revision: https://phabricator.services.mozilla.com/D125954
ac495f8eecd262ddd6eaa0de5b6342ad9297b715: Bug 1731218 - Part 4: Add typed base classes for roots r=tcampbell
Jon Coppeard <jcoppeard@mozilla.com> - Tue, 21 Sep 2021 11:17:50 +0000 - rev 592625
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1731218 - Part 4: Add typed base classes for roots r=tcampbell This adds typed base classes with trace() methods. RootedTraceable is removed and we store T directly in Rooted. Differential Revision: https://phabricator.services.mozilla.com/D125953
d7e74bfc08efd38fc3f4523de78f4a75253f6154: Bug 1731218 - Part 3: Give Rooted a base class and use this refer to these roots when we don't know the type r=tcampbell
Jon Coppeard <jcoppeard@mozilla.com> - Tue, 21 Sep 2021 11:17:50 +0000 - rev 592624
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1731218 - Part 3: Give Rooted a base class and use this refer to these roots when we don't know the type r=tcampbell The link pointers move to the base class in a similar way to PersistentRootedBase. This lets us get rid of a bunch more reinterpret_casts. Differential Revision: https://phabricator.services.mozilla.com/D125952
bc5898ff5c3d4311adba5cdc7c35dc753d127a89: Bug 1731218 - Part 2: Give PersistentRooted a base class and use this refer to these roots when we don't know the type r=tcampbell
Jon Coppeard <jcoppeard@mozilla.com> - Tue, 21 Sep 2021 11:17:49 +0000 - rev 592623
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1731218 - Part 2: Give PersistentRooted a base class and use this refer to these roots when we don't know the type r=tcampbell This removes the need for a bunch of reinterpret_casts. Differential Revision: https://phabricator.services.mozilla.com/D125951
b45dc61300b183efa0f6bcf19bf0f0a431f4d958: Bug 1731218 - Part 1: Rename rooting API base classes to 'Operations' in prepartion for adding base classes r=sfink
Jon Coppeard <jcoppeard@mozilla.com> - Tue, 21 Sep 2021 11:17:49 +0000 - rev 592622
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1731218 - Part 1: Rename rooting API base classes to 'Operations' in prepartion for adding base classes r=sfink These are really mixins used to add public methods based on the type and don't need to the base of an inheritance tree. Differential Revision: https://phabricator.services.mozilla.com/D125950
5c9145f71f553a5d739ce32abde81d2e7479ee15: Bug 1731218 - Part 0: Remove unnecessary template from Rooted constructor r=sfink
Jon Coppeard <jcoppeard@mozilla.com> - Tue, 21 Sep 2021 11:17:49 +0000 - rev 592621
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1731218 - Part 0: Remove unnecessary template from Rooted constructor r=sfink I don't think this does anything. The type is always present so we won't restrict use of this constructor by SFINAE, although I may have misunderstood the purpose of this. I tried 'fixing' the SFINAE to what I though it should be by using enable_if_t but it had the same effect as removing it. In both cases the code compiles, and if I try to construct a rooted with an inappropriate argument it fails with a sensible error. Differential Revision: https://phabricator.services.mozilla.com/D125949
8d2ac26f5f6f717b29ec57a772d4b57c3eb8b52f: Bug 1654413 - Deserialize and stream the markers and marker schemas r=emilio,gerald
Nazım Can Altınova <canaltinova@gmail.com> - Tue, 21 Sep 2021 11:08:12 +0000 - rev 592620
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1654413 - Deserialize and stream the markers and marker schemas r=emilio,gerald Now we can deserialize and stream everything to the JSON. Differential Revision: https://phabricator.services.mozilla.com/D124027
5ac58c00d75b30683c69e551dc56cb96bc8fe64f: Bug 1654413 - Add add_marker Rust profiler API and serialize the marker to the buffer r=emilio,gerald
Nazım Can Altınova <canaltinova@gmail.com> - Tue, 21 Sep 2021 11:08:11 +0000 - rev 592619
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1654413 - Add add_marker Rust profiler API and serialize the marker to the buffer r=emilio,gerald 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
65e0c7376519dc01d400a9d1b972ca581169fbd4: Bug 1654413 - Add MarkerSchema struct and its enums for marker API r=emilio,gerald
Nazım Can Altınova <canaltinova@gmail.com> - Tue, 21 Sep 2021 11:08:11 +0000 - rev 592618
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1654413 - Add MarkerSchema struct and its enums for marker API r=emilio,gerald 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
680cd33b16422537fd6c0da7fb12ae53d06462f3: Bug 1654413 - Add JSONWriter struct for marker schema r=emilio,gerald
Nazım Can Altınova <canaltinova@gmail.com> - Tue, 21 Sep 2021 11:08:11 +0000 - rev 592617
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1654413 - Add JSONWriter struct for marker schema r=emilio,gerald 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
fde7a635924e9a7e95d6c3f38daf8a4fb4452549: Bug 1654413 - Add add_text_marker Rust API r=emilio,gerald
Nazım Can Altınova <canaltinova@gmail.com> - Tue, 21 Sep 2021 11:08:10 +0000 - rev 592616
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1654413 - Add add_text_marker Rust API r=emilio,gerald 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
c82aee76d44d6b748d9f5cf1dcc062bd94dd9d81: Bug 1654413 - Add add_untyped_marker Rust API r=emilio,gerald
Nazım Can Altınova <canaltinova@gmail.com> - Tue, 21 Sep 2021 11:08:10 +0000 - rev 592615
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1654413 - Add add_untyped_marker Rust API r=emilio,gerald 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
89debdc6aaaca1740b354ff00480e0254e726216: Bug 1654413 - Add a MarkerStack Rust enum for the marker API r=emilio,gerald
Nazım Can Altınova <canaltinova@gmail.com> - Tue, 21 Sep 2021 11:08:09 +0000 - rev 592614
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1654413 - Add a MarkerStack Rust enum for the marker API r=emilio,gerald 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
36660c3995841d19276ffe49d6191e41b5c2c2da: Bug 1654413 - Add a MarkerTiming and MarkerOptions Rust structs for the marker API r=emilio,gerald
Nazım Can Altınova <canaltinova@gmail.com> - Tue, 21 Sep 2021 11:08:09 +0000 - rev 592613
Push 38811 by nbeleuzu@mozilla.com at Wed, 22 Sep 2021 04:09:13 +0000
Bug 1654413 - Add a MarkerTiming and MarkerOptions Rust structs for the marker API r=emilio,gerald 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
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip