[Sync] Address msw's comments for r149747

sync/internal_api/test/fake_sync_manager.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 "sync/internal_api/public/test/fake_sync_manager.h"
 
 #include <cstddef>
 
 #include "base/bind.h"
 #include "base/location.h"
@@ skipping 37 matching lines @@
   downloaded_types_.Clear();
   return downloaded_types;
 }
 
 ModelTypeSet FakeSyncManager::GetAndResetEnabledTypes() {
   ModelTypeSet enabled_types = enabled_types_;
   enabled_types_.Clear();
   return enabled_types;
 }
 
-void FakeSyncManager::Invalidate(
-    const ObjectIdPayloadMap& id_payloads,
-    IncomingNotificationSource source) {
+void FakeSyncManager::Invalidate(const ObjectIdPayloadMap& id_payloads,
+                                 IncomingNotificationSource source) {
   if (!sync_task_runner_->PostTask(
       FROM_HERE,
       base::Bind(&FakeSyncManager::InvalidateOnSyncThread,
                  base::Unretained(this), id_payloads, source))) {
     NOTREACHED();
   }
 }
 
 void FakeSyncManager::EnableNotifications() {
   if (!sync_task_runner_->PostTask(
@@ skipping 73 matching lines @@
 
 void FakeSyncManager::UpdateCredentials(const SyncCredentials& credentials) {
   NOTIMPLEMENTED();
 }
 
 void FakeSyncManager::UpdateEnabledTypes(const ModelTypeSet& types) {
   enabled_types_ = types;
 }
 
 void FakeSyncManager::UpdateRegisteredInvalidationIds(
-    SyncNotifierObserver* handler, const ObjectIdSet& ids) {
+    SyncNotifierObserver* handler,
+    const ObjectIdSet& ids) {
   notifier_helper_.UpdateRegisteredIds(handler, ids);
 }
 
 void FakeSyncManager::StartSyncingNormally(
       const ModelSafeRoutingInfo& routing_info) {
   // Do nothing.
 }
 
 void FakeSyncManager::SetEncryptionPassphrase(const std::string& passphrase,
                                               bool is_explicit) {
@@ skipping 57 matching lines @@
 
 bool FakeSyncManager::GetKeystoreKeyBootstrapToken(std::string* token) {
   return false;
 }
 
 void FakeSyncManager::SaveChanges() {
   // Do nothing.
 }
 
 void FakeSyncManager::StopSyncingForShutdown(const base::Closure& callback) {
   if (!sync_task_runner_->PostTask(FROM_HERE, callback)) {

[akalin, 2012/08/03 20:08:49]

Prefer to be consistent with the other posts.

     NOTREACHED();
   }
 }
 
 void FakeSyncManager::ShutdownOnSyncThread() {
   DCHECK(sync_task_runner_->RunsTasksOnCurrentThread());
 }
 
 UserShare* FakeSyncManager::GetUserShare() {
   NOTIMPLEMENTED();
@@ skipping 30 matching lines @@
   notifier_helper_.EmitOnNotificationsEnabled();
 }
 
 void FakeSyncManager::DisableNotificationsOnSyncThread(
     NotificationsDisabledReason reason) {
   DCHECK(sync_task_runner_->RunsTasksOnCurrentThread());
   notifier_helper_.EmitOnNotificationsDisabled(reason);
 }
 
 }  // namespace syncer