7282bbabab15133686eef40434da0940a9244cea: Bug 1397536: Avoid newURI overhead for MatchPattern. r=ehsan,mixedpuppy
Kris Maglione <maglione.k@gmail.com> - Wed, 06 Sep 2017 21:56:45 -0700 - rev 428907
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1397536: Avoid newURI overhead for MatchPattern. r=ehsan,mixedpuppy Ehsan, can you please review the (trivial) WebIDL changes, and Shane the WebRequest logic? The change to allow strings in MatchPattern arguments removes a huge amount of XPConnect overhead that accumulates when creating nsIURI objects for WebRequest processing. The change to re-use existing URI objects removes a huge amount of URI creation overhead. MozReview-Commit-ID: 3DJjAKJK1Sa
30fe3c84b60ee8ae1ab865733041173902892e8f: Bug 1396856: Part 4 - Update WebRequest.jsm to use ChannelWrapper bindings. r=mixedpuppy
Kris Maglione <maglione.k@gmail.com> - Wed, 06 Sep 2017 21:52:24 -0700 - rev 428906
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1396856: Part 4 - Update WebRequest.jsm to use ChannelWrapper bindings. r=mixedpuppy MozReview-Commit-ID: 7s7SOQ1XVaw
223a7c5a7c471f5fbf319029da2dcc5a093b8548: Bug 1396856: Part 3 - Add a WebIDL wrapper class for necko channels. r=ehsan,mixedpuppy
Kris Maglione <maglione.k@gmail.com> - Wed, 06 Sep 2017 14:38:23 -0700 - rev 428905
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1396856: Part 3 - Add a WebIDL wrapper class for necko channels. r=ehsan,mixedpuppy Ehsan, can you please review the DOM bindings, and Shane the request logic? The bulk of the overhead WebRequest API is in its access to nsIChannel and friends through XPConnect. Since it's not really feasible to convert channels to use WebIDL bindings directly, this generic channel wrapper class serves the same purpose. MozReview-Commit-ID: 4mNP8HiKWK
3b496e60f978d3ca2792c90ba01207cd7427aaf6: Bug 1396856: Part 2 - Add top outer window ID to LoadInfo. r=ehsan
Kris Maglione <maglione.k@gmail.com> - Wed, 06 Sep 2017 14:25:23 -0700 - rev 428904
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1396856: Part 2 - Add top outer window ID to LoadInfo. r=ehsan The WebRequest API needs to know if a given window ID is at the top level, for various reasons. It currently figures this out by mapping a channel's load context to a <browser> element, which tracks its current top outer window ID. But this is inefficient, and not friendly to C++ callers. Adding the top window ID to the load info simplifies things considerably. MozReview-Commit-ID: Fy0gxTqQZMZ
1e4b5ec2904fcf7641a322480c45a4efd6da40da: Bug 1396856: Part 1 - Remove spread call fallback overhead in event dispatch. r=zombie
Kris Maglione <maglione.k@gmail.com> - Tue, 05 Sep 2017 11:20:43 -0700 - rev 428903
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1396856: Part 1 - Remove spread call fallback overhead in event dispatch. r=zombie MozReview-Commit-ID: 3c5p9OvRqHI
ed7b5443cf8b98c807cd1fa29857b51511cd0076: Bug 1276488 - Add diagnostics about Prefs startup failures. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Wed, 06 Sep 2017 15:17:05 +1000 - rev 428902
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1276488 - Add diagnostics about Prefs startup failures. r=froydnj.
93d2637ff309bbae9c09ef0eb085771d40ff7c66: Bug 1397579 - Report stylesheet memory usage under "layout/style-sheets". r=heycam.
Nicholas Nethercote <nnethercote@mozilla.com> - Thu, 07 Sep 2017 14:36:42 +1000 - rev 428901
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1397579 - Report stylesheet memory usage under "layout/style-sheets". r=heycam. Currently it's not under "layout/", for no clear reason.
cb6fcd43e5a505475df2c38a8475bc84d485975b: Bug 1393642 (follow-up) - Fix potential leak in HTMLEditor methods. r=masayuki.
Nicholas Nethercote <nnethercote@mozilla.com> - Thu, 07 Sep 2017 12:20:51 +1000 - rev 428900
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1393642 (follow-up) - Fix potential leak in HTMLEditor methods. r=masayuki. As written, these functions will leak if they are passed strings that don't match static atoms. In practice, all strings passed *do* match static atoms, but let's fix it anyway in case that changes in the future.
31ba8f169c5d1d4b118fdd7bd627a991544e7f5a: Bug 1383501: Assert that com proxy is non-null before calling SendPDocAccessibleConstructor; r=yzen
Aaron Klotz <aklotz@mozilla.com> - Wed, 06 Sep 2017 10:40:55 -0600 - rev 428899
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1383501: Assert that com proxy is non-null before calling SendPDocAccessibleConstructor; r=yzen MozReview-Commit-ID: CuIXyoe4jsj
1e6956da301f796a885e0810090d104eab13a097: Bug 1397223 - Gracefully handle failures in SurfaceCacheImpl::StartTracking. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Wed, 06 Sep 2017 20:17:04 -0700 - rev 428898
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1397223 - Gracefully handle failures in SurfaceCacheImpl::StartTracking. r=tnikkel When the surface cache starts tracking an unlocked surface, it must insert it into the expiration tracker, so that it can be freed later if it is remains unused. ExpirationTrackerImpl::AddObjectLocked can fail due to out-of-memory conditions or during shutdown, which we previously ignored, and could leave us in a state where we think the surface is in the tracker but is not. When we later try to mark the surface as used in the tracker, it will hit a release assert because it doesn't exist. Now we handle the insertion failure by discarding the surface. Marking the surface as used can itself encounter a similar issue, and we handle it the same way. MozReview-Commit-ID: Kv6l0znnG48
354981c2ee1ff9102350393ac2c262691cf72c3d: Bug 1397235 - Ensure that we reset factor-of-2 mode for an ImageSurfaceCache when it becomes empty. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Wed, 06 Sep 2017 20:16:31 -0700 - rev 428897
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1397235 - Ensure that we reset factor-of-2 mode for an ImageSurfaceCache when it becomes empty. r=tnikkel An ImageSurfaceCache cannot enter factor-of-2 mode without a minimum number of surfaces being present in its cache. However those surfaces can be purged from the cache through various means (expire due to being disuse, volatile buffers purged, etc). Also, it is entirely possible that all the surfaces get purged, but the cache itself remains. Since factor-of-2 mode requires at least one surface (to get the owning image and its native size), we need to handle the case when the cache is emptied appropriately. As such, we now reset the factor-of-2 mode (and its pruned state) to the default (false) if we transition from non-empty to empty. MozReview-Commit-ID: EVaEqW59Asv
e4fe078dc0134815c9e19e671ab3d3ea69979955: Bug 1395871 - Open toolbar menus on mousedown, rather than oncommand.
Zibi Braniecki <zbraniecki@mozilla.com> - Fri, 01 Sep 2017 01:54:34 -0700 - rev 428896
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1395871 - Open toolbar menus on mousedown, rather than oncommand. MozReview-Commit-ID: A3P4QBBgcB8
75b91420fff7c1ded50cc8dd9396590bc0172588: Bug 1390076 - Label MainThreadInvoker using SystemGroup. r=aklotz
Bevis Tseng <btseng@mozilla.com> - Mon, 28 Aug 2017 14:11:38 +0800 - rev 428895
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1390076 - Label MainThreadInvoker using SystemGroup. r=aklotz
df28182add1cd6be822246125e30b128f97b4a91: Merge m-c to inbound, a=merge
Wes Kocher <wkocher@mozilla.com> - Wed, 06 Sep 2017 17:24:48 -0700 - rev 428894
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Merge m-c to inbound, a=merge MozReview-Commit-ID: BJ4WkYrJ9g6
149d14865d30855ae7ac9a3cc55e76638ee138f3: Backed out changeset b7726e9751ca (bug 1392841) for reftest failures in font-display-1.html a=backout
Wes Kocher <wkocher@mozilla.com> - Wed, 06 Sep 2017 16:48:20 -0700 - rev 428893
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Backed out changeset b7726e9751ca (bug 1392841) for reftest failures in font-display-1.html a=backout MozReview-Commit-ID: IDsqrO2kUQ1
dce4d03cf87aef8740af78872e8822e817af3dc6: Bug 1376038 - Part 3: Combine ghost window reporter with window reporter. r=njn
Eric Rahm <erahm@mozilla.com> - Wed, 06 Sep 2017 16:06:58 -0700 - rev 428892
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1376038 - Part 3: Combine ghost window reporter with window reporter. r=njn This combines the GhostWindowsReporter with the nsWindowMemoryReporter. It has the benefit of removing a reporter of a single value and also guarantees that we use the latests ghost windows value that is calculated in |nsWindowMemoryReporter::CollectReports| rather than a possibly cached value from a previous run.
1341377398fe331eccff1305c5ccd4a1a0b52012: Bug 1376038 - Part 2: Cache base domains during ghost window calculation. r=mccr8
Eric Rahm <erahm@mozilla.com> - Tue, 11 Jul 2017 13:58:21 -0700 - rev 428891
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1376038 - Part 2: Cache base domains during ghost window calculation. r=mccr8 Avoid hitting the rather slow effective TLD service by caching results when mapping URLs to their base domains. In testing the cache ranged from a 1:1 to a 3:1 hit:miss ratio.
366cb8c89311c297ea837387bb621a1c30da8e3d: Bug 1376038 - Part 1: Use a cached ghost window value for the distinguished amount. r=mccr8
Eric Rahm <erahm@mozilla.com> - Tue, 11 Jul 2017 13:58:20 -0700 - rev 428890
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1376038 - Part 1: Use a cached ghost window value for the distinguished amount. r=mccr8 We already periodically calculate the ghost window amount after cycle collection, this just uses a cached value of that for the distinguished amount. This avoids the overhead of a recalculating the value when reporting telemetry.
7a590264fbdf75aa003d853adae3c67f4bde6ef2: Bug 1395330 - Release IPC lifetime assertions (r=mccr8)
Bill McCloskey <billm@mozilla.com> - Wed, 30 Aug 2017 11:13:16 -0700 - rev 428889
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Bug 1395330 - Release IPC lifetime assertions (r=mccr8) MozReview-Commit-ID: 3CGjEofjojc
8bd80bb1de12d3429a155fbb066f7b03606fdd0c: Backed out changeset 9018e3691e11 (bug 1378461) for failing browser-chrome's browser_oneOffContextMenu.js. r=backout
Sebastian Hengst <archaeopteryx@coole-files.de> - Thu, 07 Sep 2017 00:26:17 +0200 - rev 428888
Push 7761 by jlund@mozilla.com at Fri, 15 Sep 2017 00:19:52 +0000
Backed out changeset 9018e3691e11 (bug 1378461) for failing browser-chrome's browser_oneOffContextMenu.js. r=backout
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip