Rename SyncNotifier->Invalidator and SyncNotifierObserver->InvalidationHandler.

chrome/browser/sync/profile_sync_service_unittest.cc

 // Copyright (c) 2012 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 "base/file_util.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "base/values.h"
 #include "chrome/browser/signin/signin_manager.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
 #include "chrome/browser/signin/token_service.h"
 #include "chrome/browser/signin/token_service_factory.h"
 #include "chrome/browser/sync/glue/bookmark_data_type_controller.h"
 #include "chrome/browser/sync/glue/data_type_controller.h"
 #include "chrome/browser/sync/profile_sync_components_factory_mock.h"
 #include "chrome/browser/sync/test_profile_sync_service.h"
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/net/gaia/gaia_constants.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/testing_pref_service.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/common/content_client.h"
 #include "content/public/test/test_browser_thread.h"
 #include "google/cacheinvalidation/include/types.h"
 #include "sync/js/js_arg_list.h"
 #include "sync/js/js_event_details.h"
 #include "sync/js/js_test_util.h"
-#include "sync/notifier/fake_sync_notifier_observer.h"
+#include "sync/notifier/fake_invalidation_handler.h"
 #include "sync/notifier/object_id_state_map_test_util.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "webkit/glue/webkit_glue.h"
 
 // TODO(akalin): Add tests here that exercise the whole
 // ProfileSyncService/SyncBackendHost stack while mocking out as
 // little as possible.
 
 namespace browser_sync {
@@ skipping 325 matching lines @@
 // expectations are not very interesting.
 TEST_F(ProfileSyncServiceTest, FailToOpenDatabase) {
   StartSyncServiceAndSetInitialSyncEnded(false, true, true, true,
                                          syncer::STORAGE_INVALID);
 
   // The backend is not ready.  Ensure the PSS knows this.
   EXPECT_FALSE(service_->sync_initialized());
 }
 
 // Register for some IDs with the ProfileSyncService and trigger some
-// invalidation messages.  They should be received by the observer.
+// invalidation messages.  They should be received by the handler.
 // Then unregister and trigger the invalidation messages again.  Those
-// shouldn't be received by the observer.
+// shouldn't be received by the handler.
 TEST_F(ProfileSyncServiceTest, UpdateRegisteredInvalidationIds) {
   StartSyncService();
 
   syncer::ObjectIdSet ids;
   ids.insert(invalidation::ObjectId(1, "id1"));
   ids.insert(invalidation::ObjectId(2, "id2"));
   const syncer::ObjectIdStateMap& states =
       syncer::ObjectIdSetToStateMap(ids, "payload");
 
-  syncer::FakeSyncNotifierObserver observer;
+  syncer::FakeInvalidationHandler handler;
 
-  service_->RegisterInvalidationHandler(&observer);
-  service_->UpdateRegisteredInvalidationIds(&observer, ids);
+  service_->RegisterInvalidationHandler(&handler);
+  service_->UpdateRegisteredInvalidationIds(&handler, ids);
 
   SyncBackendHostForProfileSyncTest* const backend =
       service_->GetBackendForTest();
 
   backend->EmitOnNotificationsEnabled();
   EXPECT_EQ(syncer::NO_NOTIFICATION_ERROR,
-            observer.GetNotificationsDisabledReason());
+            handler.GetNotificationsDisabledReason());
 
   backend->EmitOnIncomingNotification(states, syncer::REMOTE_NOTIFICATION);
-  EXPECT_THAT(states, Eq(observer.GetLastNotificationIdStateMap()));
-  EXPECT_EQ(syncer::REMOTE_NOTIFICATION, observer.GetLastNotificationSource());
+  EXPECT_THAT(states, Eq(handler.GetLastNotificationIdStateMap()));
+  EXPECT_EQ(syncer::REMOTE_NOTIFICATION, handler.GetLastNotificationSource());
 
   backend->EmitOnNotificationsDisabled(syncer::TRANSIENT_NOTIFICATION_ERROR);
   EXPECT_EQ(syncer::TRANSIENT_NOTIFICATION_ERROR,
-            observer.GetNotificationsDisabledReason());
+            handler.GetNotificationsDisabledReason());
 
-  Mock::VerifyAndClearExpectations(&observer);
+  Mock::VerifyAndClearExpectations(&handler);
 
-  service_->UnregisterInvalidationHandler(&observer);
+  service_->UnregisterInvalidationHandler(&handler);
 
   backend->EmitOnNotificationsEnabled();
   backend->EmitOnIncomingNotification(states, syncer::REMOTE_NOTIFICATION);
   backend->EmitOnNotificationsDisabled(syncer::TRANSIENT_NOTIFICATION_ERROR);
 }
 
 // Register for some IDs with the ProfileSyncService, restart sync,
 // and trigger some invalidation messages.  They should still be
-// received by the observer.
+// received by the handler.
 TEST_F(ProfileSyncServiceTest, UpdateRegisteredInvalidationIdsPersistence) {
   StartSyncService();
 
   syncer::ObjectIdSet ids;
   ids.insert(invalidation::ObjectId(3, "id3"));
   const syncer::ObjectIdStateMap& states =
       syncer::ObjectIdSetToStateMap(ids, "payload");
 
-  syncer::FakeSyncNotifierObserver observer;
+  syncer::FakeInvalidationHandler handler;
 
-  service_->RegisterInvalidationHandler(&observer);
-  service_->UpdateRegisteredInvalidationIds(&observer, ids);
+  service_->RegisterInvalidationHandler(&handler);
+  service_->UpdateRegisteredInvalidationIds(&handler, ids);
 
   service_->StopAndSuppress();
   service_->UnsuppressAndStart();
 
   SyncBackendHostForProfileSyncTest* const backend =
       service_->GetBackendForTest();
 
   backend->EmitOnNotificationsEnabled();
   EXPECT_EQ(syncer::NO_NOTIFICATION_ERROR,
-            observer.GetNotificationsDisabledReason());
+            handler.GetNotificationsDisabledReason());
 
   backend->EmitOnIncomingNotification(states, syncer::REMOTE_NOTIFICATION);
-  EXPECT_THAT(states, Eq(observer.GetLastNotificationIdStateMap()));
-  EXPECT_EQ(syncer::REMOTE_NOTIFICATION, observer.GetLastNotificationSource());
+  EXPECT_THAT(states, Eq(handler.GetLastNotificationIdStateMap()));
+  EXPECT_EQ(syncer::REMOTE_NOTIFICATION, handler.GetLastNotificationSource());
 
   backend->EmitOnNotificationsDisabled(syncer::TRANSIENT_NOTIFICATION_ERROR);
   EXPECT_EQ(syncer::TRANSIENT_NOTIFICATION_ERROR,
-            observer.GetNotificationsDisabledReason());
+            handler.GetNotificationsDisabledReason());
 }
 
 }  // namespace
 }  // namespace browser_sync