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

Unified Diff: components/sync_driver/local_device_info_provider_mock.cc

Issue 1951703002: [Sync] Clear LocalDeviceInfoProvider cache on sync DB destruction. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move NonThreadSafe to LocalDeviceInfoProviderImpl. Created 4 years, 7 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_driver/local_device_info_provider_mock.cc
diff --git a/components/sync_driver/local_device_info_provider_mock.cc b/components/sync_driver/local_device_info_provider_mock.cc
index 46fea36ae126ba6a5157dd7fd817ab862bac15f5..3ab5ce9564b279a9c6ac80a1474a701feae794c2 100644
--- a/components/sync_driver/local_device_info_provider_mock.cc
+++ b/components/sync_driver/local_device_info_provider_mock.cc
@@ -30,7 +30,7 @@ LocalDeviceInfoProviderMock::LocalDeviceInfoProviderMock(
LocalDeviceInfoProviderMock::~LocalDeviceInfoProviderMock() {}
const DeviceInfo* LocalDeviceInfoProviderMock::GetLocalDeviceInfo() const {
- return is_initialized_ ? local_device_info_.get() : NULL;
+ return is_initialized_ ? local_device_info_.get() : nullptr;
}
std::string LocalDeviceInfoProviderMock::GetSyncUserAgent() const {
@@ -45,7 +45,10 @@ void LocalDeviceInfoProviderMock::Initialize(
const std::string& cache_guid,
const std::string& signin_scoped_device_id,
const scoped_refptr<base::TaskRunner>& blocking_task_runner) {
- // Ignored for the mock provider.
+ local_device_info_.reset(new DeviceInfo(
+ cache_guid, "client_name", "chrome_version", GetSyncUserAgent(),
+ sync_pb::SyncEnums_DeviceType_TYPE_LINUX, signin_scoped_device_id));
+ SetInitialized(true);
}
void LocalDeviceInfoProviderMock::Initialize(
@@ -61,6 +64,11 @@ LocalDeviceInfoProviderMock::RegisterOnInitializedCallback(
return callback_list_.Add(callback);
}
+void LocalDeviceInfoProviderMock::Clear() {
+ local_device_info_.reset();
+ is_initialized_ = false;
+}
+
void LocalDeviceInfoProviderMock::SetInitialized(bool is_initialized) {
is_initialized_ = is_initialized;
if (is_initialized_) {

Powered by Google App Engine
This is Rietveld 408576698