[Sync] Update SyncBackendHost to forward ClearServerData to SyncManager

chrome/browser/sync/glue/sync_backend_host_impl_unittest.cc

 // Copyright 2013 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/glue/sync_backend_host_impl.h"
 
 #include <cstddef>
 
 #include "base/files/file_util.h"
 #include "base/location.h"
@@ skipping 24 matching lines @@
 #include "sync/internal_api/public/http_bridge_network_resources.h"
 #include "sync/internal_api/public/network_resources.h"
 #include "sync/internal_api/public/sessions/commit_counters.h"
 #include "sync/internal_api/public/sessions/status_counters.h"
 #include "sync/internal_api/public/sessions/update_counters.h"
 #include "sync/internal_api/public/sync_manager_factory.h"
 #include "sync/internal_api/public/test/fake_sync_manager.h"
 #include "sync/internal_api/public/util/experiments.h"
 #include "sync/protocol/encryption.pb.h"
 #include "sync/protocol/sync_protocol_error.h"
+#include "sync/test/callback_counter.h"
 #include "sync/util/test_unrecoverable_error_handler.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 using content::BrowserThread;
 using syncer::FakeSyncManager;
 using syncer::SyncManager;
 using ::testing::InvokeWithoutArgs;
 using ::testing::StrictMock;
@@ skipping 735 matching lines @@
   enabled_types_.RemoveAll(error_types);
   ready_types = ConfigureDataTypes(
       enabled_types_, Difference(syncer::ModelTypeSet::All(), enabled_types_),
       syncer::ModelTypeSet());
   EXPECT_TRUE(
       ready_types.Equals(syncer::Difference(enabled_types_, error_types)));
   EXPECT_FALSE(fake_manager_->GetTypesWithEmptyProgressMarkerToken(
       error_types).Empty());
 }
 
+// Test that a call to ClearServerData is forwarded to the underlying
+// SyncManager.
+TEST_F(SyncBackendHostTest, ClearServerDataCallsAreForwarded) {
+  InitializeBackend(true);
+  syncer::CallbackCounter callback_counter;
+  syncer::SyncManager::ClearServerDataCallback callback;

[pavely, 2015/07/29 22:57:23]

Callback is not used. Remove.

[maniscalco, 2015/07/29 23:01:09]

Good catch!  Done.

+  backend_->ClearServerData(base::Bind(&syncer::CallbackCounter::Callback,
+                                       base::Unretained(&callback_counter)));
+  fake_manager_->WaitForSyncThread();
+  EXPECT_EQ(1, callback_counter.times_called());
+}
+
 }  // namespace
 
 }  // namespace browser_sync