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

Side by Side Diff: components/autofill/core/browser/autofill_manager.cc

Issue 2142123002: [Autofill] Run autofill-profile de-dupe after sync starts if sync is enabled. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nits Created 4 years, 5 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/autofill/core/browser/autofill_manager.h" 5 #include "components/autofill/core/browser/autofill_manager.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 197 matching lines...) Expand 10 before | Expand all | Expand 10 after
208 user_did_autofill_(false), 208 user_did_autofill_(false),
209 user_did_edit_autofilled_field_(false), 209 user_did_edit_autofilled_field_(false),
210 user_did_accept_upload_prompt_(false), 210 user_did_accept_upload_prompt_(false),
211 external_delegate_(NULL), 211 external_delegate_(NULL),
212 test_delegate_(NULL), 212 test_delegate_(NULL),
213 weak_ptr_factory_(this) { 213 weak_ptr_factory_(this) {
214 if (enable_download_manager == ENABLE_AUTOFILL_DOWNLOAD_MANAGER) { 214 if (enable_download_manager == ENABLE_AUTOFILL_DOWNLOAD_MANAGER) {
215 download_manager_.reset(new AutofillDownloadManager(driver, this)); 215 download_manager_.reset(new AutofillDownloadManager(driver, this));
216 } 216 }
217 CountryNames::SetLocaleString(app_locale_); 217 CountryNames::SetLocaleString(app_locale_);
218 if (personal_data_ && client_)
219 personal_data_->OnSyncServiceConfigured(client_->GetSyncService());
Nicolas Zea 2016/07/12 20:04:47 How do you know that the sync service is configure
Roger McFarlane (Chromium) 2016/07/12 21:23:26 Because we're in the context of the AutofillManage
218 } 220 }
219 221
220 AutofillManager::~AutofillManager() {} 222 AutofillManager::~AutofillManager() {}
221 223
222 // static 224 // static
223 void AutofillManager::RegisterProfilePrefs( 225 void AutofillManager::RegisterProfilePrefs(
224 user_prefs::PrefRegistrySyncable* registry) { 226 user_prefs::PrefRegistrySyncable* registry) {
225 registry->RegisterBooleanPref( 227 registry->RegisterBooleanPref(
226 prefs::kAutofillEnabled, 228 prefs::kAutofillEnabled,
227 true, 229 true,
(...skipping 1798 matching lines...) Expand 10 before | Expand all | Expand 10 after
2026 if (i > 0) 2028 if (i > 0)
2027 fputs("Next oldest form:\n", file); 2029 fputs("Next oldest form:\n", file);
2028 } 2030 }
2029 fputs("\n", file); 2031 fputs("\n", file);
2030 2032
2031 fclose(file); 2033 fclose(file);
2032 } 2034 }
2033 #endif // ENABLE_FORM_DEBUG_DUMP 2035 #endif // ENABLE_FORM_DEBUG_DUMP
2034 2036
2035 } // namespace autofill 2037 } // namespace autofill
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698