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

Side by Side Diff: components/sync/driver/sync_service_base.cc

Issue 2494873003: [Sync] Allow sync start without sign-in if the local sync backend is on. (Closed)
Patch Set: Remove ifdefs around include. Created 4 years 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "components/sync/driver/sync_service_base.h" 5 #include "components/sync/driver/sync_service_base.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
129 base::PathService::Get(base::DIR_APP_DATA, local_sync_backend_folder)); 129 base::PathService::Get(base::DIR_APP_DATA, local_sync_backend_folder));
130 *local_sync_backend_folder = local_sync_backend_folder->Append( 130 *local_sync_backend_folder = local_sync_backend_folder->Append(
131 FILE_PATH_LITERAL("Chrome/User Data")); 131 FILE_PATH_LITERAL("Chrome/User Data"));
132 } 132 }
133 // This code as it is now will assume the same profile order is present on all 133 // This code as it is now will assume the same profile order is present on all
134 // machines, which is not a given. It is to be defined if only the Default 134 // machines, which is not a given. It is to be defined if only the Default
135 // profile should get this treatment or all profile as is the case now. The 135 // profile should get this treatment or all profile as is the case now. The
136 // solution for now will be to assume profiles are created in the same order 136 // solution for now will be to assume profiles are created in the same order
137 // on all machines and in the future decide if only the Default one should be 137 // on all machines and in the future decide if only the Default one should be
138 // considered roamed. 138 // considered roamed.
139 // See http://crbug.com/674928.
139 *local_sync_backend_folder = 140 *local_sync_backend_folder =
140 local_sync_backend_folder->Append(base_directory_.BaseName()); 141 local_sync_backend_folder->Append(base_directory_.BaseName());
141 *local_sync_backend_folder = 142 *local_sync_backend_folder =
142 local_sync_backend_folder->Append(kLoopbackServerBackendFilename); 143 local_sync_backend_folder->Append(kLoopbackServerBackendFilename);
143 #endif // defined(OS_WIN) 144 #endif // defined(OS_WIN)
144 return enable_local_sync_backend; 145 return enable_local_sync_backend;
145 } 146 }
146 147
147 } // namespace syncer 148 } // namespace syncer
OLDNEW
« no previous file with comments | « components/sync/driver/sync_service.h ('k') | components/sync/engine/engine_components_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698