diff options
author | rlarocque@chromium.org <rlarocque@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-26 21:59:19 +0000 |
---|---|---|
committer | rlarocque@chromium.org <rlarocque@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-26 21:59:19 +0000 |
commit | 9da02fdb172d447777aba9cda8a17362eff0635a (patch) | |
tree | 9bed06920136484ba9ab9212cc2a12181b5630f8 /sync/sessions/status_controller.h | |
parent | 8f7c1f76f82d5469869417b01b0b21989b926826 (diff) | |
download | chromium_src-9da02fdb172d447777aba9cda8a17362eff0635a.zip chromium_src-9da02fdb172d447777aba9cda8a17362eff0635a.tar.gz chromium_src-9da02fdb172d447777aba9cda8a17362eff0635a.tar.bz2 |
sync: Process commit responses on the sync thread
Convert ProcessCommitResponseCommand from a ModelChangingSyncerCommand
to a regular SyncerCommand. This means that it will be executed
entirely on the sync thread, rather than having its work delegated to
the native model threads.
This is built on top of the recent ModelNeutralWriteTransaction and
ModelNeutralMutableEntry work. In order to ensure that no
model-changing mutations are performed while processing the commit
response, and to prevent future code changes from introducing them, the
ProcessCommitResponseCommand's methods have been converted to use
ModelNeutral transactions only.
The use of ModelNeutral transactions and non-ModelChanging
SyncerCommands should grant some minor performance improvements and make
the code simpler.
BUG=284672
Review URL: https://codereview.chromium.org/24764004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225585 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync/sessions/status_controller.h')
-rw-r--r-- | sync/sessions/status_controller.h | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/sync/sessions/status_controller.h b/sync/sessions/status_controller.h index a547c1b..d7060d5 100644 --- a/sync/sessions/status_controller.h +++ b/sync/sessions/status_controller.h @@ -71,13 +71,6 @@ class SYNC_EXPORT_PRIVATE StatusController { return model_neutral_.num_server_changes_remaining; } - const OrderedCommitSet::Projection& commit_id_projection( - const sessions::OrderedCommitSet &commit_set) { - DCHECK(group_restriction_in_effect_) - << "No group restriction for projection."; - return commit_set.GetCommitIdProjection(group_restriction_); - } - // Various conflict counters. int num_encryption_conflicts() const; int num_hierarchy_conflicts() const; |