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

Unified Diff: chrome/browser/sync/engine/process_updates_command.h

Issue 8533024: sync: Use a single transaction in ProcessUpdates rather than one per entry. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 1 month 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/sync/engine/process_updates_command.h
diff --git a/chrome/browser/sync/engine/process_updates_command.h b/chrome/browser/sync/engine/process_updates_command.h
index f9ff968bf4293354511c3d3df569dea90ef32364..bf34b6123926880284a4f7edab7bc241fd10fcb6 100644
--- a/chrome/browser/sync/engine/process_updates_command.h
+++ b/chrome/browser/sync/engine/process_updates_command.h
@@ -11,6 +11,7 @@
namespace syncable {
class ScopedDirLookup;
+class WriteTransaction;
}
namespace sync_pb {
@@ -39,7 +40,8 @@ class ProcessUpdatesCommand : public ModelChangingSyncerCommand {
private:
ServerUpdateProcessingResult ProcessUpdate(
const syncable::ScopedDirLookup& dir,
- const sync_pb::SyncEntity& proto_update);
+ const sync_pb::SyncEntity& proto_update,
+ syncable::WriteTransaction* const trans);
DISALLOW_COPY_AND_ASSIGN(ProcessUpdatesCommand);
};

Powered by Google App Engine
This is Rietveld 408576698