[Sync] Fix memory leak in JsSyncManagerObserver unit tests

chrome/browser/sync/js_sync_manager_observer_unittest.cc

Index: chrome/browser/sync/js_sync_manager_observer_unittest.cc
diff --git a/chrome/browser/sync/js_sync_manager_observer_unittest.cc b/chrome/browser/sync/js_sync_manager_observer_unittest.cc
index 74a2a075783af0243b2540bd7647be0698775bee..2057dc7ca336ca4130219099e46f0cdf2dc244da 100644
--- a/chrome/browser/sync/js_sync_manager_observer_unittest.cc
+++ b/chrome/browser/sync/js_sync_manager_observer_unittest.cc
@@ -33,15 +33,18 @@ class JsSyncManagerObserverTest : public testing::Test {
         mock_js_event_handler_.AsWeakHandle());
   }
 
+ private:
+  // This must be destroyed after the member variables below in order
+  // for WeakHandles to be destroyed properly.
+  MessageLoop message_loop_;
+
+ protected:
   StrictMock<MockJsEventHandler> mock_js_event_handler_;
   JsSyncManagerObserver js_sync_manager_observer_;
 
   void PumpLoop() {
     message_loop_.RunAllPending();
   }
-
- private:
-  MessageLoop message_loop_;
 };
 
 TEST_F(JsSyncManagerObserverTest, NoArgNotifiations) {