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

Side by Side Diff: chrome/browser/sync/glue/autofill_model_associator.cc

Issue 6715024: fix spelling of comments (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/autofill_model_associator.h" 5 #include "chrome/browser/sync/glue/autofill_model_associator.h"
6 6
7 #include <functional> 7 #include <functional>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/string_number_conversions.h" 10 #include "base/string_number_conversions.h"
(...skipping 160 matching lines...) Expand 10 before | Expand all | Expand 10 after
171 &bundle, 171 &bundle,
172 profiles.get())) { 172 profiles.get())) {
173 return false; 173 return false;
174 } 174 }
175 } 175 }
176 176
177 // Since we're on the DB thread, we don't have to worry about updating 177 // Since we're on the DB thread, we don't have to worry about updating
178 // the autofill database after closing the write transaction, since 178 // the autofill database after closing the write transaction, since
179 // this is the only thread that writes to the database. We also don't have 179 // this is the only thread that writes to the database. We also don't have
180 // to worry about the sync model getting out of sync, because changes are 180 // to worry about the sync model getting out of sync, because changes are
181 // propogated to the ChangeProcessor on this thread. 181 // propagated to the ChangeProcessor on this thread.
182 if (!SaveChangesToWebData(bundle)) { 182 if (!SaveChangesToWebData(bundle)) {
183 LOG(ERROR) << "Failed to update autofill entries."; 183 LOG(ERROR) << "Failed to update autofill entries.";
184 return false; 184 return false;
185 } 185 }
186 186
187 if (sync_service_->GetAutofillMigrationState() != 187 if (sync_service_->GetAutofillMigrationState() !=
188 syncable::MIGRATED) { 188 syncable::MIGRATED) {
189 syncable::AutofillMigrationDebugInfo debug_info; 189 syncable::AutofillMigrationDebugInfo debug_info;
190 debug_info.autofill_entries_added_during_migration = 190 debug_info.autofill_entries_added_during_migration =
191 number_of_entries_created_; 191 number_of_entries_created_;
(...skipping 377 matching lines...) Expand 10 before | Expand all | Expand 10 after
569 sync_service_->SetAutofillMigrationState(syncable::MIGRATED); 569 sync_service_->SetAutofillMigrationState(syncable::MIGRATED);
570 570
571 VLOG(1) << "[AUTOFILL MIGRATION]" 571 VLOG(1) << "[AUTOFILL MIGRATION]"
572 << "Current autofill migration state is migrated."; 572 << "Current autofill migration state is migrated.";
573 } 573 }
574 574
575 return false; 575 return false;
576 } 576 }
577 577
578 } // namespace browser_sync 578 } // namespace browser_sync
OLDNEW
« no previous file with comments | « chrome/browser/renderer_host/browser_render_process_host.cc ('k') | chrome/browser/sync/glue/typed_url_model_associator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698