bb21ea50b1e90bdafd620695b691227c6ac8295c: Bug 1594128 - Use generic slots for text, image and solid shader varyings. r=gw
Nicolas Silva <nsilva@mozilla.com> - Thu, 21 Nov 2019 17:37:08 +0000 - rev 503314
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1594128 - Use generic slots for text, image and solid shader varyings. r=gw The 'multi-brush' shader will have to dynamically switch between different brushes. In order to support that without needing the sum of all brush varying locations, allow aliasing a number of generic slots. This patch makes the assumption that one a vec2 and a vec4 cost the same amount of varying register space, which is suggested by the glsl specification about shader locations. If it is not the case we can add more granularity to the varying slots which are all vec4 at the moment. This also assumes that an unused varying is always optimized out. Differential Revision: https://phabricator.services.mozilla.com/D53726
912718966a67fb7cf2f21c2690dce0bd5822377a: Bug 1594128 - Dynamically choose the brush kind in the shader. r=gw
Nicolas Silva <nsilva@mozilla.com> - Thu, 21 Nov 2019 17:37:01 +0000 - rev 503313
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1594128 - Dynamically choose the brush kind in the shader. r=gw This is an experiment with only image and solid to see what the infrastructure can be like. If it works out I'll extend the it with more brush types. More work will be needed to get text rendering in there as well. The multi-brush shader includes all brushes that it potentially needs suport for. Which brushes actually get compiled in is then specified via WR_FEATURE defines. Since brushes can't have the same names for their entry points, they specify the function to use via a macros (WR_BRUSH_VS_FUNCTION and WR_BRUSH_FS_FUNCTION). Differential Revision: https://phabricator.services.mozilla.com/D53725
74554530a51089d0145a91587ab1e6d4e8c762f2: Bug 1594128 - Encode the brush shader type in the instance attributes. r=gw
Nicolas Silva <nsilva@mozilla.com> - Thu, 21 Nov 2019 17:36:53 +0000 - rev 503312
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1594128 - Encode the brush shader type in the instance attributes. r=gw This will allow the upcoming super-brush shader to select its behavior at runtime. Differential Revision: https://phabricator.services.mozilla.com/D53724
e894fff26e422166618de4401464929e9b9cb62e: Bug 1594128 - Only include shader sources once during parsing. r=gw
Nicolas Silva <nsilva@mozilla.com> - Thu, 21 Nov 2019 17:36:46 +0000 - rev 503311
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1594128 - Only include shader sources once during parsing. r=gw This chanes the shader parsing code to only inject #included shader sources once (the first time) if they are included multiple times. This will allow some extra flexibility needed by the multi-brush shader. Differential Revision: https://phabricator.services.mozilla.com/D53651
86dc639426da3cabfea26fec84c1b19db7acc274: Bug 1598258 - Add a list of additional minidumps to the event delivered when a plug-in crashes r=froydnj
Gabriele Svelto <gsvelto@mozilla.com> - Fri, 22 Nov 2019 07:30:23 +0000 - rev 503310
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1598258 - Add a list of additional minidumps to the event delivered when a plug-in crashes r=froydnj This patch adds a new field to the `plugin-crashed` event that holds the list of additional minidumps associated with a crash report. The test infrastructure is modified to use it which also fixes a race when processing the .extra file. The reftest machinery has also been modified to take the new field into account. Differential Revision: https://phabricator.services.mozilla.com/D54107
4082b974a1e6a4810996f7b3b56e829987da6b81: Bug 1597704 - Move is OriginPotentially Trustworthy into Principal r=ckerschb
Sebastian Streich <sstreich@mozilla.com> - Thu, 21 Nov 2019 16:29:13 +0000 - rev 503309
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1597704 - Move is OriginPotentially Trustworthy into Principal r=ckerschb Differential Revision: https://phabricator.services.mozilla.com/D53830
82b716f10a12149bc9f34ec69f4a9114ddff13e2: Bug 1597999 - Implements ChromeUtils.getObjectNodeId. r=jimb
Alexandre Poirot <poirot.alex@gmail.com> - Thu, 21 Nov 2019 22:12:20 +0000 - rev 503308
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1597999 - Implements ChromeUtils.getObjectNodeId. r=jimb This helps retrieving the NodeId for any JS Object you pass in. This allows identifying a particular object when using HeapSnapshot API. `HeapSnapshot.computeDominatorTree()` returns a tree of object, but there is no way to identify a given JS object in it. Getting the node id of a given JS object helps you browse this tree from a given object you know is leaking. Differential Revision: https://phabricator.services.mozilla.com/D53971
d251f6137935370ddba8bb127ea11eea6fce7cfc: Bug 1587398 - Provide schema support for tags r=perftest-reviewers,octavian_negru,davehunt
Ionut Goldan <igoldan@mozilla.com> - Fri, 22 Nov 2019 07:51:07 +0000 - rev 503307
Push 36833 by btara@mozilla.com at Fri, 22 Nov 2019 21:40:53 +0000
Bug 1587398 - Provide schema support for tags r=perftest-reviewers,octavian_negru,davehunt Differential Revision: https://phabricator.services.mozilla.com/D53973
1469feeca395eff65a81d6f283da42b0866e6717: Merge autoland to mozilla-central a=merge
Andreea Pavel <apavel@mozilla.com> - Fri, 22 Nov 2019 18:40:39 +0200 - rev 503306
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
Merge autoland to mozilla-central a=merge
ddd503f15330cdbacdb626759e885f79146148c2: bug 1543115: toolkit: disable remote agent on Windows AArch64; r=froydnj
Andreas Tolfsen <ato@sny.no> - Fri, 22 Nov 2019 08:03:56 +0000 - rev 503305
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
bug 1543115: toolkit: disable remote agent on Windows AArch64; r=froydnj We can't compile the remote agent startup component (written in Rust) for Windows AArch64 due to numerous packages depending on winapi 0.2.8 which don't support AArch64. Differential Revision: https://phabricator.services.mozilla.com/D54116
836002cd9d1a27e2d7e87f6094b6e03b0fc04557: bug 1543115: remote, toolkit: bootstrap from Rust; r=remote-protocol-reviewers,maja_zf
Andreas Tolfsen <ato@sny.no> - Fri, 22 Nov 2019 08:03:21 +0000 - rev 503304
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
bug 1543115: remote, toolkit: bootstrap from Rust; r=remote-protocol-reviewers,maja_zf This bootstraps the remote agent from Rust so that we have access to write to stderr using the eprintln!() macro. There is a future intention to expand Rust usage in the remote agent by delegating CDP and WebDriver Bi-Di protocol schema validation to serde. The Rust port is faithful to the JS version in terms of functionality, and in some places improves on the original design by enforcing a strict division between flag handling code on one hand, and the remote agent server on the other. Differential Revision: https://phabricator.services.mozilla.com/D50289
27e88dc4bc643143ef012ae0461ed40a8c109974: bug 1543115: remote: expose nsIRemoteAgent as XPCOM service; r=remote-protocol-reviewers,maja_zf,nika
Andreas Tolfsen <ato@sny.no> - Fri, 22 Nov 2019 08:03:19 +0000 - rev 503303
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
bug 1543115: remote: expose nsIRemoteAgent as XPCOM service; r=remote-protocol-reviewers,maja_zf,nika This change makes it possible to access the remote agent service from C++ and Rust. Differential Revision: https://phabricator.services.mozilla.com/D50288
7e2f0b7dfe06acb695230bc038a9389fd503ee88: bug 1543115: remote: introduce nsIRemoteAgent XPIDL interface; r=remote-protocol-reviewers,maja_zf
Andreas Tolfsen <ato@sny.no> - Fri, 22 Nov 2019 08:03:12 +0000 - rev 503302
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
bug 1543115: remote: introduce nsIRemoteAgent XPIDL interface; r=remote-protocol-reviewers,maja_zf Adds an XPIDL interface for the remote agent which we will later use to initialise and start it from a new command-line handler written in Rust. Differential Revision: https://phabricator.services.mozilla.com/D50287
b52bab54005e005b7ca6b0c76a4e6d9d3a213df7: bug 1543115: remote: make RemoteAgent.listen() accept strings; r=remote-protocol-reviewers,maja_zf
Andreas Tolfsen <ato@sny.no> - Fri, 22 Nov 2019 08:03:10 +0000 - rev 503301
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
bug 1543115: remote: make RemoteAgent.listen() accept strings; r=remote-protocol-reviewers,maja_zf Crafting nsIURIs in Rust is complicated. Allow RemoteAgent.listen() to accept both strings and nsIURIs when called in JavaScript. Differential Revision: https://phabricator.services.mozilla.com/D50286
eaf3ab5ea7adb75d524e3fa309be9b7fea00160e: bug 1543115: remote: emit "remote-listening" system notification on startup; r=remote-protocol-reviewers,maja_zf
Andreas Tolfsen <ato@sny.no> - Fri, 22 Nov 2019 08:03:08 +0000 - rev 503300
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
bug 1543115: remote: emit "remote-listening" system notification on startup; r=remote-protocol-reviewers,maja_zf When calling RemoteAgent.listen() across XPIDL the function is run asynchronously. In order to find out when the remote agent has started listening we introduce a "remote-listening" system observer notification. Differential Revision: https://phabricator.services.mozilla.com/D50285
85aaf6349ebb3f19cdbd8dbbd713103c391063d7: bug 1543115: remote: set needed prefs earlier at startup; r=remote-protocol-reviewers,maja_zf
Andreas Tolfsen <ato@sny.no> - Fri, 22 Nov 2019 08:03:05 +0000 - rev 503299
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
bug 1543115: remote: set needed prefs earlier at startup; r=remote-protocol-reviewers,maja_zf Although it currently makes no difference, we should ensure the required preferences are set sooner, in case any of the internal remote agent features depend on it. Because we also cannot control when the nsICommandLineHandler for the remote agent is invoked, setting it sooner rather than later, seems a lot safer. Differential Revision: https://phabricator.services.mozilla.com/D50283
87edacb707bd5fd5241b80ddde3fdd9059f8e33e: bug 1543115: remote: make RemoteAgent.close() safer; r=remote-protocol-reviewers,maja_zf
Andreas Tolfsen <ato@sny.no> - Fri, 22 Nov 2019 08:03:03 +0000 - rev 503298
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
bug 1543115: remote: make RemoteAgent.close() safer; r=remote-protocol-reviewers,maja_zf close() is meant to be failsafe in the sense that it should be possible to call without side-effects. We are currently setting up a lot of state in listen() that is not cleaned up if the server eventually fails to start. Calling close() when this happens will ensure any state listen() has accrued is reset. Differential Revision: https://phabricator.services.mozilla.com/D50282
5deae11d6d8bc6bd9cb2fbdc5f32bc654517ed7e: bug 1543115: remote: merge init() and listen(); r=remote-protocol-reviewers,maja_zf
Andreas Tolfsen <ato@sny.no> - Fri, 22 Nov 2019 08:02:56 +0000 - rev 503297
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
bug 1543115: remote: merge init() and listen(); r=remote-protocol-reviewers,maja_zf Having init() as a separate function leads to inconsistencies about how the required state is checked. init() prevents the remote agent from being loaded when the remote.enabled preference is false or it is attempted loaded into a child process, but listen() already manipulates state before these checks are run. This is probably not the intention, but an easy mistake to make when the code flow is not crystal clear. Since we never have a need to call init() independently, this patch merges init() into listen(). Differential Revision: https://phabricator.services.mozilla.com/D50281
27f63b79ba6683e402335f3c3fb7330b36c4ff26: bug 1543115: remote: rename XPCOM manifest for conformity; r=remote-protocol-reviewers,maja_zf
Andreas Tolfsen <ato@sny.no> - Fri, 22 Nov 2019 08:02:54 +0000 - rev 503296
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
bug 1543115: remote: rename XPCOM manifest for conformity; r=remote-protocol-reviewers,maja_zf All other XPCOM component manifests are named components.conf and this makes the remote agent's conform with those. It will also become apparent in a later patch in this changeset that we need to register two XPCOM components: one implementing the remote agent service, and one for handling command-line arguments. Differential Revision: https://phabricator.services.mozilla.com/D50280
01d092f87acb2d360caf147c802bb10932734d5e: Backed out changeset a3b107922e81 (bug 1595814) for causing marionette perma failures CLOSED TREE
Ciure Andrei <aciure@mozilla.com> - Fri, 22 Nov 2019 10:42:09 +0200 - rev 503295
Push 36832 by apavel@mozilla.com at Fri, 22 Nov 2019 16:41:57 +0000
Backed out changeset a3b107922e81 (bug 1595814) for causing marionette perma failures CLOSED TREE
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip