bd35bf43d67daa82877249cb9943dce3c6993c53: Bug 1414168 - Demystify the last test in the main arena_bin_run_size_calc loop. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Tue, 07 Nov 2017 08:55:37 +0900 - rev 444161
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1414168 - Demystify the last test in the main arena_bin_run_size_calc loop. r=njn The description above the RUN_* constant definitions talks about binary fixed point math, which is one way to look at the problem, but a clearer one is to look at it as comparing ratios in a way that doesn't use divisions. So, starting from the current expression: (try_reg0_offset << RUN_BFP) <= RUN_MAX_OVRHD * try_run_size This can be rewritten as try_reg0_offset * (1 << RUN_BFP) <= RUN_MAX_OVRHD * try_run_size Dividing both sides with ((1 << RUN_BFP) * try_run_size), and simplifying, gives us: try_reg0_offset / try_run_size <= RUN_MAX_OVRHD / (1 << RUN_BFP) Replacing the constants: try_reg0_offset / try_run_size <= 0x3d / (1 << 12) or try_reg0_offset / try_run_size <= 61 / 4096 61 / 4096 is roughly 1.5%. So what the check really intends to do is check that the overhead is below 1.5%. So we introduce a helper class and a user-defined literal that makes the test more self-descriptive, while producing identical machine code. This is a lot of code to add, but I think it's one of those cases where abstraction can help make the code clearer.
5ab2a7bd031f3060cb57b072349fbc8e7178547a: Bug 1414168 - Split the condition for the main arena_bin_run_size_calc loop into pieces. r=njn
Mike Hommey <mh+mozilla@glandium.org> - Tue, 07 Nov 2017 07:42:21 +0900 - rev 444160
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1414168 - Split the condition for the main arena_bin_run_size_calc loop into pieces. r=njn This is, practically speaking, a no-op, and will hopefully help make the following changes clearer.
7ae07e7920fec8d066b757acfc112746a2ab7c1c: Bug 1415707 - Remove about:accounts redirector. r=markh
Edouard Oger <eoger@fastmail.com> - Wed, 08 Nov 2017 18:23:55 -0500 - rev 444159
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415707 - Remove about:accounts redirector. r=markh MozReview-Commit-ID: JslMt32G1AS
27403a9dec13a8d6c37b764782db51ca81ce9436: Bug 1415225 - Add a couple of mochitests for event-regions overrides. r=botond
Kartikaya Gupta <kgupta@mozilla.com> - Wed, 08 Nov 2017 20:47:19 -0500 - rev 444158
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415225 - Add a couple of mochitests for event-regions overrides. r=botond These two tests exercise the codepaths where an event regions override is set on (a) the root layer of a layer tree and (b) on an in-process subdocument. I verified that case (b) was failing with webrender enabled without the patchset and passes now. MozReview-Commit-ID: GMwk5iQstj
9e671058653199248a401a379f787fef0ccced89: Bug 1415225 - Small early-exit optimization. r=botond
Kartikaya Gupta <kgupta@mozilla.com> - Wed, 08 Nov 2017 20:47:19 -0500 - rev 444157
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415225 - Small early-exit optimization. r=botond MozReview-Commit-ID: 8h1kYevQUtT
8cfef302b7102e2953f61020a26c26313cebc4e4: Bug 1415225 - Move the EventRegionsOverride field to be on RefLayers only. r=botond
Kartikaya Gupta <kgupta@mozilla.com> - Wed, 08 Nov 2017 20:47:19 -0500 - rev 444156
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415225 - Move the EventRegionsOverride field to be on RefLayers only. r=botond We now set EventRegionsOverride flags on ref layers only, so there's no need to have the APIs to set it on container layers in general. MozReview-Commit-ID: EX57VvaZv8A
8ae80d7421d5b01d9e6ef8a7ce1c62a2f0b71149: Bug 1415225 - Stop setting the EventRegionsOverride flag on root layers. r=botond,mattwoodrow
Kartikaya Gupta <kgupta@mozilla.com> - Wed, 08 Nov 2017 20:47:18 -0500 - rev 444155
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415225 - Stop setting the EventRegionsOverride flag on root layers. r=botond,mattwoodrow As with the previous patch, instead of setting the override on the root layer, we set the flag on the nsDisplayListBuilder before building the display list, and the flag automatically forces all event regions display items to use their dispatch-to-content region instead of any other regions. Both the WebRender and non-WebRender codepaths were setting the override flag on their root layers and don't need to any more. MozReview-Commit-ID: KQV3w2nvlgs
5f14fc545115a089e128579f7a37189d6cb99d61: Bug 1415225 - Stop setting the EventRegionsOverride flag on in-process subdocument layers. r=botond
Kartikaya Gupta <kgupta@mozilla.com> - Wed, 08 Nov 2017 20:47:18 -0500 - rev 444154
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415225 - Stop setting the EventRegionsOverride flag on in-process subdocument layers. r=botond The mechanics of this change is fairly straightforward - instead of setting the override on the layer corresponding to the in-process subdocument, we just set the flag on the display list builder; that flag is already checked when building the layer event regions for descendant nsIFrames. As a side-effect, we also don't need to force a layer for in-process subdocuments just because they have document-level APZ-aware listeners. One of reasons we were doing so before was so that we would have a layer to stash the override flags on but now we don't need that any more. Note that out-of-process subdocuments are not affected; for those cases the nsSubDocumentFrame delegates BuildDisplayList to RenderFrameParent/nsDisplayRemote, which will still set the overrides on the RefLayer that is created. MozReview-Commit-ID: GTy9BmVVZ9q
4704645d4e81505c680c0238b72ff552c39a6b85: Bug 1415225 - Remove dead code. r=jrmuizel
Kartikaya Gupta <kgupta@mozilla.com> - Wed, 08 Nov 2017 20:47:18 -0500 - rev 444153
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415225 - Remove dead code. r=jrmuizel This code has been unused since the removal of layers-full WR codepaths. MozReview-Commit-ID: EAhRrZXLKKr
4b0b68c54b54431d9a91b15968b0c76deff6cc33: Bug 1413480 - Check mCDM in ChromiumCDMChild::RecvXXX before dereferencing it. r=cpearce,kikuo
James Cheng <jacheng@mozilla.com> - Wed, 08 Nov 2017 15:44:34 +0800 - rev 444152
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1413480 - Check mCDM in ChromiumCDMChild::RecvXXX before dereferencing it. r=cpearce,kikuo MozReview-Commit-ID: Dp4FOtZ8Yok
2203d23429f61f879753234b275ff141c50c11f0: Backed out 1 changesets (bug 1393817) for failing to repackage the nightly build on Windows a=backout.
dluca <dluca@mozilla.com> - Thu, 09 Nov 2017 15:34:00 +0200 - rev 444151
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Backed out 1 changesets (bug 1393817) for failing to repackage the nightly build on Windows a=backout. Backed out changeset 25f21f22a34b (bug 1393817)
d16b52f5d1955192c42c7b5c5da4e05a7dffef27: merge mozilla-inbound to mozilla-central. r=merge a=merge
Sebastian Hengst <archaeopteryx@coole-files.de> - Thu, 09 Nov 2017 11:53:05 +0200 - rev 444150
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
merge mozilla-inbound to mozilla-central. r=merge a=merge
e368fe33afd342ed8540df982ce4ef0d37617940: Bug 1415418 - Simplify GetUint(). r=glandium.
Nicholas Nethercote <nnethercote@mozilla.com> - Wed, 08 Nov 2017 16:45:42 +1100 - rev 444149
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415418 - Simplify GetUint(). r=glandium. This makes it just like GetDefaultUint(). MozReview-Commit-ID: CSoLpLdstAv
017d8233ee85a6643e0e23e11509d2a4d889da08: Bug 1415418 - Reorder Preferences.h. r=glandium.
Nicholas Nethercote <nnethercote@mozilla.com> - Wed, 08 Nov 2017 16:45:38 +1100 - rev 444148
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415418 - Reorder Preferences.h. r=glandium. The various getters and setters are in a confusing order. This patch puts them in a more sensible order. It also streamlines the comments, which were generally low-value and in some cases incorrect. MozReview-Commit-ID: 3ngzZDSt0JI
2557e97345f3817b9b70320a8079a89088b182e9: Bug 1415663 - Add crashtest. r=me
Jason Kratzer <jkratzer@mozilla.com> - Wed, 08 Nov 2017 22:39:49 -0500 - rev 444147
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415663 - Add crashtest. r=me
Jesse Ruderman <jruderman@gmail.com> - Wed, 08 Nov 2017 22:39:49 -0500 - rev 444146
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 890760 - Add crashtest. r=me
4bdb96f48bb68c7552b7051a9e5edc3bd238b7d0: Bug 1415647 Part 2 - Support renaming shortcuts during paveover installs. r=rstrong
Matt Howell <mhowell@mozilla.com> - Wed, 08 Nov 2017 11:12:06 -0800 - rev 444145
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415647 Part 2 - Support renaming shortcuts during paveover installs. r=rstrong MozReview-Commit-ID: yHfvfJpHao
7ff201a34ffc6a98c6433759ed21c910629e39ec: Bug 1415647 Part 1 - Streamline PostUpdate shortcut renaming code. r=rstrong
Matt Howell <mhowell@mozilla.com> - Wed, 08 Nov 2017 11:07:54 -0800 - rev 444144
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415647 Part 1 - Streamline PostUpdate shortcut renaming code. r=rstrong In an attempt to make the code that renames shortcuts during PostUpdate more reliable, this patch switches to renaming the existing shortcut files instead of deleting them and creating new ones, removes unused code dealing with icons, and deduplicates the code by adding a new macro. MozReview-Commit-ID: EnE2dGrunx2
89994077d440cfaac3adba3191a480a52637384f: Bug 1414138 - Keep IndexedDB in sandbox unlabeled. r=billm
Bevis Tseng <btseng@mozilla.com> - Mon, 06 Nov 2017 11:58:13 +0800 - rev 444143
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1414138 - Keep IndexedDB in sandbox unlabeled. r=billm Script evaluated inside sandbox could touch scripts exposed by its creator in content. We should keep indexedDB in sandbox unlabeled instead of labeling it with SystemGroup.
bda35e3d8ce4e6fbdc416b427f2200110ce52ff4: Bug 1415062 - part 2: Editor should use Selection::Collapse(const RawRangeBoundary&) as far as possible r=m_kato
Masayuki Nakano <masayuki@d-toybox.com> - Tue, 07 Nov 2017 19:50:25 +0900 - rev 444142
Push 1618 by Callek@gmail.com at Thu, 11 Jan 2018 17:45:48 +0000
Bug 1415062 - part 2: Editor should use Selection::Collapse(const RawRangeBoundary&) as far as possible r=m_kato In some places, editor computes index from child node for collapsing selection at the child node. However, it's expensive. Therefore, editor should use Selection::Collapse(const RawRangeBoundary&) as far as possible. MozReview-Commit-ID: LF2MwASuXzZ