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

Unified Diff: components/sync/engine_impl/update_applicator.h

Issue 2258873003: [Sync] Move sessions/ to engine/cycle/ and rename things to match. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 4 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: components/sync/engine_impl/update_applicator.h
diff --git a/components/sync/engine_impl/update_applicator.h b/components/sync/engine_impl/update_applicator.h
index e4afa3619459062712584f3a1034d7b95374713f..dfbee4a29dba4832f435b266dc878b07661551ae 100644
--- a/components/sync/engine_impl/update_applicator.h
+++ b/components/sync/engine_impl/update_applicator.h
@@ -3,7 +3,7 @@
// found in the LICENSE file.
//
// An UpdateApplicator is used to iterate over a number of unapplied updates,
-// applying them to the client using the given syncer session.
+// applying them to the client using the given syncer cycle.
//
// UpdateApplicator might resemble an iterator, but it actually keeps retrying
// failed updates until no remaining updates can be successfully applied.
@@ -18,23 +18,20 @@
#include "base/macros.h"
#include "components/sync/engine/model_safe_worker.h"
-#include "components/sync/sessions_impl/status_controller.h"
+#include "components/sync/engine_impl/cycle/status_controller.h"
skym 2016/08/19 17:17:43 I don't think this is used in this file, but seems
maxbogue 2016/08/19 19:12:01 Removed and added to components/sync/engine_impl/d
#include "components/sync/syncable/syncable_id.h"
namespace syncer {
-namespace sessions {
+class ConflictResolver;
skym 2016/08/19 17:17:43 Where is this used? Remove?
maxbogue 2016/08/19 19:12:01 Done.
+class Cryptographer;
class StatusController;
skym 2016/08/19 17:17:43 This is also getting #included, definitely don't n
maxbogue 2016/08/19 19:12:01 Done.
-}
namespace syncable {
class WriteTransaction;
class Entry;
}
-class ConflictResolver;
-class Cryptographer;
-
class UpdateApplicator {
public:
explicit UpdateApplicator(Cryptographer* cryptographer);

Powered by Google App Engine
This is Rietveld 408576698