Fix InvalidationNotifierTest leak and suppressions

chrome/browser/sync/notifier/invalidation_notifier_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/sync/notifier/invalidation_notifier.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "chrome/browser/sync/notifier/mock_sync_notifier_observer.h"
 #include "chrome/browser/sync/syncable/model_type.h"
@@ skipping 25 matching lines @@
     notifier_options.request_context_getter =
         new TestURLRequestContextGetter();
     invalidation_notifier_.reset(new InvalidationNotifier(notifier_options,
                                                           "fake_client_info"));
     invalidation_notifier_->AddObserver(&mock_observer_);
   }
 
   virtual void TearDown() {
     invalidation_notifier_->RemoveObserver(&mock_observer_);
     invalidation_notifier_.reset();
+    message_loop_.RunAllPending();
   }
 
   MessageLoop message_loop_;
   scoped_ptr<InvalidationNotifier> invalidation_notifier_;
   StrictMock<MockSyncNotifierObserver> mock_observer_;
   notifier::FakeBaseTask fake_base_task_;
   // Since this test calls HostResolver code, we need an IO thread.
   BrowserThread io_thread_;
 };
 
@@ skipping 23 matching lines @@
   // Shouldn't trigger notification state change.
   invalidation_notifier_->OnDisconnect();
   invalidation_notifier_->OnConnect(fake_base_task_.AsWeakPtr());
 
   invalidation_notifier_->OnSessionStatusChanged(false);
 }
 
 }  // namespace
 
 }  // namespace sync_notifier