Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5463)

Unified Diff: chrome/browser/extensions/settings/settings_apitest.cc

Issue 10662035: [Sync] Put everything in sync/api into csync namespace (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix comments Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/settings/settings_apitest.cc
diff --git a/chrome/browser/extensions/settings/settings_apitest.cc b/chrome/browser/extensions/settings/settings_apitest.cc
index aab4de9564437f5c669aad0f8896be920ad7b151..32f11283c959c9bcb07d4af702f7b0c880c0078f 100644
--- a/chrome/browser/extensions/settings/settings_apitest.cc
+++ b/chrome/browser/extensions/settings/settings_apitest.cc
@@ -33,35 +33,35 @@ namespace {
// TODO(kalman): test both EXTENSION_SETTINGS and APP_SETTINGS.
const syncable::ModelType kModelType = syncable::EXTENSION_SETTINGS;
-class NoopSyncChangeProcessor : public SyncChangeProcessor {
+class NoopSyncChangeProcessor : public csync::SyncChangeProcessor {
public:
- virtual SyncError ProcessSyncChanges(
+ virtual csync::SyncError ProcessSyncChanges(
const tracked_objects::Location& from_here,
- const SyncChangeList& change_list) OVERRIDE {
- return SyncError();
+ const csync::SyncChangeList& change_list) OVERRIDE {
+ return csync::SyncError();
}
virtual ~NoopSyncChangeProcessor() {};
};
-class SyncChangeProcessorDelegate : public SyncChangeProcessor {
+class SyncChangeProcessorDelegate : public csync::SyncChangeProcessor {
public:
- explicit SyncChangeProcessorDelegate(SyncChangeProcessor* recipient)
+ explicit SyncChangeProcessorDelegate(csync::SyncChangeProcessor* recipient)
: recipient_(recipient) {
DCHECK(recipient_);
}
virtual ~SyncChangeProcessorDelegate() {}
- // SyncChangeProcessor implementation.
- virtual SyncError ProcessSyncChanges(
+ // csync::SyncChangeProcessor implementation.
+ virtual csync::SyncError ProcessSyncChanges(
const tracked_objects::Location& from_here,
- const SyncChangeList& change_list) OVERRIDE {
+ const csync::SyncChangeList& change_list) OVERRIDE {
return recipient_->ProcessSyncChanges(from_here, change_list);
}
private:
// The recipient of all sync changes.
- SyncChangeProcessor* recipient_;
+ csync::SyncChangeProcessor* recipient_;
DISALLOW_COPY_AND_ASSIGN(SyncChangeProcessorDelegate);
};
@@ -99,7 +99,7 @@ class ExtensionSettingsApiTest : public ExtensionApiTest {
settings_namespace, normal_action, incognito_action, NULL, true);
}
- void InitSync(SyncChangeProcessor* sync_processor) {
+ void InitSync(csync::SyncChangeProcessor* sync_processor) {
MessageLoop::current()->RunAllPending();
InitSyncWithSyncableService(
sync_processor,
@@ -107,7 +107,7 @@ class ExtensionSettingsApiTest : public ExtensionApiTest {
GetBackendForSync(kModelType));
}
- void SendChanges(const SyncChangeList& change_list) {
+ void SendChanges(const csync::SyncChangeList& change_list) {
MessageLoop::current()->RunAllPending();
SendChangesToSyncableService(
change_list,
@@ -159,18 +159,20 @@ class ExtensionSettingsApiTest : public ExtensionApiTest {
}
void InitSyncWithSyncableService(
- SyncChangeProcessor* sync_processor, SyncableService* settings_service) {
+ csync::SyncChangeProcessor* sync_processor,
+ csync::SyncableService* settings_service) {
EXPECT_FALSE(settings_service->MergeDataAndStartSyncing(
kModelType,
- SyncDataList(),
- scoped_ptr<SyncChangeProcessor>(
+ csync::SyncDataList(),
+ scoped_ptr<csync::SyncChangeProcessor>(
new SyncChangeProcessorDelegate(sync_processor)),
- scoped_ptr<SyncErrorFactory>(
- new SyncErrorFactoryMock())).IsSet());
+ scoped_ptr<csync::SyncErrorFactory>(
+ new csync::SyncErrorFactoryMock())).IsSet());
}
void SendChangesToSyncableService(
- const SyncChangeList& change_list, SyncableService* settings_service) {
+ const csync::SyncChangeList& change_list,
+ csync::SyncableService* settings_service) {
EXPECT_FALSE(
settings_service->ProcessSyncChanges(FROM_HERE, change_list).IsSet());
}
@@ -313,7 +315,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionSettingsApiTest,
InitSync(&sync_processor);
// Set "foo" to "bar" via sync.
- SyncChangeList sync_changes;
+ csync::SyncChangeList sync_changes;
StringValue bar("bar");
sync_changes.push_back(settings_sync_util::CreateAdd(
extension_id, "foo", bar, kModelType));
@@ -361,7 +363,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionSettingsApiTest,
InitSync(&sync_processor);
// Set "foo" to "bar" via sync.
- SyncChangeList sync_changes;
+ csync::SyncChangeList sync_changes;
StringValue bar("bar");
sync_changes.push_back(settings_sync_util::CreateAdd(
extension_id, "foo", bar, kModelType));
« no previous file with comments | « chrome/browser/extensions/settings/setting_sync_data.cc ('k') | chrome/browser/extensions/settings/settings_backend.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698