[Invalidations] Require there to be no registered handlers on Invalidator destruction
Add CHECK to catch sloppy clients.
Make ProfileSyncService destroy its invalidator registrar on shut down (also to catch
sloppy clients). Comment on expected usage of Initialize() and Shutdown(),
and add DCHECKs for them.
Fix Invalidator test template to unregister handlers properly. Also fix some ProfileSyncService tests.
BUG=137086
TBR=tim@chromium.org,arv@chromium.org
Review URL: https://codereview.chromium.org/11046008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@160420 0039d316-1c4b-4281-b951-d872f2087c98
11 files changed
