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

Side by Side Diff: chrome/browser/sync/profile_sync_service.cc

Issue 9791053: Revert 129359 - [Sync] [Mac] Tweak OneClickSigninDialog nib (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 8 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/profile_sync_service.h" 5 #include "chrome/browser/sync/profile_sync_service.h"
6 6
7 #include <cstddef> 7 #include <cstddef>
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <utility> 10 #include <utility>
(...skipping 637 matching lines...) Expand 10 before | Expand all | Expand 10 after
648 // Backend is initialized but we're not in sync setup, so this must be an 648 // Backend is initialized but we're not in sync setup, so this must be an
649 // autostart - mark our sync setup as completed and we'll start syncing 649 // autostart - mark our sync setup as completed and we'll start syncing
650 // below. 650 // below.
651 SetSyncSetupCompleted(); 651 SetSyncSetupCompleted();
652 NotifyObservers(); 652 NotifyObservers();
653 } 653 }
654 654
655 if (HasSyncSetupCompleted()) { 655 if (HasSyncSetupCompleted()) {
656 ConfigureDataTypeManager(); 656 ConfigureDataTypeManager();
657 } else { 657 } else {
658 // DCHECK(FirstSetupInProgress()); 658 DCHECK(FirstSetupInProgress());
659 } 659 }
660 } 660 }
661 661
662 void ProfileSyncService::OnSyncCycleCompleted() { 662 void ProfileSyncService::OnSyncCycleCompleted() {
663 UpdateLastSyncedTime(); 663 UpdateLastSyncedTime();
664 if (GetSessionModelAssociator()) { 664 if (GetSessionModelAssociator()) {
665 // Trigger garbage collection of old sessions now that we've downloaded 665 // Trigger garbage collection of old sessions now that we've downloaded
666 // any new session data. TODO(zea): Have this be a notification the session 666 // any new session data. TODO(zea): Have this be a notification the session
667 // model associator listens too. Also consider somehow plumbing the current 667 // model associator listens too. Also consider somehow plumbing the current
668 // server time as last reported by CheckServerReachable, so we don't have to 668 // server time as last reported by CheckServerReachable, so we don't have to
(...skipping 941 matching lines...) Expand 10 before | Expand all | Expand 10 after
1610 // See http://stackoverflow.com/questions/6224121/is-new-this-myclass-undefine d-behaviour-after-directly-calling-the-destru. 1610 // See http://stackoverflow.com/questions/6224121/is-new-this-myclass-undefine d-behaviour-after-directly-calling-the-destru.
1611 ProfileSyncService* old_this = this; 1611 ProfileSyncService* old_this = this;
1612 this->~ProfileSyncService(); 1612 this->~ProfileSyncService();
1613 new(old_this) ProfileSyncService( 1613 new(old_this) ProfileSyncService(
1614 new ProfileSyncComponentsFactoryImpl(profile, 1614 new ProfileSyncComponentsFactoryImpl(profile,
1615 CommandLine::ForCurrentProcess()), 1615 CommandLine::ForCurrentProcess()),
1616 profile, 1616 profile,
1617 signin, 1617 signin,
1618 behavior); 1618 behavior);
1619 } 1619 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698