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

Side by Side Diff: chrome/browser/webdata/autocomplete_syncable_service.cc

Issue 12494020: Remove knowledge of Chrome-specific SyncableService classes from WebDataService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add DCHECK. Created 7 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) 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/webdata/autocomplete_syncable_service.h" 5 #include "chrome/browser/webdata/autocomplete_syncable_service.h"
6 6
7 #include "base/location.h" 7 #include "base/location.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/utf_string_conversions.h" 9 #include "base/utf_string_conversions.h"
10 #include "chrome/browser/profiles/profile.h" 10 #include "chrome/browser/profiles/profile.h"
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 return true; 72 return true;
73 } else { 73 } else {
74 new_timestamps->insert(new_timestamps->begin(), 74 new_timestamps->insert(new_timestamps->begin(),
75 timestamps.begin(), 75 timestamps.begin(),
76 timestamps.end()); 76 timestamps.end());
77 return false; 77 return false;
78 } 78 }
79 } 79 }
80 } 80 }
81 81
82 void* UserDataKey() {
83 return reinterpret_cast<void*>(
84 &AutocompleteSyncableService::CreateForWebDataService);
85 }
86
82 } // namespace 87 } // namespace
83 88
84 AutocompleteSyncableService::AutocompleteSyncableService( 89 AutocompleteSyncableService::AutocompleteSyncableService(
85 WebDataService* web_data_service) 90 WebDataService* web_data_service)
86 : web_data_service_(web_data_service), 91 : web_data_service_(web_data_service),
87 cull_expired_entries_(false) { 92 cull_expired_entries_(false) {
88 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB)); 93 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
89 DCHECK(web_data_service_); 94 DCHECK(web_data_service_);
90 notification_registrar_.Add( 95 notification_registrar_.Add(
91 this, chrome::NOTIFICATION_AUTOFILL_ENTRIES_CHANGED, 96 this, chrome::NOTIFICATION_AUTOFILL_ENTRIES_CHANGED,
92 content::Source<WebDataService>(web_data_service)); 97 content::Source<WebDataService>(web_data_service));
93 } 98 }
94 99
95 AutocompleteSyncableService::~AutocompleteSyncableService() { 100 AutocompleteSyncableService::~AutocompleteSyncableService() {
96 DCHECK(CalledOnValidThread()); 101 DCHECK(CalledOnValidThread());
97 } 102 }
98 103
104 // static
105 void AutocompleteSyncableService::CreateForWebDataService(
106 WebDataService* web_data) {
107 web_data->GetDBUserData()->SetUserData(
108 UserDataKey(), new AutocompleteSyncableService(web_data));
109 }
110
111 // static
112 AutocompleteSyncableService* AutocompleteSyncableService::FromWebDataService(
113 WebDataService* web_data) {
114 return static_cast<AutocompleteSyncableService*>(
115 web_data->GetDBUserData()->GetUserData(UserDataKey()));
116 }
117
99 AutocompleteSyncableService::AutocompleteSyncableService() 118 AutocompleteSyncableService::AutocompleteSyncableService()
100 : web_data_service_(NULL), 119 : web_data_service_(NULL),
101 cull_expired_entries_(false) { 120 cull_expired_entries_(false) {
102 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB)); 121 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::DB));
103 } 122 }
104 123
105 syncer::SyncMergeResult AutocompleteSyncableService::MergeDataAndStartSyncing( 124 syncer::SyncMergeResult AutocompleteSyncableService::MergeDataAndStartSyncing(
106 syncer::ModelType type, 125 syncer::ModelType type,
107 const syncer::SyncDataList& initial_sync_data, 126 const syncer::SyncDataList& initial_sync_data,
108 scoped_ptr<syncer::SyncChangeProcessor> sync_processor, 127 scoped_ptr<syncer::SyncChangeProcessor> sync_processor,
(...skipping 348 matching lines...) Expand 10 before | Expand all | Expand 10 after
457 UTF16ToUTF8(entry.key().value()))); 476 UTF16ToUTF8(entry.key().value())));
458 return syncer::SyncData::CreateLocalData(tag, tag, autofill_specifics); 477 return syncer::SyncData::CreateLocalData(tag, tag, autofill_specifics);
459 } 478 }
460 479
461 // static 480 // static
462 std::string AutocompleteSyncableService::KeyToTag(const std::string& name, 481 std::string AutocompleteSyncableService::KeyToTag(const std::string& name,
463 const std::string& value) { 482 const std::string& value) {
464 std::string ns(kAutofillEntryNamespaceTag); 483 std::string ns(kAutofillEntryNamespaceTag);
465 return ns + net::EscapePath(name) + "|" + net::EscapePath(value); 484 return ns + net::EscapePath(name) + "|" + net::EscapePath(value);
466 } 485 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698