6d26e408810ffb9775993017fe8f8da26ec552ef: Bug 1225923 - part 3 - fix AppendElement(nsDependentString(...)) calls in dom/telephony/; r=btseng
Nathan Froyd <froydnj@mozilla.com> - Wed, 18 Nov 2015 10:53:05 -0500 - rev 313261
Push 8060 by gijskruitbosch@gmail.com at Wed, 02 Dec 2015 10:24:00 +0000
Bug 1225923 - part 3 - fix AppendElement(nsDependentString(...)) calls in dom/telephony/; r=btseng I think the intent of these calls is to not copy the information strings passed in, but to simply convert them to a friendlier nsString container for processing. In the TelephonyDialCallback case, we're copying the strings only to convert them to an array of JS::Value, which copies the data again. We might as well avoid one of the copies. In the TelephonyParent case, we're copying the strings only to send them over an IPC channel. Therefore, it's safe to not copy the strings initially, because the original strings will remain live long enough to send the IPC message.
c18269dc70b8446906adaf809ad6b44e3c347327: Bug 1225923 - part 2 - fix AppendElement(nsDependentString(...)) call in HTMLInputElement.cpp; r=baku
Nathan Froyd <froydnj@mozilla.com> - Wed, 18 Nov 2015 10:36:40 -0500 - rev 313260
Push 8060 by gijskruitbosch@gmail.com at Wed, 02 Dec 2015 10:24:00 +0000
Bug 1225923 - part 2 - fix AppendElement(nsDependentString(...)) call in HTMLInputElement.cpp; r=baku I think the intent of this call is to not copy the filename data passed in, but to simply convert it to a friendlier nsString container for processing. We don't have to worry about the nsTArray<nsString> overload of MozSetFileNameArray holding references to the strings outside the call, as the elements of the array are copied into new strings as appropriate.
fe7c4e4d145119da9e35b29214078e8b6b6acee9: Bug 1225923 - part 1 - convert all needs-to-copy instances of AppendElement(nsDependentString(...)); r=erahm
Nathan Froyd <froydnj@mozilla.com> - Wed, 18 Nov 2015 10:25:19 -0500 - rev 313259
Push 8060 by gijskruitbosch@gmail.com at Wed, 02 Dec 2015 10:24:00 +0000
Bug 1225923 - part 1 - convert all needs-to-copy instances of AppendElement(nsDependentString(...)); r=erahm When people write: array.AppendElement(nsDependentString(...)); (resp. nsDependentCString), it's not clear whether they expect the newly constructed dependent string to live in the array, or whether they're just making a nsString-like holder whose contents can be freely copied into the array's newly-created nsString. Sometimes the latter is what you prefer, and sometimes the former. In all cases, however, the latter behavior is what you get. Let's try to make that behavior more explicit by pre-constructing nsString elements and then using Assign to show that copying is taking place. This patch involves no functional change in behavior (it ought to be epsilon faster due to using AppendElements, rather than repeatedly calling AppendElement).
42d9f12af41a7491bfde4875ac6fc44a0c77afa1: Bug 1229028 - remove #ifdef NS_BUILD_REFCNT_LOGGING methods from nsXULPrototypeNode; r=bz
Nathan Froyd <froydnj@mozilla.com> - Mon, 30 Nov 2015 10:15:44 -0500 - rev 313258
Push 8060 by gijskruitbosch@gmail.com at Wed, 02 Dec 2015 10:24:00 +0000
Bug 1229028 - remove #ifdef NS_BUILD_REFCNT_LOGGING methods from nsXULPrototypeNode; r=bz These methods are subsumed by the automatic generation of AddRef/Release methods featuring the class name, and are not called in any event.
14ea3aec5d8492396eac15b547f9e6efe5f18a27: Bug 1229176 - make check for ChromeOnly interfaces for header inclusion more complete; r=bz
Nathan Froyd <froydnj@mozilla.com> - Mon, 30 Nov 2015 20:57:57 -0500 - rev 313257
Push 8060 by gijskruitbosch@gmail.com at Wed, 02 Dec 2015 10:24:00 +0000
Bug 1229176 - make check for ChromeOnly interfaces for header inclusion more complete; r=bz If an interface has a {Chrome,}Constructor(), it doesn't show up as a normal member. If the interface has a ChromeConstructor, we need to include nsContentUtils.h in the generated file for a ThreadsafeIsCallerChrome check. There is an existing check for a descriptor's ChromeOnly-ness in CGBindingRoot; this check is used to determine whether nsContentUtils.h is included in the generated file. But the check in descriptorHasChromeOnly doesn't detect this (ChromeOnly) constructor, and so nsContentUtils.h won't be included if there are no other ChromeOnly members, or if the interface itself is not ChromeOnly. Therefore, we need to take the constructor of the interface (if any) into account when checking for ChromeOnly-ness.
ed2b80d144f9eaf50d68cfeb68d613671ef5eb52: Bug 1228451 - Set short-lived cert lifetime to the max OCSP response lifetime. r=keeler
Mark Goodwin <mgoodwin@mozilla.com> - Mon, 30 Nov 2015 22:05:07 +0000 - rev 313256
Push 8060 by gijskruitbosch@gmail.com at Wed, 02 Dec 2015 10:24:00 +0000
Bug 1228451 - Set short-lived cert lifetime to the max OCSP response lifetime. r=keeler
5545627073b95f3a240a3078082d9ad472c8aa24: Bug 1198450 - GCC (mingw) fixup.
Jacek Caban <jacek@codeweavers.com> - Tue, 01 Dec 2015 14:20:41 +0100 - rev 313255
Push 8060 by gijskruitbosch@gmail.com at Wed, 02 Dec 2015 10:24:00 +0000
Bug 1198450 - GCC (mingw) fixup.
d9a1efa7d1ab0c74cc2f297e627221d454515fe3: Bug 1228542 - Resetting AuthorStyleSheets. r=bz
Gabor Krizsanits <gkrizsanits@mozilla.com> - Tue, 01 Dec 2015 13:54:34 +0100 - rev 313254
Push 8060 by gijskruitbosch@gmail.com at Wed, 02 Dec 2015 10:24:00 +0000
Bug 1228542 - Resetting AuthorStyleSheets. r=bz
0b3bcb3158ba59b2525c0d95a4ffe57541580229: Bug 1229258: Implement dummy asmMergeWith/retargetWithOffset functions in none backend; r=h4writer
Benjamin Bouvier <benj@benj.me> - Tue, 01 Dec 2015 13:06:35 +0100 - rev 313253
Push 8060 by gijskruitbosch@gmail.com at Wed, 02 Dec 2015 10:24:00 +0000
Bug 1229258: Implement dummy asmMergeWith/retargetWithOffset functions in none backend; r=h4writer
1968000b4c043c65d8fbfbc028992f729ab3ee7d: Backed out changeset d821c903bd2b (bug 1171863) for frequent failures in browser_animation_timeline_pause_button.js
Wes Kocher <wkocher@mozilla.com> - Tue, 01 Dec 2015 15:55:26 -0800 - rev 313252
Push 8060 by gijskruitbosch@gmail.com at Wed, 02 Dec 2015 10:24:00 +0000
Backed out changeset d821c903bd2b (bug 1171863) for frequent failures in browser_animation_timeline_pause_button.js
c9ebf1e96b757a883da862f80336d93208a70184: Bug 686281 - mask-composite reftests; r?dbaron draft
CJKu <cku@mozilla.com> - Wed, 02 Dec 2015 18:21:11 +0800 - rev 313251
Push 8059 by cku@mozilla.com at Wed, 02 Dec 2015 10:21:42 +0000
Bug 686281 - mask-composite reftests; r?dbaron
f2872ff214eaccdc03846f40d2abb62b793da0a9: Bug 686281 - A static assertion to keep value correctness of NS_RULE_NODE_IS_ANIMATION_RULE; r=dbaron. draft
CJKu <cku@mozilla.com> - Wed, 02 Dec 2015 18:21:11 +0800 - rev 313250
Push 8059 by cku@mozilla.com at Wed, 02 Dec 2015 10:21:42 +0000
Bug 686281 - A static assertion to keep value correctness of NS_RULE_NODE_IS_ANIMATION_RULE; r=dbaron.
4a576e59d32c6195f737ed41534c563304f4a358: Bug 686281 - Expands will-change of a shorthand prop to longhand ones; r=dbaron. draft
CJKu <cku@mozilla.com> - Wed, 02 Dec 2015 18:21:11 +0800 - rev 313249
Push 8059 by cku@mozilla.com at Wed, 02 Dec 2015 10:21:42 +0000
Bug 686281 - Expands will-change of a shorthand prop to longhand ones; r=dbaron.
756ddf2fb142918cc06999cc3cced716a3a47a24: Bug 686281 - Mask mochitest; r?dbaron. draft
CJKu <cku@mozilla.com> - Wed, 02 Dec 2015 18:21:10 +0800 - rev 313248
Push 8059 by cku@mozilla.com at Wed, 02 Dec 2015 10:21:42 +0000
Bug 686281 - Mask mochitest; r?dbaron.
61aa5b413ae017ef05350adcb5fdfded2d61d6e6: Bug 686281 - Remove nsStyleSVGReset::mMask; r?dbaron draft
CJKu <cku@mozilla.com> - Wed, 02 Dec 2015 18:21:10 +0800 - rev 313247
Push 8059 by cku@mozilla.com at Wed, 02 Dec 2015 10:21:42 +0000
Bug 686281 - Remove nsStyleSVGReset::mMask; r?dbaron
4bedf977cd2ed1af189605e6606b1ca44224620b: Bug 686281 - Mask CSS webkit-alias; r=dbaron. draft
CJKu <cku@mozilla.com> - Wed, 02 Dec 2015 18:21:10 +0800 - rev 313246
Push 8059 by cku@mozilla.com at Wed, 02 Dec 2015 10:21:42 +0000
Bug 686281 - Mask CSS webkit-alias; r=dbaron.
616ad6f639144a97f3e0169e0314b919911eddc0: Bug 686281 - Mask CSS animation; r=dbaron. draft
CJKu <cku@mozilla.com> - Wed, 02 Dec 2015 18:21:10 +0800 - rev 313245
Push 8059 by cku@mozilla.com at Wed, 02 Dec 2015 10:21:42 +0000
Bug 686281 - Mask CSS animation; r=dbaron.
a44454eeba2cb6c767eacccce2ba18613782b11b: Bug 686281 - Mask CSS rendering; r=mstange draft
CJKu <cku@mozilla.com> - Wed, 02 Dec 2015 18:21:10 +0800 - rev 313244
Push 8059 by cku@mozilla.com at Wed, 02 Dec 2015 10:21:42 +0000
Bug 686281 - Mask CSS rendering; r=mstange
87bbc1ed6a18a5c706e00b0a761e3dc24afc84b1: Bug 686281 - Mask CSS parsing and Mask DOM API. r?dbaron draft
CJKu <cku@mozilla.com> - Wed, 02 Dec 2015 18:21:10 +0800 - rev 313243
Push 8059 by cku@mozilla.com at Wed, 02 Dec 2015 10:21:42 +0000
Bug 686281 - Mask CSS parsing and Mask DOM API. r?dbaron
468055dd4541bcc6dcad068260598006216ebdae: Bug 686281 - Implement CSS mask style; r?dbaron. draft
CJKu <cku@mozilla.com> - Wed, 02 Dec 2015 18:21:10 +0800 - rev 313242
Push 8059 by cku@mozilla.com at Wed, 02 Dec 2015 10:21:42 +0000
Bug 686281 - Implement CSS mask style; r?dbaron.
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip