зеркало из https://github.com/mozilla/gecko-dev.git
064ed56076
* 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 --HG-- extra : rebase_source : f382f0a883c5aa1f6a4466fefe22ad1a88ab6d20 |
||
|---|---|---|
| .. | ||
| tests | ||
| BookmarkHTMLUtils.jsm | ||
| BookmarkJSONUtils.jsm | ||
| Bookmarks.jsm | ||
| ClusterLib.js | ||
| ColorAnalyzer.js | ||
| ColorAnalyzer_worker.js | ||
| ColorConversion.js | ||
| Database.cpp | ||
| Database.h | ||
| ExtensionSearchHandler.jsm | ||
| FaviconHelpers.cpp | ||
| FaviconHelpers.h | ||
| Helpers.cpp | ||
| Helpers.h | ||
| History.cpp | ||
| History.h | ||
| History.jsm | ||
| PageIconProtocolHandler.js | ||
| PlaceInfo.cpp | ||
| PlaceInfo.h | ||
| PlacesBackups.jsm | ||
| PlacesCategoriesStarter.js | ||
| PlacesDBUtils.jsm | ||
| PlacesRemoteTabsAutocompleteProvider.jsm | ||
| PlacesSearchAutocompleteProvider.jsm | ||
| PlacesSyncUtils.jsm | ||
| PlacesTransactions.jsm | ||
| PlacesUtils.jsm | ||
| SQLFunctions.cpp | ||
| SQLFunctions.h | ||
| Shutdown.cpp | ||
| Shutdown.h | ||
| UnifiedComplete.js | ||
| VisitInfo.cpp | ||
| VisitInfo.h | ||
| jar.mn | ||
| moz.build | ||
| mozIAsyncFavicons.idl | ||
| mozIAsyncHistory.idl | ||
| mozIAsyncLivemarks.idl | ||
| mozIColorAnalyzer.idl | ||
| mozIPlacesAutoComplete.idl | ||
| mozIPlacesPendingOperation.idl | ||
| nsAnnoProtocolHandler.cpp | ||
| nsAnnoProtocolHandler.h | ||
| nsAnnotationService.cpp | ||
| nsAnnotationService.h | ||
| nsFaviconService.cpp | ||
| nsFaviconService.h | ||
| nsIAnnotationService.idl | ||
| nsIBrowserHistory.idl | ||
| nsIFaviconService.idl | ||
| nsINavBookmarksService.idl | ||
| nsINavHistoryService.idl | ||
| nsITaggingService.idl | ||
| nsLivemarkService.js | ||
| nsMaybeWeakPtr.h | ||
| nsNavBookmarks.cpp | ||
| nsNavBookmarks.h | ||
| nsNavHistory.cpp | ||
| nsNavHistory.h | ||
| nsNavHistoryQuery.cpp | ||
| nsNavHistoryQuery.h | ||
| nsNavHistoryResult.cpp | ||
| nsNavHistoryResult.h | ||
| nsPIPlacesDatabase.idl | ||
| nsPlacesExpiration.js | ||
| nsPlacesIndexes.h | ||
| nsPlacesMacros.h | ||
| nsPlacesModule.cpp | ||
| nsPlacesTables.h | ||
| nsPlacesTriggers.h | ||
| nsTaggingService.js | ||
| toolkitplaces.manifest | ||
| unifiedcomplete-top-urls.json | ||