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

Unified Diff: chrome/browser/sync/engine/cleanup_disabled_types_command.cc

Issue 9699057: [Sync] Move 'sync' target to sync/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address Tim's comments Created 8 years, 9 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: chrome/browser/sync/engine/cleanup_disabled_types_command.cc
diff --git a/chrome/browser/sync/engine/cleanup_disabled_types_command.cc b/chrome/browser/sync/engine/cleanup_disabled_types_command.cc
deleted file mode 100644
index fcac8a9255368f818a06b837742efed82af6891e..0000000000000000000000000000000000000000
--- a/chrome/browser/sync/engine/cleanup_disabled_types_command.cc
+++ /dev/null
@@ -1,70 +0,0 @@
-// Copyright (c) 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.
-
-#include "chrome/browser/sync/engine/cleanup_disabled_types_command.h"
-
-#include <algorithm>
-
-#include "chrome/browser/sync/sessions/sync_session.h"
-#include "chrome/browser/sync/sessions/sync_session_context.h"
-#include "chrome/browser/sync/syncable/model_type.h"
-#include "chrome/browser/sync/syncable/syncable.h"
-
-namespace browser_sync {
-
-CleanupDisabledTypesCommand::CleanupDisabledTypesCommand() {}
-CleanupDisabledTypesCommand::~CleanupDisabledTypesCommand() {}
-
-SyncerError CleanupDisabledTypesCommand::ExecuteImpl(
- sessions::SyncSession* session) {
- using syncable::ModelTypeSet;
- using syncable::ModelTypeSetToString;
- // Because a full directory purge is slow, we avoid purging
- // undesired types unless we have reason to believe they were
- // previously enabled. Because purging could theoretically fail on
- // the first sync session (when there's no previous routing info) we
- // pay the full directory scan price once and do a "deep clean" of
- // types that may potentially need cleanup so that we converge to
- // the correct state.
- //
- // in_previous | !in_previous
- // |
- // initial_sync_ended should clean | may have attempted cleanup
- // !initial_sync_ended should clean | may have never been enabled, or
- // | could have been disabled before
- // | initial sync ended and cleanup
- // | may not have happened yet
- // | (failure, browser restart
- // | before another sync session,..)
-
- const ModelTypeSet enabled_types =
- GetRoutingInfoTypes(session->routing_info());
-
- const ModelTypeSet previous_enabled_types =
- GetRoutingInfoTypes(
- session->context()->previous_session_routing_info());
-
- ModelTypeSet to_cleanup = Difference(ModelTypeSet::All(), enabled_types);
-
- // If |previous_enabled_types| is non-empty (i.e., not the first
- // sync session), set |to_cleanup| to its intersection with
- // |previous_enabled_types|.
- if (!previous_enabled_types.Empty()) {
- to_cleanup.RetainAll(previous_enabled_types);
- }
-
- DVLOG(1) << "enabled_types = " << ModelTypeSetToString(enabled_types)
- << ", previous_enabled_types = "
- << ModelTypeSetToString(previous_enabled_types)
- << ", to_cleanup = " << ModelTypeSetToString(to_cleanup);
-
- if (to_cleanup.Empty())
- return SYNCER_OK;
-
- session->context()->directory()->PurgeEntriesWithTypeIn(to_cleanup);
- return SYNCER_OK;
-}
-
-} // namespace browser_sync
-

Powered by Google App Engine
This is Rietveld 408576698