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

Unified Diff: components/sync/device_info/device_info_service.h

Issue 2461723002: [Sync] DeviceInfoService static method and error cleanup. (Closed)
Patch Set: Moved local suffix into a constant. Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: components/sync/device_info/device_info_service.h
diff --git a/components/sync/device_info/device_info_service.h b/components/sync/device_info/device_info_service.h
index c5156d07b0b56cc1db3161d4967054936b5ba1b8..952f2f82374cad5b6d5d9b6e96160f1346c6ce83 100644
--- a/components/sync/device_info/device_info_service.h
+++ b/components/sync/device_info/device_info_service.h
@@ -75,16 +75,6 @@ class DeviceInfoService : public ModelTypeService, public DeviceInfoTracker {
using ClientIdToSpecifics =
std::map<std::string, std::unique_ptr<sync_pb::DeviceInfoSpecifics>>;
- static std::unique_ptr<sync_pb::DeviceInfoSpecifics> CopyToSpecifics(
- const DeviceInfo& info);
-
- // Allocate new DeviceInfo from SyncData.
- static std::unique_ptr<DeviceInfo> CopyToModel(
- const sync_pb::DeviceInfoSpecifics& specifics);
- // Conversion as we prepare to hand data to the processor.
- static std::unique_ptr<EntityData> CopyToEntityData(
- const sync_pb::DeviceInfoSpecifics& specifics);
-
// Store SyncData in the cache and durable storage.
void StoreSpecifics(std::unique_ptr<sync_pb::DeviceInfoSpecifics> specifics,
ModelTypeStore::WriteBatch* batch);
@@ -139,10 +129,6 @@ class DeviceInfoService : public ModelTypeService, public DeviceInfoTracker {
// datatype, since these errors prevent us from knowing if sync is enabled.
void ReportStartupErrorToSync(const std::string& msg);
- // Find the timestamp for the last time this |device_info| was edited.
- static base::Time GetLastUpdateTime(
- const sync_pb::DeviceInfoSpecifics& specifics);
-
// |local_device_info_provider_| isn't owned.
const LocalDeviceInfoProvider* const local_device_info_provider_;
« no previous file with comments | « no previous file | components/sync/device_info/device_info_service.cc » ('j') | components/sync/device_info/device_info_service.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698