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

Unified Diff: chrome/browser/sync/glue/data_type_manager_mock.cc

Issue 10834351: [sync] Divorce DataTypeManager from NotificationService notifications by creating a new DataTypeMa… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Restore expectations in DTMImpl unittest + Treat observer_ as const Created 8 years, 4 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/glue/data_type_manager_mock.cc
diff --git a/chrome/browser/sync/glue/data_type_manager_mock.cc b/chrome/browser/sync/glue/data_type_manager_mock.cc
index 9c420b059725e1cb42ceafd9c3d2f24adc3ebda3..d0c155a3239537dc308f0fe68c6df8ccaab211a2 100644
--- a/chrome/browser/sync/glue/data_type_manager_mock.cc
+++ b/chrome/browser/sync/glue/data_type_manager_mock.cc
@@ -10,31 +10,6 @@ namespace browser_sync {
DataTypeManagerMock::DataTypeManagerMock()
: result_(OK, syncer::ModelTypeSet()) {
-
- // By default, calling Configure will send a SYNC_CONFIGURE_START
- // and SYNC_CONFIGURE_DONE notification with a DataTypeManager::OK
- // detail.
- ON_CALL(*this, Configure(testing::_, testing::_)).
- WillByDefault(testing::DoAll(
- NotifyFromDataTypeManager(
- this,
- static_cast<int>(chrome::NOTIFICATION_SYNC_CONFIGURE_START)),
- NotifyFromDataTypeManagerWithResult(
- this, static_cast<int>(chrome::NOTIFICATION_SYNC_CONFIGURE_DONE),
- &result_)));
-
- // By default, calling ConfigureWithoutNigori will send a SYNC_CONFIGURE_START
- // and SYNC_CONFIGURE_DONE notification with a DataTypeManager::OK
- // detail.
- ON_CALL(*this, ConfigureWithoutNigori(testing::_, testing::_)).
- WillByDefault(testing::DoAll(
- NotifyFromDataTypeManager(
- this,
- static_cast<int>(chrome::NOTIFICATION_SYNC_CONFIGURE_START)),
- NotifyFromDataTypeManagerWithResult(
- this,
- static_cast<int>(chrome::NOTIFICATION_SYNC_CONFIGURE_DONE),
- &result_)));
}
DataTypeManagerMock::~DataTypeManagerMock() {}
« no previous file with comments | « chrome/browser/sync/glue/data_type_manager_mock.h ('k') | chrome/browser/sync/glue/data_type_manager_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698