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

Unified Diff: chrome/test/live_sync/profile_sync_service_test_harness.cc

Issue 3915004: Convert LOG(INFO) to VLOG(1) - chrome/test/. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 10 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: chrome/test/live_sync/profile_sync_service_test_harness.cc
===================================================================
--- chrome/test/live_sync/profile_sync_service_test_harness.cc (revision 63075)
+++ chrome/test/live_sync/profile_sync_service_test_harness.cc (working copy)
@@ -310,7 +310,7 @@
// static
bool ProfileSyncServiceTestHarness::AwaitQuiescence(
std::vector<ProfileSyncServiceTestHarness*>& clients) {
- LOG(INFO) << "AwaitQuiescence.";
+ VLOG(1) << "AwaitQuiescence.";
bool return_value = true;
for (std::vector<ProfileSyncServiceTestHarness*>::iterator it =
clients.begin(); it != clients.end(); ++it) {
@@ -407,13 +407,11 @@
service()->OnUserChoseDatatypes(false, synced_datatypes);
wait_state_ = WAITING_FOR_SYNC_TO_FINISH;
AwaitSyncCycleCompletion("Waiting for datatype configuration.");
- LOG(INFO) << "EnableSyncForDatatype(): Enabled sync for datatype "
- << syncable::ModelTypeToString(datatype)
- << " on Client " << id_ << ".";
+ VLOG(1) << "EnableSyncForDatatype(): Enabled sync for datatype "
+ << syncable::ModelTypeToString(datatype) << " on Client " << id_;
} else {
- LOG(INFO) << "EnableSyncForDatatype(): Sync already enabled for datatype "
- << syncable::ModelTypeToString(datatype)
- << " on Client " << id_ << ".";
+ VLOG(1) << "EnableSyncForDatatype(): Sync already enabled for datatype "
+ << syncable::ModelTypeToString(datatype) << " on Client " << id_;
}
}
}
@@ -430,13 +428,11 @@
synced_datatypes.erase(it);
service()->OnUserChoseDatatypes(false, synced_datatypes);
AwaitSyncCycleCompletion("Waiting for datatype configuration.");
- LOG(INFO) << "DisableSyncForDatatype(): Disabled sync for datatype "
- << syncable::ModelTypeToString(datatype)
- << " on Client " << id_ << ".";
+ VLOG(1) << "DisableSyncForDatatype(): Disabled sync for datatype "
+ << syncable::ModelTypeToString(datatype) << " on Client " << id_;
} else {
- LOG(INFO) << "DisableSyncForDatatype(): Sync already disabled for datatype "
- << syncable::ModelTypeToString(datatype)
- << " on Client " << id_ << ".";
+ VLOG(1) << "DisableSyncForDatatype(): Sync already disabled for datatype "
+ << syncable::ModelTypeToString(datatype) << " on Client " << id_;
}
}
@@ -457,8 +453,8 @@
service()->OnUserChoseDatatypes(true, synced_datatypes);
wait_state_ = WAITING_FOR_SYNC_TO_FINISH;
AwaitSyncCycleCompletion("Waiting for datatype configuration.");
- LOG(INFO) << "EnableSyncForAllDatatypes(): Enabled sync for all datatypes"
- << " on Client " << id_ << ".";
+ VLOG(1) << "EnableSyncForAllDatatypes(): Enabled sync for all datatypes on "
+ "Client " << id_;
}
}
@@ -468,8 +464,8 @@
<< "EnableSyncForAllDatatypes(): service() is null.";
service()->DisableForUser();
wait_state_ = SYNC_DISABLED;
- LOG(INFO) << "DisableSyncForAllDatatypes(): Disabled sync for all datatypes"
- << " on Client " << id_ << ".";
+ VLOG(1) << "DisableSyncForAllDatatypes(): Disabled sync for all datatypes on "
+ "Client " << id_;
}
int64 ProfileSyncServiceTestHarness::GetUpdatedTimestamp() {
@@ -483,16 +479,17 @@
void ProfileSyncServiceTestHarness::LogClientInfo(std::string message) {
const SyncSessionSnapshot* snap = GetLastSessionSnapshot();
if (snap) {
- LOG(INFO) << "Client " << id_ << ": " << message << ": "
- << "max_local_timestamp: " << snap->max_local_timestamp
- << ", has_more_to_sync: " << snap->has_more_to_sync
- << ", unsynced_count: " << snap->unsynced_count
- << ", has_unsynced_items: " << service()->backend()->HasUnsyncedItems()
- << ", notifications_enabled: " << GetStatus().notifications_enabled
- << ", service_is_pushing_changes: " << ServiceIsPushingChanges()
- << ".";
+ VLOG(1) << "Client " << id_ << ": " << message
+ << ": max_local_timestamp: " << snap->max_local_timestamp
+ << ", has_more_to_sync: " << snap->has_more_to_sync
+ << ", unsynced_count: " << snap->unsynced_count
+ << ", has_unsynced_items: "
+ << service()->backend()->HasUnsyncedItems()
+ << ", notifications_enabled: "
+ << GetStatus().notifications_enabled
+ << ", service_is_pushing_changes: " << ServiceIsPushingChanges();
} else {
- LOG(INFO) << "Client " << id_ << ": " << message << ": "
- << "Sync session snapshot not available.";
+ VLOG(1) << "Client " << id_ << ": " << message
+ << ": Sync session snapshot not available.";
}
}
« no previous file with comments | « chrome/test/live_sync/live_sync_test.cc ('k') | chrome/test/live_sync/two_client_live_bookmarks_sync_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698