Fix InvalidationNotifierTest leak and suppressions

tools/heapcheck/suppressions.txt

 # There are three kinds of suppressions in this file.
 # 1. third party stuff we have no control over
 #
 # 2. intentional unit test errors, or stuff that is somehow a false positive
 # in our own code, or stuff that is so trivial it's not worth fixing
 #
 # 3. Suppressions for real chromium bugs that are not yet fixed.
 # These should all be in chromium's bug tracking system (but a few aren't yet).
 # Periodically we should sweep this file and the bug tracker clean by
 # running overnight and removing outdated bugs/suppressions.
@@ skipping 1094 matching lines @@
 {
    bug_79568
    Heapcheck:Leak
    ...
    fun:browser_sync::SyncerThread::HandleConsecutiveContinuationError
    fun:browser_sync::SyncerThread::ScheduleNextSync
    fun:browser_sync::SyncerThread::FinishSyncSessionJob
    fun:browser_sync::SyncerThread::DoSyncSessionJob
 }
 {
-   bug_79651_a
-   Heapcheck:Leak
-   ...
-   fun:sync_notifier::InvalidationNotifier::UpdateCredentials
-   fun:sync_notifier::::InvalidationNotifierTest_Basic_Test::TestBody
-}
-{
-   bug_79651_b
-   Heapcheck:Leak
-   ...
-   fun:*basic_string*
-   fun:sync_notifier::::InvalidationNotifierTest_Basic_Test::TestBody
-}
-{
    bug_80196
    Heapcheck:Leak
    fun:CallbackRunner* NewCallback
    fun:browser_sync::SyncBackendHost::ConfigureDataTypes
    fun:browser_sync::SyncBackendHostForProfileSyncTest::ConfigureDataTypes
    fun:browser_sync::DataTypeManagerImpl::Restart
    fun:browser_sync::DataTypeManagerImpl::Configure
 }
 {
-    bug_80238
-    Heapcheck:Leak
-    ...
-    fun:sync_notifier::::InvalidationNotifierTest::SetUp
-}
-{
    bug_80467
    Heapcheck:Leak
    ...
    fun:ExtensionService::RecordPermissionMessagesHistogram
 }
 {
    bug_80472
    Heapcheck:Leak
    ...
    fun:fileapi::::FileSystemURLRequestJobTest::SetUp
@@ skipping 10 matching lines @@
    ...
    fun:RenderWidgetHost::WasRestored
 }
 {
    bug_80550_b
    Heapcheck:Leak
    ...
    fun:RenderWidgetHost::WasHidden
 }
 {
-   bug_79651
-   Heapcheck:Leak
-   ...
-   fun:TestURLRequestContextGetter::GetURLRequestContext
-   fun:*SingleLoginAttempt
-   fun:notifier::Login::StartConnection
-   fun:sync_notifier::InvalidationNotifier::UpdateCredentials
-   fun:sync_notifier::NonBlockingInvalidationNotifier::Core::UpdateCredentials
-   fun:void DispatchToMethod
-}
-{
    bug_80654
    Heapcheck:Leak
    fun:__gnu_cxx::new_allocator::allocate
    fun:std::_Vector_base::_M_allocate
    fun:std::vector::_M_insert_aux
    fun:std::vector::push_back
    fun:ObserverListBase::AddObserver
    fun:net::SSLConfigService::AddObserver
    fun:*SSLClientSocketPool
    fun:*ClientSocketPoolManager
@@ skipping 456 matching lines @@
    fun:__gnu_cxx::new_allocator::allocate
    fun:std::_Vector_base::_M_allocate
    fun:std::vector::_M_insert_aux
    fun:std::vector::push_back
    fun:ScopedVector::push_back
    fun:browser_sync::AutofillProfileSyncableService::CreateOrUpdateProfile
    fun:browser_sync::AutofillProfileSyncableService::MergeDataAndStartSyncing
    fun:browser_sync::SyncableServiceAdapter::AssociateModels
    fun:browser_sync::NonFrontendDataTypeController::StartAssociation
 }
+{
+   bug_96904
+   Heapcheck:Leak
+   ...
+   fun:TestURLRequestContextGetter::GetURLRequestContext
+   fun:*SingleLoginAttempt
+   fun:notifier::Login::StartConnection
+   fun:sync_notifier::InvalidationNotifier::UpdateCredentials
+   fun:sync_notifier::NonBlockingInvalidationNotifier::Core::UpdateCredentials
+   fun:void DispatchToMethod
+}