f05a0084c5f20abf47fa332ab5ec5c953ba48eb1: Bug 1645729 - avoid printing a warning to the console about a log level's preference. r=rfkelly
Mark Hammond <mhammond@skippinet.com.au> - Mon, 15 Jun 2020 04:51:02 +0000 - rev 535628
Push 37506 by abutkovits@mozilla.com at Mon, 15 Jun 2020 09:26:24 +0000
Bug 1645729 - avoid printing a warning to the console about a log level's preference. r=rfkelly Specifically: > Log warning: The log 'Services.Common.RESTRequest' is configured to use the preference 'services.common.log.logger.rest.request' - you must adjust the level by setting this preference, not by using the level setter This is because the hawkrequest module attempts to explicitly set the level from the preference value, but these days Log.jsm manages that itself. Also skips obtaining the log itself until it's needed, as it almost never actually is. Differential Revision: https://phabricator.services.mozilla.com/D79633
995cb36e46d49d74ec262dc7d85b4fb3497cd8f0: Backed out changeset df0d6b993ad6 (bug 1642121) for causing build bustages
Noemi Erli <nerli@mozilla.com> - Mon, 15 Jun 2020 06:44:42 +0300 - rev 535627
Push 37506 by abutkovits@mozilla.com at Mon, 15 Jun 2020 09:26:24 +0000
Backed out changeset df0d6b993ad6 (bug 1642121) for causing build bustages
df0d6b993ad6feb4ebe87c69d630015466dbfeb5: Bug 1642121 - Remove directory dependence of GDB init files. r=nalexander
Steve Fink <sfink@mozilla.com> - Mon, 15 Jun 2020 03:27:49 +0000 - rev 535626
Push 37506 by abutkovits@mozilla.com at Mon, 15 Jun 2020 09:26:24 +0000
Bug 1642121 - Remove directory dependence of GDB init files. r=nalexander I wrote this patch to address two problems: 1. if I do `mach run` from a directory other than $topsrcdir, $objdir, or $objdir/dist/bin, then .gdbinit will not be loaded. 2. Debugging the firefox binary will never load the JS prettyprinters in any case. I believe this patch fixes other problems as well, such as .gdbinit_python not being found, and the gdbpp pretty-printers not getting loaded in various situations. The main changes of this patch are: 1. Move .gdbinit into build/ (and $objdir/build/) to delay it from getting loaded until the search path is configured. 2. Move libxul.so-gdb.py into the correct directory. 3. Use either libxul.so-gdb.py or js-gdb.py to configure the correct search path then load .gdbinit, and have .gdbinit load all of the pretty-printers (Gecko and JS). 4. Use a single preprocessed file to configure the source directory. Use relative paths within the objdir for everything else. Differential Revision: https://phabricator.services.mozilla.com/D77589
6343210ca9117911824175ac7dc3c873d804e290: Bug 1640901 - Update CompositorBridgeID when ImageHost is async in ImageLayerComposite::SetLayerManager() r=mattwoodrow
sotaro <sotaro.ikeda.g@gmail.com> - Sun, 14 Jun 2020 07:29:20 +0000 - rev 535625
Push 37506 by abutkovits@mozilla.com at Mon, 15 Jun 2020 09:26:24 +0000
Bug 1640901 - Update CompositorBridgeID when ImageHost is async in ImageLayerComposite::SetLayerManager() r=mattwoodrow When Tab is moved to another window, all async ImageHosts need to update CompositorBridgeID. It is used for calling ScheduleComposition(). Differential Revision: https://phabricator.services.mozilla.com/D79068
52126fd9ffdf8322286962810e68257e28b1df93: Bug 1643656 - Remove prefers-color-scheme: no-preference. r=gl,remote-protocol-reviewers,hiro,whimboo
Emilio Cobos Álvarez <emilio@crisal.io> - Sun, 14 Jun 2020 23:26:04 +0000 - rev 535624
Push 37506 by abutkovits@mozilla.com at Mon, 15 Jun 2020 09:26:24 +0000
Bug 1643656 - Remove prefers-color-scheme: no-preference. r=gl,remote-protocol-reviewers,hiro,whimboo It was removed from the spec. Differential Revision: https://phabricator.services.mozilla.com/D78834
d925281cab42480882b2298bc0982c6575d93988: Bug 1641682. Don't kick off a validation network request if the original image network request hasn't finished. r=aosmond
Timothy Nikkel <tnikkel@gmail.com> - Sat, 13 Jun 2020 08:15:45 +0000 - rev 535623
Push 37506 by abutkovits@mozilla.com at Mon, 15 Jun 2020 09:26:24 +0000
Bug 1641682. Don't kick off a validation network request if the original image network request hasn't finished. r=aosmond Differential Revision: https://phabricator.services.mozilla.com/D78858
c68fe15a81fc2dc9fc5765f3be2573519c09b6c1: Bug 1644878 - Don't define Map/Set identical iterator methods in self-hosted JS. r=arai
Tom Schuster <evilpies@gmail.com> - Sun, 14 Jun 2020 18:59:08 +0000 - rev 535622
Push 37505 by nerli@mozilla.com at Sun, 14 Jun 2020 21:53:00 +0000
Bug 1644878 - Don't define Map/Set identical iterator methods in self-hosted JS. r=arai I think we were defining those methods in self-hosted JS, because that automatically gives us the property that they all have the same identity. This causes bad error messages. Instead redefine them in the finishInit ClassSpec hook. We still have the "wrong" methods in the 'methods' array so that JSXray tests continue to pass. Differential Revision: https://phabricator.services.mozilla.com/D79615
f01316d43bf05969d18ad5c8302e18ba712da537: Bug 1638360 - Only show bookmark menu items for nodes with a bookmarkGuid. r=robwu
Sonia Singla <soniasingla.1812@gmail.com> - Sun, 14 Jun 2020 16:50:12 +0000 - rev 535621
Push 37505 by nerli@mozilla.com at Sun, 14 Jun 2020 21:53:00 +0000
Bug 1638360 - Only show bookmark menu items for nodes with a bookmarkGuid. r=robwu Differential Revision: https://phabricator.services.mozilla.com/D79605
a4dda3c5b7bfe1bcf1052194804048a7e027932a: Bug 1645587 - Add missing null-check for family in FontList::SetLocalNames, in case font families are blocked by the font whitelist or visibility prefs. r=jwatt
Jonathan Kew <jkew@mozilla.com> - Sun, 14 Jun 2020 13:08:21 +0000 - rev 535620
Push 37505 by nerli@mozilla.com at Sun, 14 Jun 2020 21:53:00 +0000
Bug 1645587 - Add missing null-check for family in FontList::SetLocalNames, in case font families are blocked by the font whitelist or visibility prefs. r=jwatt Differential Revision: https://phabricator.services.mozilla.com/D79595
c5704e3fc7fa5e48eb3f9edd4319f4ba2c0d6999: Bug 1400317 - Add new async calls/singleton for remoting system parameters r=jmathies
Chris Martin <cmartin@mozilla.com> - Sun, 14 Jun 2020 14:23:03 +0000 - rev 535619
Push 37505 by nerli@mozilla.com at Sun, 14 Jun 2020 21:53:00 +0000
Bug 1400317 - Add new async calls/singleton for remoting system parameters r=jmathies There are a number of system parameters that return simple floats and bools and are just different forms of system parameter query. This introduces a new singleton and IPDL calls to send these values from parent to content processes and cache them in content. I started with these 2 variables because their values don't go stale. In a later changeset, I will add more logic to invalidate cached values that go stale, such as for the SPI_GETFLATMENU metric. Differential Revision: https://phabricator.services.mozilla.com/D76639
e9818ea993d02a724e56da4dd9167fc1179de68f: Backed out 2 changesets (bug 477157) for border rendering regressions.
Emilio Cobos Álvarez <emilio@crisal.io> - Sun, 14 Jun 2020 11:45:06 +0200 - rev 535618
Push 37505 by nerli@mozilla.com at Sun, 14 Jun 2020 21:53:00 +0000
Backed out 2 changesets (bug 477157) for border rendering regressions. Backed out changeset 04543b8ded50 (bug 477157) Backed out changeset 5df17ecbcaa1 (bug 477157) MANUAL PUSH: backout
b39b3f0acb4bfbd85c967c9cbb7c26155fca2620: Bug 1642234 - Convert various SVG methods to take a DOMMatrix2DInit r=emilio
longsonr <longsonr@gmail.com> - Sun, 14 Jun 2020 07:26:27 +0000 - rev 535617
Push 37505 by nerli@mozilla.com at Sun, 14 Jun 2020 21:53:00 +0000
Bug 1642234 - Convert various SVG methods to take a DOMMatrix2DInit r=emilio See https://github.com/w3c/svgwg/issues/326 Differential Revision: https://phabricator.services.mozilla.com/D77618
75b4198a731db7dcbfc6280bf25639b083e36cca: Bug 1644985 - Clean up WeakRef and FinalizationRegistry data when cross compartment wrappers are nuked r=sfink
Jon Coppeard <jcoppeard@mozilla.com> - Sat, 13 Jun 2020 23:05:15 +0000 - rev 535616
Push 37505 by nerli@mozilla.com at Sun, 14 Jun 2020 21:53:00 +0000
Bug 1644985 - Clean up WeakRef and FinalizationRegistry data when cross compartment wrappers are nuked r=sfink This crash happens because we try and clean up the map from target to WeakRef in the WeakRef finalizer, and the target can be dead by this point if it's a nuked CCW (before it is nuked the CCW ensures this sweep order does not happen). The fix is to fix up the map when CCWs to WeakRefs are nuked. Fortunately there's already a hook where the GC is told about this. The same issue applies to FinalizationRecordObjects. This fix is slightly different because they don't have a target pointer so we can't find the map entry. Instead we clear the record and cleanup happens later when it gets swept. Differential Revision: https://phabricator.services.mozilla.com/D79533
ede1c973aa858b334a495870fd9bbc019b0e94f9: Bug 1601179 - Enable async stacks but limit captured async stacks to debuggees. r=jorendorff,smaug
Logan Smyth <loganfsmyth@gmail.com> - Sun, 14 Jun 2020 02:41:45 +0000 - rev 535615
Push 37504 by rmaries@mozilla.com at Sun, 14 Jun 2020 09:24:22 +0000
Bug 1601179 - Enable async stacks but limit captured async stacks to debuggees. r=jorendorff,smaug The 'asyncStack' flag on JS execution contexts is used as a general switch to enable async stack capture across all locations in SpiderMonkey, but this causes problems because it can at times be too much of a performance burden to general and track all of these stacks. Since the introduction of this option, we have only enabled it on Nightly and DevEdition for non-mobile builds, which has left a lot of users unable to take advantage of this data while debugging. This patch enables async stack traces across all of Firefox, but introduces a new pref to toggle the scope of the actual expensive part of async stacks, which is _capturing_ them and keeping them alive in memory. The new pref limits the capturing of async stack traces to only debuggees, unless an explicit pref is flipped to capture async traces for all cases. This means that while async stacks are technically enabled, and code could manually capture a stack and pass it back to SpiderMonkey and see that stack reflected in later captured stacks, SpiderMonkey itself and related async DOM APIs, among others, will not capture stacks or pass them to SpiderMonkey, so there should be no general change in performance by enabling the broader feature itself, unless the user is actively debugging the page. One effect of this patch is that if you have the debugger open and then close it, objects that have async stacks associated with them will retain those stacks and they will continue to show up in stack traces, no _new_ stacks will be captured. jorendorff and I have decided that this is okay because the expectation that the debugger fully revert every possible effect that it could have on a page is a nice goal but not a strict requirement. Differential Revision: https://phabricator.services.mozilla.com/D68503
13436bad5624df6f36d3a26ff0542848d25c24e1: Bug 1598026 - add OOP frame mutation tests. Re-enable oop tree expansion test. r=jdescottes
Yura Zenevich <yura.zenevich@gmail.com> - Fri, 12 Jun 2020 17:32:45 +0000 - rev 535614
Push 37504 by rmaries@mozilla.com at Sun, 14 Jun 2020 09:24:22 +0000
Bug 1598026 - add OOP frame mutation tests. Re-enable oop tree expansion test. r=jdescottes Differential Revision: https://phabricator.services.mozilla.com/D78596
9920987a04bcaa4f2a4ede2770d2c744804ff386: Bug 1633625 - Implement a Find50Sequencer that searches for the garbagePerFrame resulting in 50% frame drop at 60fps r=jonco
Steve Fink <sfink@mozilla.com> - Sat, 13 Jun 2020 22:56:16 +0000 - rev 535613
Push 37504 by rmaries@mozilla.com at Sun, 14 Jun 2020 09:24:22 +0000
Bug 1633625 - Implement a Find50Sequencer that searches for the garbagePerFrame resulting in 50% frame drop at 60fps r=jonco Differential Revision: https://phabricator.services.mozilla.com/D75444
f0a75705061d129ca2af095a91a0f30064a690c1: Bug 1626425 - enable COEP reporting web-platform-tests r=dom-workers-and-storage-reviewers,ttung
Eden Chuang <echuang@mozilla.com> - Sat, 13 Jun 2020 21:13:22 +0000 - rev 535612
Push 37504 by rmaries@mozilla.com at Sun, 14 Jun 2020 09:24:22 +0000
Bug 1626425 - enable COEP reporting web-platform-tests r=dom-workers-and-storage-reviewers,ttung Differential Revision: https://phabricator.services.mozilla.com/D78859
f4f558d1cc36ef3e1c3d3c0ca9ab576a315cedea: Bug 1644819 Part 3 - Add a crashtest. r=dholbert
Ting-Yu Lin <tlin@mozilla.com> - Sat, 13 Jun 2020 19:26:07 +0000 - rev 535611
Push 37504 by rmaries@mozilla.com at Sun, 14 Jun 2020 09:24:22 +0000
Bug 1644819 Part 3 - Add a crashtest. r=dholbert Differential Revision: https://phabricator.services.mozilla.com/D79586
3b87f74ba068dbc588e61e7b71196d2d3a00049c: Bug 1644819 Part 2 - Prevent a split frame from being a dynamic reflow root. r=dholbert
Ting-Yu Lin <tlin@mozilla.com> - Sat, 13 Jun 2020 04:04:13 +0000 - rev 535610
Push 37504 by rmaries@mozilla.com at Sun, 14 Jun 2020 09:24:22 +0000
Bug 1644819 Part 2 - Prevent a split frame from being a dynamic reflow root. r=dholbert Differential Revision: https://phabricator.services.mozilla.com/D79585
4d39bd6871b85ab99737fb100f246a65ec7f610b: Bug 1644819 Part 1 - Assert that SharedFlexData is valid in flex container's continuations. r=dholbert
Ting-Yu Lin <tlin@mozilla.com> - Sat, 13 Jun 2020 04:03:30 +0000 - rev 535609
Push 37504 by rmaries@mozilla.com at Sun, 14 Jun 2020 09:24:22 +0000
Bug 1644819 Part 1 - Assert that SharedFlexData is valid in flex container's continuations. r=dholbert Differential Revision: https://phabricator.services.mozilla.com/D79584
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 tip