d8da9bc56468f58be505e61e72e78bff1d2b4b61: Bug 1249367 - Make background finalization a GC phase (and clean up Zones properly); r=jonco
Terrence Cole <terrence@mozilla.com> - Mon, 22 Feb 2016 08:51:50 -0800 - rev 334291
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249367 - Make background finalization a GC phase (and clean up Zones properly); r=jonco
5a1a5726b179155823013cd617738540753eb4de: Bug 1249389 - followup - fix compilation bustage in gfxFT2FontList.cpp; r=me
Nathan Froyd <froydnj@mozilla.com> - Wed, 24 Feb 2016 15:48:35 -0500 - rev 334290
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249389 - followup - fix compilation bustage in gfxFT2FontList.cpp; r=me
3d625cdac1382caf11bb3118585ddc2ad199fe5c: Bug 1243231 - Implement a taskcluster-based b2g hazard build, r=garndt,terrence
Steve Fink <sfink@mozilla.com> - Mon, 22 Feb 2016 17:09:02 -0800 - rev 334289
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1243231 - Implement a taskcluster-based b2g hazard build, r=garndt,terrence
4121f9b10c82af02598ccca3a688a67531440b64: Bug 1249389 - part 7 - clean up calls to GetBuffer in TestStartupCache; r=erahm
Nathan Froyd <froydnj@mozilla.com> - Thu, 18 Feb 2016 14:57:14 -0500 - rev 334288
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249389 - part 7 - clean up calls to GetBuffer in TestStartupCache; r=erahm With the previous patch, we're able to eliminate some raw pointer usages in TestStartupCache, which is always nice.
2a3bc10a643708d268364d928c2342bb5930f8d7: Bug 1249389 - part 6 - provide UniquePtr overload for nsIStartupCache::GetBuffer; r=erahm
Nathan Froyd <froydnj@mozilla.com> - Thu, 18 Feb 2016 14:35:21 -0500 - rev 334287
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249389 - part 6 - provide UniquePtr overload for nsIStartupCache::GetBuffer; r=erahm The lone remaining startup cache-related uses of nsAutoArrayPtr are both in TestStartupCache.cpp, for use with nsIStartupCache::GetBuffer. The uses can't use StartupCache::GetBuffer because StartupCache::GetBuffer isn't visible outside of libxul, and TestStartupCache is a normal C++ unit test. The Right Thing is to convert TestStartupCache to a gtest so we can see libxul internal symbols and then delete nsIStartupCache entirely. That's a bit complicated, as TestStartupCache doesn't fit nicely into gtest's framework. The simpler solution is to add a UniquePtr overload in the interface that hides the XPCOM outparam management details.
bcfb36dfee0fea8de1555b658b6b04d30ccad263: Bug 1249389 - part 5 - use UniquePtr instead of nsAutoArrayPtr in mozilla::scache::CacheEntry; r=erahm
Nathan Froyd <froydnj@mozilla.com> - Thu, 18 Feb 2016 14:14:28 -0500 - rev 334286
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249389 - part 5 - use UniquePtr instead of nsAutoArrayPtr in mozilla::scache::CacheEntry; r=erahm
50332bf18a2fd8f6be9b69fb0e2482da8d528987: Bug 1249389 - part 4 - make StartupCache::GetBuffer take a UniquePtr outparam; r=erahm
Nathan Froyd <froydnj@mozilla.com> - Thu, 18 Feb 2016 12:26:28 -0500 - rev 334285
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249389 - part 4 - make StartupCache::GetBuffer take a UniquePtr outparam; r=erahm This change eliminates a number of nsAutoArrayPtr usages, as well as making the pattern GetBuffer() -> NewObjectInputStreamFromBuffer more pleasant.
1f79d41d9098733aefe7da16c690ec4624ce4e7b: Bug 1249389 - part 3 - assert the size of nsZipItemPtr's storage type; r=erahm
Nathan Froyd <froydnj@mozilla.com> - Thu, 18 Feb 2016 12:07:03 -0500 - rev 334284
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249389 - part 3 - assert the size of nsZipItemPtr's storage type; r=erahm The only uses of this class use a template argument with a size of |char| (uint8_t and char), and the class isn't designed to accomodate template arguments of larger size (e.g. the implementation of Forget() neglects to divide by sizeof(T) for allocating a return buffer). Let's enforce this with a static_assert. This change makes the class safer to use and also makes future changes simpler to reason about.
14e29cda9552844c1bf593df50ab85c555d345b3: Bug 1249389 - part 2 - change NewBufferFromStorageStream's outparam into a UniquePtr; r=erahm
Nathan Froyd <froydnj@mozilla.com> - Thu, 18 Feb 2016 12:04:40 -0500 - rev 334283
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249389 - part 2 - change NewBufferFromStorageStream's outparam into a UniquePtr; r=erahm Similar to the previous change to NewObjectInputStreamFromBuffer, we want to make the ownership transfer out of NewBufferFromStorageStream more obvious. Doing this also lets us get rid of some uses of nsAutoArrayPtr, which is less idiomatic than UniquePtr.
b688e63090589bcee8766ea557f1a74682c020d5: Bug 1249389 - part 1 - change NewObjectInputStreamFromBuffer to take a UniquePtr argument; r=erahm
Nathan Froyd <froydnj@mozilla.com> - Thu, 18 Feb 2016 11:14:02 -0500 - rev 334282
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249389 - part 1 - change NewObjectInputStreamFromBuffer to take a UniquePtr argument; r=erahm Because NewObjectInputStreamFromBuffer takes a raw pointer as input, the typical coding pattern to use it is: nsAutoArrayPtr<char> buf; // assign something to buf nsresult rv = NewObjectInputStreamFromBuffer(buf, ...); if (NS_FAILED(rv)) { ... return rv; } buf.forget(); Which is clumsy, error-prone, and obscures the ownership transfer of the pointer into the stream returned by NewObjectInputStreamFromBuffer. Let's address all of these concerns by changing the argument to a UniquePtr<char[]>.
615be94be0865921a73a7d2f81fbdb6bb1640bc6: Bug 1249389 - part 0 - use getter_Transfers in TestStartupCache.cpp; r=erahm
Nathan Froyd <froydnj@mozilla.com> - Thu, 18 Feb 2016 11:22:12 -0500 - rev 334281
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249389 - part 0 - use getter_Transfers in TestStartupCache.cpp; r=erahm TestWriteObject() in TestStartupCache.cpp uses this odd pattern of acquiring a raw pointer from the startup cache, and then stashing that raw pointer into an nsAutoArrayPtr. We can do better by using the getter_Transfers idiom and thereby always using the smart pointer.
6104903f75c5f154cc6c5e8d89dbf0a279284e59: Bug 1246215 - Console prevents let re-declaration even when first was an error; r=jryans,shu
Morgan Phillips <winter2718@gmail.com> - Wed, 24 Feb 2016 12:16:16 -0600 - rev 334280
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1246215 - Console prevents let re-declaration even when first was an error; r=jryans,shu
2bab88e97af5f29f32861135af8481412ee50499: Bug 1201667 - Traverse child when retrieving proxy unignored children r=tbsaunde
Lorien Hu <lorien@lorienhu.com> - Thu, 03 Sep 2015 13:02:07 -0700 - rev 334279
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1201667 - Traverse child when retrieving proxy unignored children r=tbsaunde
ea3d2ad1b20c410a5d1aed593f46280f4baba261: Bug 1250650 - disable mozpayment tests android. r=fabrice
Joel Maher <jmaher@mozilla.com> - Tue, 23 Feb 2016 12:55:10 -0800 - rev 334278
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1250650 - disable mozpayment tests android. r=fabrice MozReview-Commit-ID: 5VhJtaK6ksH
002d7897c93c8892ded59abb3e408541b111f57d: Bug 1249689 - generate and provide a Symbol for each add-on on startup r=mossop
Robert Helmer <rhelmer@mozilla.com> - Mon, 11 Jan 2016 15:35:19 -0800 - rev 334277
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249689 - generate and provide a Symbol for each add-on on startup r=mossop MozReview-Commit-ID: LoPGSrJtlkr
93390f1d1faa6a133d5ed53629df227030304198: Bug 1249689 - replace bootstrapScope with an activeAddons Map() that contains it r=mossop
Robert Helmer <rhelmer@mozilla.com> - Mon, 22 Feb 2016 17:39:21 -0800 - rev 334276
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1249689 - replace bootstrapScope with an activeAddons Map() that contains it r=mossop MozReview-Commit-ID: 4s3c5OOQYLv
07f601e5ce85d64f01434cacd3361c21b5e73e33: backout c42ff6847631 (bug 1250572) for causing aborts during startup
Daniel Holbert <dholbert@cs.stanford.edu> - Wed, 24 Feb 2016 11:54:35 -0800 - rev 334275
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
backout c42ff6847631 (bug 1250572) for causing aborts during startup
84ba22179d9775a7b79e78df2a9e099ba18ebaac: Bug 1187450 - avoid leaking cstr in SPSProfiler::allocProfileString. r=jorendorff
Bogdan Postelnicu <bogdan.postelnicu@softvision.ro> - Tue, 23 Feb 2016 11:12:35 +0200 - rev 334274
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1187450 - avoid leaking cstr in SPSProfiler::allocProfileString. r=jorendorff MozReview-Commit-ID: GtBKL4M2hfm
6ba4a380e9020f75cf016ca358c3982031385434: Bug 1243623. Don't skip unregistering a table part if we have a split table. r=mats
Robert O'Callahan <robert@ocallahan.org> - Wed, 10 Feb 2016 12:18:55 +1300 - rev 334273
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1243623. Don't skip unregistering a table part if we have a split table. r=mats MozReview-Commit-ID: 2GHprw8YGsx
5031e3d7f2dcfe7388c73539b7e47e38990b6107: Bug 1243796 - Tracking bug for Mar-7-2016 migration work r=jlund DONTBUILD
Rail Aliiev <rail@mozilla.com> - Wed, 24 Feb 2016 11:42:32 -0800 - rev 334272
Push 11499 by bmo:mh+mozilla@glandium.org at Wed, 24 Feb 2016 21:35:36 +0000
Bug 1243796 - Tracking bug for Mar-7-2016 migration work r=jlund DONTBUILD
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 +300000 tip