diff options
| author | rlarocque@chromium.org <rlarocque@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-12-04 02:51:36 +0000 |
|---|---|---|
| committer | rlarocque@chromium.org <rlarocque@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-12-04 02:51:36 +0000 |
| commit | a95754871e9e86ee503c46d12e8704c02f6c6e35 (patch) | |
| tree | 92be09176ae15cd5e02595e3586db4f4d2026499 /sync/sync_tests.gypi | |
| parent | 01e117df184677e024d7f4f17fee7fdfeb1e4053 (diff) | |
| download | chromium_src-a95754871e9e86ee503c46d12e8704c02f6c6e35.zip chromium_src-a95754871e9e86ee503c46d12e8704c02f6c6e35.tar.gz chromium_src-a95754871e9e86ee503c46d12e8704c02f6c6e35.tar.bz2 | |
sync: Per-type update application
This change moves the update application functionality from the
ApplyUpdatesAndResolveConflictsCommand into the
SyncDirectoryUpdateHandler class. This change will allow us to
implement update application differently for different types.
Because update application happens on the model threads, the
ApplyUpdatesAndResolveConflictsCommand had to be aware of
ModelSafeRoutingInfo, ModelSafeWorkers, and other concepts intended to
hide threading details. The new code takes a different approach. It
hides the threading details specific to each type inside its
SyncDirectoryUpateHandler by initializing it with a scoped_refptr to its
associated ModelSafeWorker.
The ApplyUpdatesAndResolveConflictsCommand was the last SyncerCommand.
With its removal, we can also remove the definitions of SyncerCommand,
ModelChangingSyncerCommand and SyncerCommandTest.
BUG=278484
Review URL: https://codereview.chromium.org/72403003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238532 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync/sync_tests.gypi')
| -rw-r--r-- | sync/sync_tests.gypi | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/sync/sync_tests.gypi b/sync/sync_tests.gypi index 010cb5b..683ca52 100644 --- a/sync/sync_tests.gypi +++ b/sync/sync_tests.gypi @@ -40,8 +40,6 @@ 'test/engine/fake_sync_scheduler.h', 'test/engine/mock_connection_manager.cc', 'test/engine/mock_connection_manager.h', - 'test/engine/syncer_command_test.cc', - 'test/engine/syncer_command_test.h', 'test/engine/test_directory_setter_upper.cc', 'test/engine/test_directory_setter_upper.h', 'test/engine/test_id_factory.h', @@ -243,10 +241,8 @@ 'internal_api/public/util/immutable_unittest.cc', 'internal_api/public/util/weak_handle_unittest.cc', 'engine/apply_control_data_updates_unittest.cc', - 'engine/apply_updates_and_resolve_conflicts_command_unittest.cc', 'engine/backoff_delay_provider_unittest.cc', 'engine/download_unittest.cc', - 'engine/model_changing_syncer_command_unittest.cc', 'engine/sync_scheduler_unittest.cc', 'engine/syncer_proto_util_unittest.cc', 'engine/syncer_unittest.cc', |