Made server_notifier_thread listen to UNKNOWN data type.

This is a temporary workaround so that server-issued notifications work
for new sync data types.

BUG=none
TEST=manual

Review URL: http://codereview.chromium.org/3114012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56290 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 13 insertions, 3 deletions

diff --git a/chrome/browser/sync/notifier/chrome_invalidation_client.cc b/chrome/browser/sync/notifier/chrome_invalidation_client.cc
index f183e61..6701261 100644
--- a/chrome/browser/sync/notifier/chrome_invalidation_client.cc
+++ b/chrome/browser/sync/notifier/chrome_invalidation_client.cc
@@ -76,6 +76,10 @@ void ChromeInvalidationClient::RegisterTypes() {
        i < syncable::MODEL_TYPE_COUNT; ++i) {
     registration_manager_->RegisterType(syncable::ModelTypeFromInt(i));
   }
+  // TODO(akalin): This is a hack to make new sync data types work
+  // with server-issued notifications.  Remove this when it's not
+  // needed anymore.
+  registration_manager_->RegisterType(syncable::UNSPECIFIED);
 }
 
 void ChromeInvalidationClient::Invalidate(
diff --git a/chrome/browser/sync/notifier/registration_manager.cc b/chrome/browser/sync/notifier/registration_manager.cc
index 3895c91..ccadac4 100644
--- a/chrome/browser/sync/notifier/registration_manager.cc
+++ b/chrome/browser/sync/notifier/registration_manager.cc
@@ -60,8 +60,7 @@ void RegistrationManager::MarkRegistrationLost(
   RegistrationStatusMap::iterator it =
       registration_status_.find(model_type);
   if (it == registration_status_.end()) {
-    LOG(ERROR) << "Unknown model type: "
-               << syncable::ModelTypeToString(model_type);
+    LOG(ERROR) << "Unknown model type: " << model_type;
     return;
   }
   it->second = UNREGISTERED;
diff --git a/chrome/browser/sync/notifier/server_notifier_thread.cc b/chrome/browser/sync/notifier/server_notifier_thread.cc
index 19417f9..8b2aba3 100644
--- a/chrome/browser/sync/notifier/server_notifier_thread.cc
+++ b/chrome/browser/sync/notifier/server_notifier_thread.cc
@@ -55,7 +55,14 @@ void ServerNotifierThread::SendNotification(
 
 void ServerNotifierThread::OnInvalidate(syncable::ModelType model_type) {
   DCHECK_EQ(MessageLoop::current(), worker_message_loop());
-  LOG(INFO) << "OnInvalidate: " << syncable::ModelTypeToString(model_type);
+  // TODO(akalin): This is a hack to make new sync data types work
+  // with server-issued notifications.  Remove this when it's not
+  // needed anymore.
+  if (model_type == syncable::UNSPECIFIED) {
+    LOG(INFO) << "OnInvalidate: UNKNOWN";
+  } else {
+    LOG(INFO) << "OnInvalidate: " << syncable::ModelTypeToString(model_type);
+  }
   // TODO(akalin): Signal notification only for the invalidated types.
   parent_message_loop_->PostTask(
       FROM_HERE,