6ebd9f34c08ea7bfbf89a25dd7fa639f96d38302: Bug 1295192 - part 4 - use a non-null-checked placement new operator in layout code; r=dholbert
Nathan Froyd <froydnj@gmail.com> - Wed, 17 Aug 2016 15:28:45 -0400 - rev 309832
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1295192 - part 4 - use a non-null-checked placement new operator in layout code; r=dholbert The standard placement new function is declared to not throw, which means that, per spec, a null check on its result is required. There are a number of places throughout layout/ where we know that we are passing non-null pointers to placement new (and receiving them as a return value), and we are therefore doing useless work performing these null checks. Therefore, we should be using an operator new overload that doesn't require the null check. MFBT has just such an overload, so use that.
ded99645e6dffa489eab7360bf536bf88220f5eb: Bug 1295192 - part 3 - remove CPP_THROW_NEW from display list code; r=dholbert
Nathan Froyd <froydnj@gmail.com> - Wed, 17 Aug 2016 15:28:45 -0400 - rev 309831
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1295192 - part 3 - remove CPP_THROW_NEW from display list code; r=dholbert nsDisplayListBuilder::Allocate is infallible. Therefore, nsDisplayListItem::operator new, which calls Allocate to obtain memory, does not need to declare itself as throwing. And so on for functions that call nsDisplayListBuilder::Allocate.
03b66848f748859939710a1ed1aeab51a3625950: Bug 1295192 - part 2 - remove CPP_THROW_NEW from style struct type-safe operator new overloads; r=bholley
Nathan Froyd <froydnj@gmail.com> - Wed, 17 Aug 2016 15:28:45 -0400 - rev 309830
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1295192 - part 2 - remove CPP_THROW_NEW from style struct type-safe operator new overloads; r=bholley operator new overloads that declare that they don't throw exceptions require a null check on their return value, per the C++ spec. We know that Servo isn't going to call these functions with null pointers, so remove the CPP_THROW_NEW annotation and save ourselves some work.
6e323796605a15549fe37b1bec6dc904c93ef886: Bug 1295192 - part 1 - remove CPP_THROW_NEW on layout struct operator new overloads that forward to nsPresShell::AllocateByObjectID; r=dholbert
Nathan Froyd <froydnj@gmail.com> - Wed, 17 Aug 2016 15:28:45 -0400 - rev 309829
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1295192 - part 1 - remove CPP_THROW_NEW on layout struct operator new overloads that forward to nsPresShell::AllocateByObjectID; r=dholbert Structs in our style system use an arena-style allocation system, managed by the presshell to which they belong. All of the relevant overloads that forward allocation requests to the presshell declare themselves as CPP_THROW_NEW, which indicates that they do not throw exceptions. The C++ specification states that operator new overloads that declare themselves to not throw exceptions require a null check on their return value. However, the relevant presshell allocation method, AllocateByObjectID, is infallible and will never return a null pointer. The callers of all of these methods are therefore doing useless (compiler-generated) null checks. Let's get rid of those useless checks by removing the CPP_THROW_NEW annotations. This change declares these methods will return non-null pointers and throw exceptions in case of errors--but as we don't use exceptions, and AllocateByObjectID will abort on OOM, everything works out OK.
e232e2267d5f5fb009cb5d3d1a446e5b28ae8915: Bug 1258747 - Support for alternative data in CacheFile, r=honzab
Michal Novotny <michal.novotny@gmail.com> - Wed, 17 Aug 2016 12:58:44 +0200 - rev 309828
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1258747 - Support for alternative data in CacheFile, r=honzab
7e2cfb44947efe8028d4b6cc6f072af56495f0bc: Backed out changeset 8ead64ff09e8 (bug 1294013)
Iris Hsiao <ihsiao@mozilla.com> - Wed, 17 Aug 2016 18:50:40 +0800 - rev 309827
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Backed out changeset 8ead64ff09e8 (bug 1294013)
5471793e8e65ac52c8d33a67c944f319d028005d: Backed out changeset 9eec5a814230 (bug 1294013)
Iris Hsiao <ihsiao@mozilla.com> - Wed, 17 Aug 2016 18:50:34 +0800 - rev 309826
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Backed out changeset 9eec5a814230 (bug 1294013)
33c91255b886042253ff6afb5f25976c92e0a359: Backed out changeset 33283d086876 (bug 1294013)
Iris Hsiao <ihsiao@mozilla.com> - Wed, 17 Aug 2016 18:50:29 +0800 - rev 309825
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Backed out changeset 33283d086876 (bug 1294013)
827512670fd6b9e5ae5724acf7d8c406e6dfde0d: Backed out changeset 0289f779f30b (bug 1294013)
Iris Hsiao <ihsiao@mozilla.com> - Wed, 17 Aug 2016 18:50:24 +0800 - rev 309824
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Backed out changeset 0289f779f30b (bug 1294013)
97f275254cecdc54d07c68a8d0be263d90dbaa2c: Backed out changeset d14511be496e (bug 1294013) for Windows build bustage
Iris Hsiao <ihsiao@mozilla.com> - Wed, 17 Aug 2016 18:50:18 +0800 - rev 309823
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Backed out changeset d14511be496e (bug 1294013) for Windows build bustage
272a2ab0d60d40bcf6751c36dd823651cfda023d: Bug 1281382 - Fix setTargetAtTime using incorrect starting value when earlier event is skipped; r=karlt
Dan Minor <dminor@mozilla.com> - Fri, 12 Aug 2016 14:57:19 -0400 - rev 309822
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1281382 - Fix setTargetAtTime using incorrect starting value when earlier event is skipped; r=karlt This updates mLastComputedValue when removing events during the call to CleanupEventsOlderThan. MozReview-Commit-ID: 1Veyv8kLIna
88ba08e09310b4fb71fdf35fa478f7fa1d723d92: Bug 1295027 - Fix spurious rooting hazard r=me
Jon Coppeard <jcoppeard@mozilla.com> - Wed, 17 Aug 2016 11:02:25 +0100 - rev 309821
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1295027 - Fix spurious rooting hazard r=me
d14511be496e4ddc23f763ff4df467c9d34c891f: Bug 1294013 - Remove callHook from parseResumptionValue(Helper). r=jimb
Eddy Bruel <ejpbruel@mozilla.com - Wed, 17 Aug 2016 11:45:59 +0200 - rev 309820
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1294013 - Remove callHook from parseResumptionValue(Helper). r=jimb
0289f779f30b0187a93fd0441ce77c7120798788: Bug 1294013 - Replace parseResumptionValueHelper with processResumptionValue in handleUncaughtException. r=jimb
Eddy Bruel <ejpbruel@mozilla.com - Wed, 17 Aug 2016 11:44:44 +0200 - rev 309819
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1294013 - Replace parseResumptionValueHelper with processResumptionValue in handleUncaughtException. r=jimb
33283d086876e8d273b1dc80d9f6ab437ccb27e8: Bug 1294013 - Factor out processResumptionValue. r=jimb
Eddy Bruel <ejpbruel@mozilla.com - Wed, 17 Aug 2016 11:42:45 +0200 - rev 309818
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1294013 - Factor out processResumptionValue. r=jimb
9eec5a8142305c3e48c9152732ce526a7b81842c: Bug 1294013 - Factor out CheckResumptionValue. r=jimb
Eddy Bruel <ejpbruel@mozilla.com - Wed, 17 Aug 2016 11:40:35 +0200 - rev 309817
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1294013 - Factor out CheckResumptionValue. r=jimb
8ead64ff09e8ef5004981be47c7b4b7cf099e30d: Bug 1294013 - Factor out ParseResumptionValue. r=jimb
Eddy Bruel <ejpbruel@mozilla.com - Wed, 17 Aug 2016 11:39:05 +0200 - rev 309816
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1294013 - Factor out ParseResumptionValue. r=jimb
c92292e06b2d7a451b5bd11709ad56eabc0b197f: Bug 1295035 - Check for failure when allocating chunks r=terrence
Jon Coppeard <jcoppeard@mozilla.com> - Wed, 17 Aug 2016 09:41:29 +0100 - rev 309815
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1295035 - Check for failure when allocating chunks r=terrence
20e66ea8de8eceb9489274116761b4f699c8f029: Bug 1295027 - Only mark objects for pretenuring after end of trace session r=terrence
Jon Coppeard <jcoppeard@mozilla.com> - Wed, 17 Aug 2016 09:41:29 +0100 - rev 309814
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1295027 - Only mark objects for pretenuring after end of trace session r=terrence
24ebd63a53ec60b329e81e1838aa4d9433360c19: Bug 1295551 - Sweep dictionary mode objects after minor GC r=terrence
Jon Coppeard <jcoppeard@mozilla.com> - Wed, 17 Aug 2016 09:41:29 +0100 - rev 309813
Push 30570 by kwierso@gmail.com at Wed, 17 Aug 2016 23:38:48 +0000
Bug 1295551 - Sweep dictionary mode objects after minor GC r=terrence
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip