Bug 1402284 - Move arena tree related globals to a static singleton of a new class. r?njn We create the ArenaCollection class to handle operations on the arena tree. Ideally, iter() would trigger locking, but the prefork/postfork code complicates things, so we leave this for later.

Bug 1402284 - Separate arenas created from moz_arena_* functions from others. r?njn We introduce the notion of private arenas, separate from other arenas (main and thread-local). They are kept in a separate arena tree, and arena lookups from moz_arena_* functions only access the tree of private arenas. Iteration still goes through all arenas, private and non-private.

Bug 1402284 - Move arena tree related globals to a static singleton of a new class. r?njn We create the arena_t::Collection class to handle operations on the arena tree. Ideally, iter() would trigger locking, but the prefork/postfork code complicates things, so we leave this for later.

Bug 1402284 - Initialize arena_t objects via a constructor instead of manually with an Init method. r?njn Note we use the deprecated `new (fallible_t())` form because using `new (fallible)` would require some figuring out for non-Firefox builds (e.g. standalone js).

Bug 1402284 - Make RedBlackTree::{Insert,Remove} work when the type has a constructor. r?njn RedBlackTree::{Insert,Remove} allocate an object on the stack for its RedBlackTreeNode, and that shouldn't have side effects if the type happens to have a constructor. This will allow to add constructors to some of the mozjemalloc types.

Bug 1413096 - Remove SIZEOF_PTR and SIZEOF_PTR_2POW. r?njn

Bug 1413096 - Replace ffs with MathAlgorithms functions. r?njn - In the cases where it's used on powers of 2, replace it with FloorLog2() + 1. - In the cases where it's used on any kind of number, replace it with CountTrailingZeroes, which is `ffs(x) - 1`. - In the case of tiny allocations in arena_t::MallocSmall, we rearrange the code so that the intent is clearer, which also simplifies the expression for the mBins offset: mBins[0] is the first tiny bucket, for allocations of sizes 1 << TINY_MIN_2POW, mBins[1] for allocations of size 1 << (TINY_MIN_2POW + 1), etc. up to small_min. So the offset is really the log2 of the normalized size.

Bug 1413096 - Remove unnecessary call to ffs. r?njn Comparing ffs(x) == ffs(y), when x and y are guaranteed to be powers of 2 (or 0, or 1), is the same as x == y.

Bug 1413096 - Remove pow2_ceil in favor of RoundUpPow2. r?njn

Bug 1413096 - Use CheckedInt for overflow check in calloc. r?njn Also use in in _recalloc.

Bug 1413096 - Add "using namespace mozilla" to mozjemalloc.cpp. r?njn

Bug 1413096 - Add missing header guard. r?njn

Bug 1412722 - Remove RedBlackTree sentinel. r?njn The sentinel was taking as much space as one element of the tree, while only really used for its RedBlackTreeNode, wasting space. This results in some decrease in struct sizes, for example on 64-bits linux: - arena_bin_t: 80 -> 56 - arena_t (excluding mBins): 224 -> 144 - arena_t + dynamic size of mBins: 3024 -> 2104 It also decreases the size of several globals: - gChunksBySize, gChunksByAddress, huge: 64 -> 8 - gArenaTree: 312 -> 8

Bug 1411855 - remove unused and no inline function in toolbar;r=rickychien MozReview-Commit-ID: HXiTJSskRPr

Bug 1412437 - Remove nsIDOMHTMLTextAreaElement MozReview-Commit-ID: JAON7Rd7IAo

Bug 1412761 - copy editing;r=fischer MozReview-Commit-ID: LDZFo94cMNZ

Bug 1412761 - remove dup test case;r=fischer MozReview-Commit-ID: 9MLGQI3SiIf

Bug 1413111 - Use nsTSubstring::Split in nsCSSRuleProcessor::LangPseudoMatches instead of manually go through with FindChar.

Bug 1413111 - Use nsTSubstring::Split in nsCSSRuleProcessor::LangPseudoMatches instead of manually go through with FindChar.

Bug 1405506: Flush buffered data when disconnecting suspended channel. r?mixedpuppy MozReview-Commit-ID: F9rIhQHr8i