4dce29de45e3c5ecf8476bd77195c11a4ebe64ad: Bug 1400193 (part 2) - Shrink PLDHashTable. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 15 Sep 2017 20:04:29 +1000 - rev 433135
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1400193 (part 2) - Shrink PLDHashTable. r=froydnj.
This patch reduces sizeof(PLDHashTable) as follows.
- 64-bit: from 40 bytes to 32
- 32-bit: from 28 bytes to 20
It does this by doing the following.
- It moves mGeneration from EntryStore to PLDHashTable, to avoid unnecessary
padding on 64-bit. This requires tweaking EntryStore::Set() as explained in a
comment.
- It also shrinks mGeneration from uint32_t to uint16_t, saving 2 bytes of
data.
- It shrinks mEntrySize from uint32_t to uint8_t, to cut 3 bytes of data.
- It shrinks mHashShift from int16_t to uint8_t, trimming another byte of data,
and moves it, saving another 2 bytes of padding.
And it reorders the fields so the word-sized ones are at the start, which makes
it easier to imagine the memory layout.
The patch also adds a test, and fixes some misordered function arguments in
existing tests.
bd2852c72caef1f1ac622ff8862d8dc72fc072bd: Bug 1400193 (part 1) - Fix subtle bug in PLDHashTable's move constructor. r=froydnj.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 15 Sep 2017 20:02:33 +1000 - rev 433134
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1400193 (part 1) - Fix subtle bug in PLDHashTable's move constructor. r=froydnj.
The current code replaces one PLDHashTable's mGeneration with another. If the
two generation values happen to be equal, it will look as though the storage
hasn't changed when really it has.
The fix is to use EntryStore::Set() to update mEntryStore, which increments
mGeneration appropriately.
d16d4dacc88434dee7dfb9df9fdb90da5f599bf2: Merge m-c to inbound, a=merge
Wes Kocher <wkocher@mozilla.com> - Thu, 21 Sep 2017 17:04:36 -0700 - rev 433133
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Merge m-c to inbound, a=merge
MozReview-Commit-ID: IsEMW5gXG3W
84bde7821021d9661c841e7d40bfb7e3a18393ee: Bug 1400777 (part 5) - Clean up nsElementTable.{cpp,h}. r=mrbkap.
Nicholas Nethercote <nnethercote@mozilla.com> - Wed, 20 Sep 2017 08:32:51 +1000 - rev 433132
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1400777 (part 5) - Clean up nsElementTable.{cpp,h}. r=mrbkap.
This patch makes some style fixes and other minor improvements.
50d9715e7b0ada2111baa0d51fa888135f78d1d7: Bug 1400777 (part 4) - Split nsHTMLElement. r=mrbkap.
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 19 Sep 2017 11:11:24 +1000 - rev 433131
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1400777 (part 4) - Split nsHTMLElement. r=mrbkap.
This patch splits out most of nsHTMLElement into a new type HTMLElement within
nsElementTable.cpp. Only the static methods IsContainer() and IsBlock() need to
remain exposed via nsHTMLElement. The patch moves TestBits() into
nsElementTable.cpp as well.
2de6882645603af89ea672180239825dce408918: Bug 1400777 (part 3) - De-expose gHTMLElements[]. r=mrbkap.
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 19 Sep 2017 11:09:30 +1000 - rev 433130
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1400777 (part 3) - De-expose gHTMLElements[]. r=mrbkap.
2b23e3f158dd747de9c40c068d4e9a5da583cf98: Bug 1400777 (part 2) - De-expose HTML group constants[]. r=mrbkap.
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 19 Sep 2017 11:08:39 +1000 - rev 433129
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1400777 (part 2) - De-expose HTML group constants[]. r=mrbkap.
The patch also removes some low-value comments.
5835c5943a4881567e2d1e9ce1a8a6b0f97cf200: Bug 1400777 (part 1) - Remove eHTMLTags. r=mrbkap.
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 19 Sep 2017 11:07:49 +1000 - rev 433128
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1400777 (part 1) - Remove eHTMLTags. r=mrbkap.
It's just a too-cute-by-half typedef for nsHTMLTag.
f4e02a5eb8f07ce46c75d141a68b69c46efe0c14: Bug 1387920 - Drop all ImageContainers' ImageClients during disabling WebRender r=nical
sotaro <sotaro.ikeda.g@gmail.com> - Fri, 22 Sep 2017 08:21:09 +0900 - rev 433127
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1387920 - Drop all ImageContainers' ImageClients during disabling WebRender r=nical
fdffa3d2976838b613580035b2f031adba29976c: Bug 1386712 - Add null-checking to fix test bustage. r=me
Dylan Roeh <droeh@mozilla.com> - Thu, 21 Sep 2017 17:55:36 -0500 - rev 433126
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1386712 - Add null-checking to fix test bustage. r=me
976a82102f131f232cca5e88d1746fb63226c3f9: Bug 1400528 - make test_bug574663.html wait until the 'blank' attribute has actually been removed, r=mconley.
Florian Quèze <florian@queze.net> - Fri, 22 Sep 2017 00:50:54 +0200 - rev 433125
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1400528 - make test_bug574663.html wait until the 'blank' attribute has actually been removed, r=mconley.
8871efefdba925fd4fa7e8741cb545dd64131271: Bug 1401100 - Remove nsIMemoryReporterCallback. r=erahm.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 22 Sep 2017 08:09:29 +1000 - rev 433124
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1401100 - Remove nsIMemoryReporterCallback. r=erahm.
nsIHandleReportCallback, typedef of nsIMemoryReporterCallback, has been the
preferred name for a long time and is used in most places. This patch removes
nsIMemoryReporterCallback.
c86578718e191af2dac78dfde75b002448ea80ec: Bug 1386712 - Load about:blank in CustomTabsActivity.finish() to immediately stop media playback. r=snorp
Dylan Roeh <droeh@mozilla.com> - Tue, 19 Sep 2017 15:09:16 -0500 - rev 433123
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1386712 - Load about:blank in CustomTabsActivity.finish() to immediately stop media playback. r=snorp
2558072d1c5f1d951a0e9011c07f043676212ee7: Bug 1401421 - Remove profiler_get_backtrace_noalloc. r=mstange.
Nicholas Nethercote <nnethercote@mozilla.com> - Wed, 20 Sep 2017 11:46:26 +1000 - rev 433122
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1401421 - Remove profiler_get_backtrace_noalloc. r=mstange.
Bug 1385953 removed MemProfiler, which was its only user.
8f4a95f9255cfdc6f9bf41cdcb11ba2110f27603: Bug 1402123 - Disable MediaStream-MediaElement-preload-none.https.html on non-stylo builds a=bustage
Wes Kocher <wkocher@mozilla.com> - Thu, 21 Sep 2017 14:10:30 -0700 - rev 433121
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1402123 - Disable MediaStream-MediaElement-preload-none.https.html on non-stylo builds a=bustage
MozReview-Commit-ID: 6LRGcW4ukVB
220a9b1bed87420e6ec9afae78bdd94771f9af3f: Bug 1401524 - Ensure SurfaceCache state coherency whenever we perform an operation that may discard surfaces. r=tnikkel
Andrew Osmond <aosmond@mozilla.com> - Thu, 21 Sep 2017 16:56:38 -0400 - rev 433120
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1401524 - Ensure SurfaceCache state coherency whenever we perform an operation that may discard surfaces. r=tnikkel
There are a number of operations with the surface cache which may result
in individual surfaces for a particular image cache to be removed. If an
image cache is emptied, and we are in factor of 2 mode, we should reset
it to the default mode, because we require at least one surface to be
available to determine the native/ideal size. Additionally, if the cache
is not locked, it should be removed entirely from the surface cache. We
handle this correctly in methods such as Lookup and LookupBestMatch, but
Prune and CollectSizeOfSurfaces can also cause this to happen, as
recently done in
bug 1370412 and
bug 1380649.
8b73ac23236691ba8bfc692c7b28f572981bca39: Backed out changeset e89d2565799b (bug 1047098) for xpcshell bustage a=backout
Wes Kocher <wkocher@mozilla.com> - Thu, 21 Sep 2017 13:48:10 -0700 - rev 433119
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Backed out changeset e89d2565799b (
bug 1047098) for xpcshell bustage a=backout
MozReview-Commit-ID: FRFALkRxDxl
6f63440391b0dc1b0f96628882ea5843bde02bcb: Backed out changeset 04888373f12b (bug 1401878) for xpcshell bustage a=backout
Wes Kocher <wkocher@mozilla.com> - Thu, 21 Sep 2017 13:48:04 -0700 - rev 433118
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Backed out changeset 04888373f12b (
bug 1401878) for xpcshell bustage a=backout
MozReview-Commit-ID: AYiI2MJbBbC
bcab76b04fbf5e1846bdaa062fcfdd93b5b45a5f: Bug 1383501: Obtain and report the current activation context's manifest path; r=jimm
Aaron Klotz <aklotz@mozilla.com> - Tue, 19 Sep 2017 14:35:27 -0600 - rev 433117
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1383501: Obtain and report the current activation context's manifest path; r=jimm
MozReview-Commit-ID: HJ1zNoEMzvS
8af47160570c05088128428bae1eb69bf48b0dc4: Bug 1369533: Return dead wrappers rather than null for dead CallbackObject values. r=bz
Kris Maglione <maglione.k@gmail.com> - Mon, 04 Sep 2017 17:21:06 -0700 - rev 433116
Push
8114 by jlorenzo@mozilla.com at Thu, 02 Nov 2017 16:33:21 +0000
Bug 1369533: Return dead wrappers rather than null for dead CallbackObject values. r=bz
We don't have access to an appropriate context to create the dead wrapper in
when the callback is nuked, so instead, this patch creates a new dead wrapper
in the caller compartment each time the property is accessed. This is the same
behavior we'd get when trying to re-wrap a cross-compartment dead wrapper, so
it's consistent with the way we handle these situations elsewhere.
MozReview-Commit-ID: 3cMeR4z8EOe