Use InvalidationClient::Create() instead of creating an Impl directly.

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 92 matching lines @@
   RegisterTypes();
   RunAndDeleteClosure(callback);
 }
 
 void ServerNotifierThread::StartInvalidationListener() {
   DCHECK_EQ(MessageLoop::current(), worker_message_loop());
 
   StopInvalidationListener();
   chrome_invalidation_client_.reset(new ChromeInvalidationClient());
 
-  // TODO(akalin): Make cache_guid() part of the app name.  If we do
+  // TODO(akalin): Make cache_guid() part of the client ID.  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());
+  const std::string kClientId = "server_notifier_thread";
+  chrome_invalidation_client_->Start(kClientId, this, xmpp_client());
 }
 
 void ServerNotifierThread::RegisterTypesAndSignalSubscribed() {
   DCHECK_EQ(MessageLoop::current(), worker_message_loop());
   RegisterTypes();
   parent_message_loop_->PostTask(
       FROM_HERE,
       NewRunnableMethod(
           this,
           &ServerNotifierThread::SignalSubscribed));
@@ skipping 51 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