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

Side by Side Diff: components/sync_driver/model_association_manager.cc

Issue 133503011: Move files from //chrome/browser/sync to sync_driver component. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/sync/glue/model_association_manager.h" 5 #include "components/sync_driver/model_association_manager.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <functional> 8 #include <functional>
9 9
10 #include "base/debug/trace_event.h" 10 #include "base/debug/trace_event.h"
11 #include "base/logging.h" 11 #include "base/logging.h"
12 #include "base/message_loop/message_loop.h" 12 #include "base/message_loop/message_loop.h"
13 #include "base/metrics/histogram.h" 13 #include "base/metrics/histogram.h"
14 #include "content/public/browser/browser_thread.h"
15 #include "sync/internal_api/public/base/model_type.h" 14 #include "sync/internal_api/public/base/model_type.h"
16 15
17 using content::BrowserThread;
18 using syncer::ModelTypeSet; 16 using syncer::ModelTypeSet;
19 17
20 namespace browser_sync { 18 namespace browser_sync {
21 19
22 namespace { 20 namespace {
23 21
24 static const syncer::ModelType kStartOrder[] = { 22 static const syncer::ModelType kStartOrder[] = {
25 syncer::NIGORI, // Listed for completeness. 23 syncer::NIGORI, // Listed for completeness.
26 syncer::DEVICE_INFO, // Listed for completeness. 24 syncer::DEVICE_INFO, // Listed for completeness.
27 syncer::EXPERIMENTS, // Listed for completeness. 25 syncer::EXPERIMENTS, // Listed for completeness.
(...skipping 301 matching lines...) Expand 10 before | Expand all | Expand 10 after
329 type, base::TimeTicks::Now())); 327 type, base::TimeTicks::Now()));
330 } 328 }
331 } 329 }
332 330
333 void ModelAssociationManager::TypeStartCallback( 331 void ModelAssociationManager::TypeStartCallback(
334 syncer::ModelType type, 332 syncer::ModelType type,
335 base::TimeTicks type_start_time, 333 base::TimeTicks type_start_time,
336 DataTypeController::StartResult start_result, 334 DataTypeController::StartResult start_result,
337 const syncer::SyncMergeResult& local_merge_result, 335 const syncer::SyncMergeResult& local_merge_result,
338 const syncer::SyncMergeResult& syncer_merge_result) { 336 const syncer::SyncMergeResult& syncer_merge_result) {
339 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
blundell 2014/02/01 13:18:32 Do you think it's worth preserving this DCHECK by
Nicolas Zea 2014/02/03 19:21:36 Removing it is fine, all these classes are UI thre
340
341 // TODO(haitaol): temporary fix for 335606. 337 // TODO(haitaol): temporary fix for 335606.
342 if (slow_types_.Has(type)) 338 if (slow_types_.Has(type))
343 return; 339 return;
344 340
345 // This happens when slow associating type is disabled by new configuration. 341 // This happens when slow associating type is disabled by new configuration.
346 if (!desired_types_.Has(type)) 342 if (!desired_types_.Has(type))
347 return; 343 return;
348 344
349 slow_types_.Remove(type); 345 slow_types_.Remove(type);
350 346
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
451 447
452 result_processor_->OnModelAssociationDone(result); 448 result_processor_->OnModelAssociationDone(result);
453 } 449 }
454 450
455 base::OneShotTimer<ModelAssociationManager>* 451 base::OneShotTimer<ModelAssociationManager>*
456 ModelAssociationManager::GetTimerForTesting() { 452 ModelAssociationManager::GetTimerForTesting() {
457 return &timer_; 453 return &timer_;
458 } 454 }
459 455
460 } // namespace browser_sync 456 } // namespace browser_sync
OLDNEW
« no previous file with comments | « components/sync_driver/model_association_manager.h ('k') | components/sync_driver/model_association_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698