Fixed app name used for server-issued notifications.

chrome/browser/sync/notifier/server_notifier_thread.cc

 // Copyright (c) 2010 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/server_notifier_thread.h"
 
 #include <string>
 #include <vector>
 
 #include "base/logging.h"
@@ skipping 93 matching lines @@
   RegisterTypes();
   RunAndDeleteClosure(callback);
 }
 
 void ServerNotifierThread::StartInvalidationListener() {
   DCHECK_EQ(MessageLoop::current(), worker_message_loop());
 
   StopInvalidationListener();
   chrome_invalidation_client_.reset(new ChromeInvalidationClient());
 
-  // TODO(akalin): If we can figure out a unique per-session key that
-  // is preserved by the server, we can use that instead of kAppName.
-  // What this buys us is that we then won't receive any notifications
-  // that were generated by ourselves.
-  const std::string kAppName = "cc_sync_listen_notifications";
+  // TODO(akalin): Make cache_guid() part of the app name.  If we do
+  // so and we somehow propagate it up to the server somehow, we can
+  // make it so that we won't receive any notifications that were
+  // generated from our own changes.
+  const std::string kAppName = "server_notifier_thread";
   chrome_invalidation_client_->Start(kAppName, this, xmpp_client());
 }
 
 void ServerNotifierThread::RegisterTypesAndSignalSubscribed() {
   DCHECK_EQ(MessageLoop::current(), worker_message_loop());
   RegisterTypes();
   parent_message_loop_->PostTask(
       FROM_HERE,
       NewRunnableMethod(
           this,
@@ skipping 60 matching lines @@
   DCHECK_EQ(MessageLoop::current(), worker_message_loop());
 
   if (chrome_invalidation_client_.get()) {
     // TODO(akalin): Need to do unregisters here?
     chrome_invalidation_client_->Stop();
   }
   chrome_invalidation_client_.reset();
 }
 
 }  // namespace sync_notifier