gecko-dev/browser/components/places
Marco Bonardo 77808a4218 Bug 1244650 - Failure to clear Forms and Search Data on exit. r=yoric
The problem is due to sanitization happening too late in the shutdown cycle.
The Sanitizer depends on Places shutdown, that recently moved to async shutdown.
That change caused shutdown to happen completely at profile-before-change, unfortunately
during that phase it's impossible to predict which services are already shutdown.
The patch restores the previous Places shutdown procedure, thus clients are notified
earlier, during profile-change-teardown.

Additional meaningful changes:
* Fixes FX_SANITIZE_TOTAL telemetry to properly count total time taken by sanitize.
* Makes each cleanup operation isolated from other errors to try cleaning up as most as possible.
* In case of multiple sanitization sub steps, each step is isolated by a try/catch, the last seen exception is reported upstream.
* Makes FX_SANITIZE_HISTORY actually measure history, not other random stuff.
* Removes TOPIC_SIMULATE_PLACES_MUST_CLOSE_1 since we can now just use profile-change-teardown for shutdown phase 1.

MozReview-Commit-ID: HroLvbi25IC

--HG--
extra : rebase_source : 5cd01a803fbc1f72dc5174dd0c5a5b5aea473fc2
2016-02-04 13:51:34 +01:00
..
content Bug 1245649: Enable no-nested-ternary. r=mconley 2016-02-03 21:17:16 -08:00
tests Bug 1244650 - Failure to clear Forms and Search Data on exit. r=yoric 2016-02-04 13:51:34 +01:00
jar.mn Bug 1188787 - Use AppConstants.jsm instead of preprocessing in browser/components/places. r=mak 2015-08-28 07:37:00 -04:00
moz.build Bug 1188787 - Use AppConstants.jsm instead of preprocessing in browser/components/places. r=mak 2015-08-28 07:37:00 -04:00
PlacesUIUtils.jsm Bug 1201331 (part 4) - replace "Tabs from other devices" with "Synced Tabs" on menus and history panel. r=Gijs 2015-12-03 10:05:03 +11:00