searching for reviewer(mccr8)
0578378a87248e250d3f15fa3ae0943dc21de6a2: Bug 1353660 - Proxy destruction of nsHashPropertyBag's hash table to the main thread. r=mccr8, a=gchang
Nathan Froyd <froydnj@mozilla.com> - Wed, 05 Apr 2017 15:31:20 -0400 - rev 375788
Push 11027 by ryanvm@gmail.com at Thu, 06 Apr 2017 21:05:28 +0000
Bug 1353660 - Proxy destruction of nsHashPropertyBag's hash table to the main thread. r=mccr8, a=gchang We need this because the stored values in the hash table may themselves be main-thread only objects, and destroying them off the main thread will cause crashes.
3c4a6d36149c38695e0cdccd9e7bbe261d1e17c8: Bug 1335751 - Check all gray marking state before cycle collection in debug builds r=mccr8
Jon Coppeard <jcoppeard@mozilla.com> - Sun, 05 Mar 2017 09:23:33 +0000 - rev 375057
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1335751 - Check all gray marking state before cycle collection in debug builds r=mccr8
e9068fac39682ec1c776fbaf5e7eadb34f2580b8: Bug 1342181 - Run weak mapping fixup after GC before CC r=mccr8
Jon Coppeard <jcoppeard@mozilla.com> - Sun, 05 Mar 2017 09:19:24 +0000 - rev 375055
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1342181 - Run weak mapping fixup after GC before CC r=mccr8
5a927aeb8fb3d765371c8b24204187f57ed54a8e: Bug 1338623 - Add a slower but more exact gray marking check for checking correctness r=sfink r=mccr8
Jon Coppeard <jcoppeard@mozilla.com> - Thu, 02 Mar 2017 10:22:47 +0000 - rev 374628
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1338623 - Add a slower but more exact gray marking check for checking correctness r=sfink r=mccr8
ddef07d1ef6c7389726c63a165f89f70b33ae57b: Bug 1336467 - Don't blacken weakmap key unless both the delegate and the map are black r=mccr8
Jon Coppeard <jcoppeard@mozilla.com> - Thu, 16 Feb 2017 10:29:33 +0000 - rev 372306
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1336467 - Don't blacken weakmap key unless both the delegate and the map are black r=mccr8
33f49f751b63cde40ffe16d4c5fee0b4a64a6a79: Bug 1339591 - Rearrange construction order of RAII objects in SMIL code; r=mccr8
Brian Birtles <birtles@gmail.com> - Thu, 16 Feb 2017 15:36:04 +0900 - rev 372298
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1339591 - Rearrange construction order of RAII objects in SMIL code; r=mccr8 MozReview-Commit-ID: FXFF0Phvgjj
0f1afbc59a1cdd3012f1309bc64c0effaa9cae34: Bug 1339566: Acquire kungFuDeathGrip a bit earlier, in some nsDocShell code. r=mccr8
Daniel Holbert <dholbert@cs.stanford.edu> - Wed, 15 Feb 2017 09:56:38 -0800 - rev 372142
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1339566: Acquire kungFuDeathGrip a bit earlier, in some nsDocShell code. r=mccr8 MozReview-Commit-ID: AXTQqwyJPmU
ef98d3fdb36f644c7acec9bd0e89b95a91b48862: Bug 1337577 - Ensure that console listeners never touch web content, including in tests (r=mccr8)
Bill McCloskey <billm@mozilla.com> - Thu, 02 Feb 2017 12:59:20 -0800 - rev 371739
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1337577 - Ensure that console listeners never touch web content, including in tests (r=mccr8) MozReview-Commit-ID: 7UVaNUyj5yq
d031f49053c596b79ef8bc7bcd9a2f1aac021282: Bug 1337575 - Use SystemGroup for GC/CC (r=mccr8)
Bill McCloskey <billm@mozilla.com> - Wed, 01 Feb 2017 15:18:44 -0800 - rev 371738
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1337575 - Use SystemGroup for GC/CC (r=mccr8) MozReview-Commit-ID: 9vLuFnuHS9U
1526bb09ea8360bbb01e940ba8dc406070047ea7: bug 1331320 disable e10s browser_tab_dragdrop.js in remaining linux builds r=mccr8
Karl Tomlinson <karlt+@karlt.net> - Wed, 01 Feb 2017 16:13:43 +1300 - rev 370567
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
bug 1331320 disable e10s browser_tab_dragdrop.js in remaining linux builds r=mccr8 The intermittent abnormal plugin exit due to bug 1237853 was not detected by opt builds due to the use of exit(1) and lack of leak checking (which would have demonstrated bug 1309420 or bug 1239258). The abnormal exit is now detected in opt builds also, and so this change disables testing with these builds also. MozReview-Commit-ID: FnBGNqnJrDp
64a70b17ddeb893a69867053164f3a4b14933375: Bug 1334432 - Improve GC/CC logging in multi-e10s, r=mccr8
Olli Pettay <Olli.Pettay@helsinki.fi> - Thu, 02 Feb 2017 23:32:17 +0200 - rev 361382
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1334432 - Improve GC/CC logging in multi-e10s, r=mccr8
d17fabbdbe8ffb9ecc838da63fde255e436b054e: Bug 1110928, ensure zone GC collects the zone from which canvas context is originated, r=mccr8
Olli Pettay <Olli.Pettay@helsinki.fi> - Mon, 23 Jan 2017 19:26:37 +0200 - rev 361277
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1110928, ensure zone GC collects the zone from which canvas context is originated, r=mccr8
2aa9841bd6e3795e69d062a68290ce11ba0ffb96: Bug 1110928, trigger ZoneGC after evicting from bfcache, r=mccr8
Olli Pettay <Olli.Pettay@helsinki.fi> - Mon, 23 Jan 2017 19:25:13 +0200 - rev 361276
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1110928, trigger ZoneGC after evicting from bfcache, r=mccr8
bb0cdf573e9de3a08c3b08eb9ae7673ad05be6d0: Bug 1110928 trigger full GC only when closing top level outer window , r=mccr8
Olli Pettay <Olli.Pettay@helsinki.fi> - Mon, 23 Jan 2017 19:23:57 +0200 - rev 361275
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1110928 trigger full GC only when closing top level outer window , r=mccr8
7311c06a7271a8f33f4f73aa0c88e8d4806e488b: Bug 1324002 - Mark atoms separately in each zone, r=jonco,mccr8,peterv.
Brian Hackett <bhackett1024@gmail.com> - Mon, 30 Jan 2017 06:31:47 -0700 - rev 360728
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1324002 - Mark atoms separately in each zone, r=jonco,mccr8,peterv.
b87d9d7b10b5ec2552853af76f37ce2faa2fbade: Bug 1332322 part 3. Rename nsWrapperCache::IsBlackAndDoesNotNeedTracing to nsWrapperCache::HasKnownLiveWrapperAndDoesNotNeedTracing. r=mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Tue, 24 Jan 2017 20:39:37 -0500 - rev 359996
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1332322 part 3. Rename nsWrapperCache::IsBlackAndDoesNotNeedTracing to nsWrapperCache::HasKnownLiveWrapperAndDoesNotNeedTracing. r=mccr8
77087c94f9314d39ed9b7e3002659f5a6fdf49b6: Bug 1332322 part 2. Rename nsWrapperCache::IsBlack to nsWrapperCache::HasKnownLiveWrapper. r=mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Tue, 24 Jan 2017 20:38:58 -0500 - rev 359995
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1332322 part 2. Rename nsWrapperCache::IsBlack to nsWrapperCache::HasKnownLiveWrapper. r=mccr8
6b60e2ecd3e85c2fbf5339b8df2bd732a3d77867: Bug 1332322 part 1. Add an nsWrapperCache function to mark the wrapper as 'live' for GC purposes. r=mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Tue, 24 Jan 2017 20:33:54 -0500 - rev 359994
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1332322 part 1. Add an nsWrapperCache function to mark the wrapper as 'live' for GC purposes. r=mccr8
226f342c26fb55960ef220e4d8ce11940d3233a7: Bug 1332321. Make all cycle collection participants explicitly say whether they need skippability themselves, or just want whatever skippability their descendants want. r=mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Tue, 24 Jan 2017 16:11:56 -0500 - rev 359938
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1332321. Make all cycle collection participants explicitly say whether they need skippability themselves, or just want whatever skippability their descendants want. r=mccr8 This way we can't end up with a situation in which an ancestor doesn't care about skippability but a descendant does and doesn't get it, because the ancestor just claimed no skippability was needed.
97a60b61a65a93d1420245bcf4c4a3d5bcc911b6: Bug 1110928, ensure zone GC collects the zone from which canvas context is originated, r=mccr8
Olli Pettay <Olli.Pettay@helsinki.fi> - Mon, 23 Jan 2017 19:26:37 +0200 - rev 359813
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1110928, ensure zone GC collects the zone from which canvas context is originated, r=mccr8
5b62dda2ebd0f7bc46b0d0cbbb349852857f772c: Bug 1110928, trigger ZoneGC after evicting from bfcache, r=mccr8
Olli Pettay <Olli.Pettay@helsinki.fi> - Mon, 23 Jan 2017 19:25:13 +0200 - rev 359812
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1110928, trigger ZoneGC after evicting from bfcache, r=mccr8
b587547a6df17b9565bccfe7f6d77f2624b0896e: Bug 1110928 trigger full GC only when closing top level outer window , r=mccr8
Olli Pettay <Olli.Pettay@helsinki.fi> - Mon, 23 Jan 2017 19:23:57 +0200 - rev 359811
Push 10863 by jlorenzo@mozilla.com at Mon, 06 Mar 2017 23:02:23 +0000
Bug 1110928 trigger full GC only when closing top level outer window , r=mccr8
3707c5162a5aedee521dd9b1cc87aed40e00beb7: Bug 1336467 - Don't blacken weakmap key unless both the delegate and the map are black r=mccr8 a=jcristau
Jon Coppeard <jcoppeard@mozilla.com> - Thu, 16 Feb 2017 10:29:33 +0000 - rev 359378
Push 10783 by cbook@mozilla.com at Wed, 22 Feb 2017 14:58:56 +0000
Bug 1336467 - Don't blacken weakmap key unless both the delegate and the map are black r=mccr8 a=jcristau
2af32640ce76b13b41f2108652856907cc7dc623: Bug 1331320 - Disable e10s browser_tab_dragdrop.js in remaining linux builds. r=mccr8, a=jcristau
Karl Tomlinson <karlt+@karlt.net> - Wed, 01 Feb 2017 16:13:43 +1300 - rev 359306
Push 10760 by ryanvm@gmail.com at Fri, 17 Feb 2017 21:22:33 +0000
Bug 1331320 - Disable e10s browser_tab_dragdrop.js in remaining linux builds. r=mccr8, a=jcristau The intermittent abnormal plugin exit due to bug 1237853 was not detected by opt builds due to the use of exit(1) and lack of leak checking (which would have demonstrated bug 1309420 or bug 1239258). The abnormal exit is now detected in opt builds also, and so this change disables testing with these builds also. MozReview-Commit-ID: FnBGNqnJrDp
db2e6ff03ae1c57ebd4b4dae874d744bd1b8a41b: Bug 1339566: Acquire kungFuDeathGrip a bit earlier, in some nsDocShell code. r=mccr8 a=jcristau
Daniel Holbert <dholbert@cs.stanford.edu> - Fri, 17 Feb 2017 11:07:08 -0800 - rev 359284
Push 10758 by dholbert@mozilla.com at Fri, 17 Feb 2017 19:07:37 +0000
Bug 1339566: Acquire kungFuDeathGrip a bit earlier, in some nsDocShell code. r=mccr8 a=jcristau MozReview-Commit-ID: AXTQqwyJPmU
e2244b7749692299ee9977b17fddf9adf17d38e6: Bug 1339591 - Rearrange construction order of RAII objects in SMIL code; r=mccr8 a=gchang
Brian Birtles <birtles@gmail.com> - Thu, 16 Feb 2017 15:36:04 +0900 - rev 359280
Push 10756 by cbook@mozilla.com at Fri, 17 Feb 2017 15:59:56 +0000
Bug 1339591 - Rearrange construction order of RAII objects in SMIL code; r=mccr8 a=gchang MozReview-Commit-ID: FXFF0Phvgjj
538094a7900cdec31ad57faf9a6937104edce590: Bug 1332172 - Remove XPC_MAP_WANT_*. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Mon, 23 Jan 2017 13:33:58 +1100 - rev 358637
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1332172 - Remove XPC_MAP_WANT_*. r=mccr8. nsIXPCScriptable flags handling in xpc_map_end.h is a bit of a mess. - Half the flags relate to whether various functions are defined (PreCreate, GetProperty, etc). These are set using the XPC_MAP_WANT_* macros; for each one xpc_map_end.h inserts the corresponding flag using the preprocessor (see XPC_MAP_CLASSNAME::GetScriptableFlags()). - The other half of the flags relate to other things (IS_GLOBAL_OBJECT, DONT_REFLECT_INTERFACE_NAMES, etc). These are set using the XPC_MAP_FLAGS macro. Having two similar but different mechanisms to set the flags for a class is confusing. (Indeed, until recently we had some classes where a single flag was redundantly specified via both mechanisms.) Note also that the classes done in dom/base/nsIDOMClassInfo.h also specify all the flags in a single value, similar to how XPC_MAP_FLAGS works. This patch removes the XPC_MAP_WANT_* macros. All flags are now set via XPC_MAP_FLAGS. This is a significant simplification to xpc_map_end.h and all the places that use it. The downside of this change is that I had to change the flag constants from class constants (i.e. nsIXPCScriptable::FOO) to macros (i.e. NSIXPCSCRIPTABLE_FOO) because they need to be used in #if statements like this in xpc_map_end.h: #if !((XPC_MAP_FLAGS) & NSIXPCSCRIPTABLE_WANT_PRECREATE) and you can't use a '::'-qualified name inside a #if. I think this downside is outweighed by the simplification described above. Overall the patch removes 80 lines of code.
02bd976e715734b67a6c4c9e1300205c4e21f986: Bug 851892 part 12. Convert CSSStyleRule to WebIDL. r=peterv,heycam,mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Fri, 13 Jan 2017 10:41:03 -0500 - rev 358456
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 851892 part 12. Convert CSSStyleRule to WebIDL. r=peterv,heycam,mccr8 The .style PutForwards bit is coming along for the ride; I did check that Chrome already implements this, so we're not the only ones.
aa9330ed3a0791eea57ddb2e1e08fb38dc924cdd: Bug 851892 part 11. Convert CSSImportRule to WebIDL. r=peterv,heycam,mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Fri, 13 Jan 2017 10:41:03 -0500 - rev 358455
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 851892 part 11. Convert CSSImportRule to WebIDL. r=peterv,heycam,mccr8 Note that the .media PutForwards is a new feature coming along for the ride now that we're using the spec IDL.
31103a03f2c293494dd2bd0bae3583b795a92df3: Bug 851892 part 12. Convert CSSStyleRule to WebIDL. r=peterv,heycam,mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Fri, 13 Jan 2017 10:41:03 -0500 - rev 358415
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 851892 part 12. Convert CSSStyleRule to WebIDL. r=peterv,heycam,mccr8 The .style PutForwards bit is coming along for the ride; I did check that Chrome already implements this, so we're not the only ones.
eab8ccda41b81f0cfad58d850745c1cc7d1fe88d: Bug 851892 part 11. Convert CSSImportRule to WebIDL. r=peterv,heycam,mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Fri, 13 Jan 2017 10:41:03 -0500 - rev 358414
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 851892 part 11. Convert CSSImportRule to WebIDL. r=peterv,heycam,mccr8 Note that the .media PutForwards is a new feature coming along for the ride now that we're using the spec IDL.
689462b3c0c93c331025bd62c2c93472663b0113: Bug 1332531. Improve the documentation for cycle collector skippability bits. r=smaug,mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Fri, 20 Jan 2017 12:59:15 -0500 - rev 358403
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1332531. Improve the documentation for cycle collector skippability bits. r=smaug,mccr8
a63edff17c8f61d797f6c1defb77746ea6840249: Bug 1330904 - Remove XPCNativeScriptableCreateInfo. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 13 Jan 2017 11:26:02 +1100 - rev 358121
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1330904 - Remove XPCNativeScriptableCreateInfo. r=mccr8. XPCNativeScriptableCreateInfo is now a very thin wrapper around nsIXPCScriptable. Removing it simplifies things quite a bit. Note especially the change to GatherScriptableCreateInfo(), which is a confusing function. Previously its fourth argument was never touched it was called, but it did have a return value. Now the fourth argument is touched and effectively replaces that return value, and the function now returns void. * * * Bug 1330904 - fixups. r=mccr8. I will fold this into the previous patch before landing.
27c422b6b825c6889e4ecb17737d5c16ef9fe859: Bug 1273251: Part 4 - Drop CallbackObject's JS objects for nuked compartments during CC. r=peterv,mccr8
Kris Maglione <maglione.k@gmail.com> - Mon, 14 Nov 2016 19:49:28 -0800 - rev 358077
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1273251: Part 4 - Drop CallbackObject's JS objects for nuked compartments during CC. r=peterv,mccr8 MozReview-Commit-ID: 6lPdmUtKREt
3782cf885346adb952de0f720bb48b7ea9557ed9: Bug 1273251: Part 2 - Mark extension window compartments as nuked, and nuke all wrappers. r=mccr8
Kris Maglione <maglione.k@gmail.com> - Mon, 14 Nov 2016 08:05:00 -0800 - rev 358075
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1273251: Part 2 - Mark extension window compartments as nuked, and nuke all wrappers. r=mccr8 MozReview-Commit-ID: E0ImYZIgEjU
a8da3c34983f2c673715440dfe8ad615f4c61189: Bug 851892 part 12. Convert CSSStyleRule to WebIDL. r=peterv,heycam,mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Fri, 13 Jan 2017 10:41:03 -0500 - rev 357459
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 851892 part 12. Convert CSSStyleRule to WebIDL. r=peterv,heycam,mccr8 The .style PutForwards bit is coming along for the ride; I did check that Chrome already implements this, so we're not the only ones.
29cf8acbd21eaeb1a8ddc5eb452f2c02e14beaf8: Bug 851892 part 11. Convert CSSImportRule to WebIDL. r=peterv,heycam,mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Fri, 13 Jan 2017 10:41:03 -0500 - rev 357458
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 851892 part 11. Convert CSSImportRule to WebIDL. r=peterv,heycam,mccr8 Note that the .media PutForwards is a new feature coming along for the ride now that we're using the spec IDL.
4e48a59adc5ebc39d93d928ab35d10bbb9e65795: Bug 851892 part 12. Convert CSSStyleRule to WebIDL. r=peterv,heycam,mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Fri, 13 Jan 2017 10:41:03 -0500 - rev 357362
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 851892 part 12. Convert CSSStyleRule to WebIDL. r=peterv,heycam,mccr8 The .style PutForwards bit is coming along for the ride; I did check that Chrome already implements this, so we're not the only ones.
2a5a9437777ffd046070cdaef8874be6d3e87c95: Bug 851892 part 11. Convert CSSImportRule to WebIDL. r=peterv,heycam,mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Fri, 13 Jan 2017 10:41:03 -0500 - rev 357361
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 851892 part 11. Convert CSSImportRule to WebIDL. r=peterv,heycam,mccr8 Note that the .media PutForwards is a new feature coming along for the ride now that we're using the spec IDL.
18ff0b6063121d502dd610560512ff91f46c97e9: Bug 1329846 (part 2) - Remove NATIVE_HAS_FLAG. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Wed, 11 Jan 2017 07:40:19 +1100 - rev 357243
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1329846 (part 2) - Remove NATIVE_HAS_FLAG. r=mccr8. It's only used in three places, and it no longer makes the code more readable.
105d7762f4b06db69cef6fffd2129890dd9fbbb0: Bug 1329846 (part 1) - Remove XPCNativeScriptableInfo. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Tue, 10 Jan 2017 12:47:57 +1100 - rev 357242
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1329846 (part 1) - Remove XPCNativeScriptableInfo. r=mccr8. XPCNativeScriptableInfo is now a very thin wrapper around nsIXPCScriptable, and it uses manual memory management. Removing it simplifies things quite a bit. In particular, when setting XPCWrappedNative::mScriptable in XPCWrappedNative::WrapNewGlobal() and XPCWrappedNative::Init() we no longer have to worry about sharing the XPCNativeScriptableInfo object with the proto. And XPCWrappedNative::{Init,Destroy}() have similar simplifications.
2e1a0c4886c5393498627d394c4f661e6f2ea6b2: Bug 1325406 - Refactor rooting base class templates r=sfink r=mccr8
Jon Coppeard <jcoppeard@mozilla.com> - Tue, 10 Jan 2017 10:12:14 +0000 - rev 356769
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1325406 - Refactor rooting base class templates r=sfink r=mccr8
42c1ba6cb5f400c7e4a8a68fe11713de6a69ba38: Bug 1324330 (part 6) - Streamline nsIXPCScriptable flags. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 23 Dec 2016 08:30:45 +1100 - rev 356697
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1324330 (part 6) - Streamline nsIXPCScriptable flags. r=mccr8. This patch adjusts the values so there are no skipped bits, and removes the no-longer-used RESERVED flag.
4e2cfc2d396fd43d2a825b527656a60247912414: Bug 1325542 - Remove XPCNativeScriptableFlags. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 23 Dec 2016 16:14:32 +1100 - rev 356696
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1325542 - Remove XPCNativeScriptableFlags. r=mccr8. WantFoo() and the similar methods remain, but they've been moved from XPCNativeScriptableFlags to nsIXPCScriptable. One consequence of this change is that in places where we used to get the flags from an XPCNativeScriptableCreateInfo we now need a null check on the nsIXPCScriptable. (This isn't true when getting flags from XPCNativeScriptableInfo, however, because nsIXPCScriptable is always non-null within that type.)
52e2571353b8ba1f023f1684662ece5f4f3ed785: Bug 1324330 (part 5) - Remove nsDOMClassInfo::mName. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 23 Dec 2016 08:30:31 +1100 - rev 356531
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1324330 (part 5) - Remove nsDOMClassInfo::mName. r=mccr8. It's identical to the mClass.name.
3a2066a8e8a822217907a86b0c10227a9214557d: Bug 1324330 (part 4) - Remove some unused functions. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 23 Dec 2016 08:30:29 +1100 - rev 356530
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1324330 (part 4) - Remove some unused functions. r=mccr8.
a193a2da927441688192f9ee07db640e51d0120b: Bug 1324330 (part 3) - Don't mix |Cant| and |Cannot| in function names in XPCWrappedNativeJSOps.cpp. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 23 Dec 2016 08:30:27 +1100 - rev 356529
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1324330 (part 3) - Don't mix |Cant| and |Cannot| in function names in XPCWrappedNativeJSOps.cpp. r=mccr8. Use |Cannot| consistently.
5b05e5a264b4a5e8e8aabdcf7198bf541f618cb4: Bug 1324330 (part 2) - Make XPC_MAP_FLAGS non-optional when using xpc_map_end.h. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Fri, 23 Dec 2016 08:30:22 +1100 - rev 356528
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1324330 (part 2) - Make XPC_MAP_FLAGS non-optional when using xpc_map_end.h. r=mccr8. I'd like to remove XPC_MAP_WANT_* altogether and use XPC_MAP_FLAGS for all the nsIXPCScriptable flag setting, but I haven't worked out how to handle the method definitions in xpc_map_end.h yet. In the meantime, it seems good to make the flag setting more consistent and explicit. In particular, the three "Module" classes have "#define XPC_MAP_WANT_CALL" and "#define XPC_MAP_FLAGS nsIXPCScriptable::WANT_CALL" which both have the same effect. The patch removes the latter, to make them consistent with other classes.
a8921eeffbb3e0b4fe67041281b5e23f92eab270: Bug 1324330 (part 1) - Mark js::Class and related types as NONHEAP. r=mccr8.
Nicholas Nethercote <nnethercote@mozilla.com> - Mon, 19 Dec 2016 14:18:41 +1100 - rev 356527
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1324330 (part 1) - Mark js::Class and related types as NONHEAP. r=mccr8.
50c889c86cb89df0f39514e67ed87d5df29f5f9b: Bug 1326301. Put the global allocated by SimpleGlobalObject in the system zone, so we don't create tons of zones when we're converting JSON to dictionaries many times over. r=mccr8
Boris Zbarsky <bzbarsky@mit.edu> - Wed, 04 Jan 2017 14:52:26 -0500 - rev 355994
Push 10621 by jlund@mozilla.com at Mon, 23 Jan 2017 16:02:43 +0000
Bug 1326301. Put the global allocated by SimpleGlobalObject in the system zone, so we don't create tons of zones when we're converting JSON to dictionaries many times over. r=mccr8