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

Unified Diff: chrome/browser/sync/profile_sync_service_unittest.cc

Issue 535683002: Fix use-after-free in HDDDTC shutdown (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix tests Created 6 years, 3 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: chrome/browser/sync/profile_sync_service_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_unittest.cc b/chrome/browser/sync/profile_sync_service_unittest.cc
index 809335861ff82752b85d3486807160d59992f2f7..accbcb4c5317b70bc97622746a1613e14663f4ed 100644
--- a/chrome/browser/sync/profile_sync_service_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_unittest.cc
@@ -58,6 +58,7 @@ class FakeDataTypeManager : public sync_driver::DataTypeManager {
}
virtual void ReenableType(syncer::ModelType type) OVERRIDE {}
+ virtual void ResetDataTypeErrors() OVERRIDE {}
virtual void PurgeForMigration(syncer::ModelTypeSet undesired_types,
syncer::ConfigureReason reason) OVERRIDE {}
virtual void Stop() OVERRIDE {};
@@ -251,7 +252,7 @@ class ProfileSyncServiceTest : public ::testing::Test {
}
void ExpectDataTypeManagerCreation(int times) {
- EXPECT_CALL(*components_factory_, CreateDataTypeManager(_, _, _, _, _, _))
+ EXPECT_CALL(*components_factory_, CreateDataTypeManager(_, _, _, _, _))
.Times(times)
.WillRepeatedly(ReturnNewDataTypeManager());
}

Powered by Google App Engine
This is Rietveld 408576698