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

Unified Diff: sync/engine/build_commit_command.h

Issue 25638003: sync: Implement per-type commit interface (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Attempt to fix win compile Created 7 years, 2 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
« no previous file with comments | « no previous file | sync/engine/build_commit_command.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/engine/build_commit_command.h
diff --git a/sync/engine/build_commit_command.h b/sync/engine/build_commit_command.h
deleted file mode 100644
index a47c62afe0c97bbb7e7da1095563be010998697c..0000000000000000000000000000000000000000
--- a/sync/engine/build_commit_command.h
+++ /dev/null
@@ -1,82 +0,0 @@
-// Copyright 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.
-
-#ifndef SYNC_ENGINE_BUILD_COMMIT_COMMAND_H_
-#define SYNC_ENGINE_BUILD_COMMIT_COMMAND_H_
-
-#include "base/basictypes.h"
-#include "base/compiler_specific.h"
-#include "base/gtest_prod_util.h"
-#include "sync/base/sync_export.h"
-#include "sync/engine/syncer_command.h"
-#include "sync/syncable/entry_kernel.h"
-#include "sync/util/extensions_activity.h"
-
-namespace syncer {
-
-namespace sessions {
-class OrderedCommitSet;
-}
-
-namespace syncable {
-class Entry;
-class BaseTransaction;
-}
-
-// A class that contains the code used to serialize a set of sync items into a
-// protobuf commit message. This conversion process references the
-// syncable::Directory, which is why it must be called within the same
-// transaction as the GetCommitIdsCommand that fetched the set of items to be
-// committed.
-//
-// See SyncerCommand documentation for more info.
-class SYNC_EXPORT_PRIVATE BuildCommitCommand : public SyncerCommand {
- public:
- // The batch_commit_set parameter contains a set of references to the items
- // that should be committed.
- //
- // The commit_message parameter is an output parameter which will contain the
- // fully initialized commit message once ExecuteImpl() has been called.
- BuildCommitCommand(
- syncable::BaseTransaction* trans,
- const sessions::OrderedCommitSet& batch_commit_set,
- sync_pb::ClientToServerMessage* commit_message,
- ExtensionsActivity::Records* extensions_activity_buffer);
- virtual ~BuildCommitCommand();
-
- // SyncerCommand implementation.
- virtual SyncerError ExecuteImpl(sessions::SyncSession* session) OVERRIDE;
-
- // Helper function that takes a snapshot of |meta_entry| and puts it into a
- // protobuf suitable for use in a commit request message.
- static void BuildCommitItem(const syncable::Entry& meta_entry,
- sync_pb::SyncEntity* sync_entry);
-
- private:
- FRIEND_TEST_ALL_PREFIXES(BuildCommitCommandTest, InterpolatePosition);
-
- void AddExtensionsActivityToMessage(sessions::SyncSession* session,
- sync_pb::CommitMessage* message);
-
- // Fills the config_params field of |message|.
- void AddClientConfigParamsToMessage(sessions::SyncSession* session,
- sync_pb::CommitMessage* message);
-
- DISALLOW_COPY_AND_ASSIGN(BuildCommitCommand);
-
- // A pointer to a valid transaction not owned by this class.
- syncable::BaseTransaction* trans_;
-
- // Input parameter; see constructor comment.
- const sessions::OrderedCommitSet& batch_commit_set_;
-
- // Output parameter; see constructor comment.
- sync_pb::ClientToServerMessage* commit_message_;
-
- ExtensionsActivity::Records* extensions_activity_buffer_;
-};
-
-} // namespace syncer
-
-#endif // SYNC_ENGINE_BUILD_COMMIT_COMMAND_H_
« no previous file with comments | « no previous file | sync/engine/build_commit_command.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698