Bug 1368209 - Refactor `Engine::_processIncoming` into three stages. r=eoger,tcsc * In the first stage, we fetch changed records, newest first, up to the download limit. We keep track of the oldest record modified time we see. * Once we've fetched all records, we reconcile, noting records that fail to decrypt or reconcile for the next sync. We then ask the store to apply all remaining records. Previously, `applyIncomingBatchSize` specified how many records to apply at a time. I removed this because it added an extra layer of indirection that's no longer necessary, now that download batching buffers all records in memory, and all stores are async. * In the second stage, we fetch IDs for all remaining records changed between the last sync and the oldest modified time we saw in the first stage. We *don't* set the download limit here, to ensure we add *all* changed records to our backlog, and we use the `"oldest"` sort order instead of `"index"`. * In the third stage, we backfill as before. We don't want large deltas to delay other engines from syncing, so we still only take IDs up to the download limit from the backlog, and include failed IDs from the previous sync. On subsequent syncs, we'll keep fetching from the backlog until it's empty. Other changes to note in this patch: * `Collection::_rebuildURL` now allows callers to specify both `older` and `newer`. According to :rfkelly, this is explicitly and intentionally supported. * Tests that exercise `applyIncomingBatchSize` are gone, since that's no longer a thing. * The test server now shuffles records if the sort order is unspecified. MozReview-Commit-ID: 4EXvNOa8mIo

Bug 1368209 - Add a test for fetching backlogged history records. r=tcsc The test captures the existing logic in `_processIncoming`, even though it's not quite correct: * First, we fetch all records changed since the last sync, up to the download limit, and without an explicit sort order. This happens to work correctly now because the Python server uses "newest" by default, but can change in the future. * If we reached the download limit fetching records, we request IDs for all records changed since the last sync, also up to the download limit, and sorted by index. This is likely to return IDs for records we've already seen, since the index is based on the frecency. It's also likely to miss IDs for other changed records, because the number of changed records might be higher than the download limit. * Since we then fast-forward the last sync time, we'll never download any remaining changed records that we didn't add to our backlog. * Finally, we backfill previously failed and backlogged records. MozReview-Commit-ID: 7uQLXMseMIU

Bug 1368209 - Add support for `sortindex` and `older` to the mock Sync server. r=tcsc MozReview-Commit-ID: 6YI1OEE8x7R

Bug 1403358 Only reference ActivationContext if Acessibility is enabled r=aklotz This also enables the crash reporter on the MinGW build, as this is the only thing blocking that from working. MozReview-Commit-ID: Hygd7UUQvwl

Bug 1413680 - DrawTargetRecording should return the real size instead of the dummy drawtarget's size. r=jrmuizel MozReview-Commit-ID: E63f8ZIGYm2

Bug 1413397 - Avoid empty border's fallback. r=kats MozReview-Commit-ID: 1KXYjcObJdi

Bug 1407561 - Lazy loading of tooltip text when user hovers the status column. r=gasolin MozReview-Commit-ID: BXP8B10Q5LT

Bug 1414097 - Convert nsDisplaySelectionOverlay to WebRender. r=ethlin MozReview-Commit-ID: 5icpe4OR0Qa

Bug 1411808. P3 - InitAsClone() shouldn't call |mMediaCache->OpenStream(this)| until initialization is done. r=gerald We don't want MediaCache to use a half-initialized stream. MozReview-Commit-ID: LjPLOYwy0Wd

Bug 1411808. P2 - don't call mClient->IsSuspended() off the main thread in Update(). r=gerald By mirroring the suspend status of the client, Update() is able to make decisions on reading streams without calling mClient->IsSuspended(). MozReview-Commit-ID: G4gS2VGiMjj

Bug 1411808. P1 - run Update() loops off the main thread. r=gerald Note we will fix bugs required to run Update() off the main thread in the following patches. MozReview-Commit-ID: CYwT5kDjD9R

Bug 1413484 - tighten some assertions. r=gerald mCallback->GetMediaOwner() now is always called before Close() and won't return null. So is owner->GetMediaElement(). MozReview-Commit-ID: HbxbyuogYAB

Bug 1413829 - RequestListContent scroll to bottom should avoid sync reflow r=Honza MozReview-Commit-ID: KwPVFJfRQQK

Bug 1413941 - Fix broken netmonitor launchpad regression from bug 1360457 r=Honza MozReview-Commit-ID: 6EzLlNfQ4FX

Bug 1414073 - Rename arena_bin_t fields. r=njn At the same time, fold malloc_bin_stats_t into it.

Bug 1413858 - Resume() needs to reset mPendingSeekOffset before calling Seek(). r=gerald MozReview-Commit-ID: GbZ3SvSTMvR

servo: Merge #18750 - Make transforms generic (from Manishearth:transform-generic); r=emilio,xidorn This makes the specified and computed value of transform share a generic backing enum. This will eventually be a complete fix for https://bugzilla.mozilla.org/show_bug.cgi?id=1391145 , and also incidentally fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1405881 Currently WIP -- the generic transform exists and is used, but this currently misses some animation and glue cases. Source-Repo: https://github.com/servo/servo Source-Revision: c494d25e24d515509a5d8bb86a30669ee01742b9

Bug 1413648 fix default extension icon use, r=aswan MozReview-Commit-ID: 3Dh2DjGIAgn

merge mozilla-central to mozilla-inbound. r=merge a=merge

Bug 1333126 - Use win64 PE unwind metadata to improve client-side stack walking; r=gsvelto This includes tests that cover both regular CFI stack walking as well as pathological corner cases. MozReview-Commit-ID: GDARnPSemyu