diff options
author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-22 01:22:29 +0000 |
---|---|---|
committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-22 01:22:29 +0000 |
commit | 0a126223a953345b09464fcc95631a96fad35315 (patch) | |
tree | 1947a11e1e08dbb6d19eaf51ff5d58cac6d165ec /chrome/browser/sync/notifier | |
parent | 93811c13d691dcc8d21451b6816a030750d81fc8 (diff) | |
download | chromium_src-0a126223a953345b09464fcc95631a96fad35315.zip chromium_src-0a126223a953345b09464fcc95631a96fad35315.tar.gz chromium_src-0a126223a953345b09464fcc95631a96fad35315.tar.bz2 |
Convert LOG(INFO) to VLOG(1) - misc. chrome/browser/sync/*.
Also remove some "using"s, remove some extra {}s, and fix function argument wrapping to comply with the style guide.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/3975003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63470 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/sync/notifier')
-rw-r--r-- | chrome/browser/sync/notifier/chrome_system_resources_unittest.cc | 2 | ||||
-rw-r--r-- | chrome/browser/sync/notifier/server_notifier_thread.cc | 13 |
2 files changed, 6 insertions, 9 deletions
diff --git a/chrome/browser/sync/notifier/chrome_system_resources_unittest.cc b/chrome/browser/sync/notifier/chrome_system_resources_unittest.cc index fa5f456..306a8b6 100644 --- a/chrome/browser/sync/notifier/chrome_system_resources_unittest.cc +++ b/chrome/browser/sync/notifier/chrome_system_resources_unittest.cc @@ -77,7 +77,7 @@ class ChromeSystemResourcesTest : public testing::Test { TEST_F(ChromeSystemResourcesTest, CurrentTime) { invalidation::Time current_time = chrome_system_resources_.current_time(); - LOG(INFO) << "current_time returned: " << current_time.ToInternalValue(); + VLOG(1) << "current_time returned: " << current_time.ToInternalValue(); } // Make sure Log() doesn't crash or leak. diff --git a/chrome/browser/sync/notifier/server_notifier_thread.cc b/chrome/browser/sync/notifier/server_notifier_thread.cc index 4c73445..53aadb0 100644 --- a/chrome/browser/sync/notifier/server_notifier_thread.cc +++ b/chrome/browser/sync/notifier/server_notifier_thread.cc @@ -61,8 +61,8 @@ void ServerNotifierThread::Logout() { void ServerNotifierThread::SendNotification( const OutgoingNotificationData& data) { DCHECK_EQ(MessageLoop::current(), parent_message_loop_); - NOTREACHED() << "Shouldn't send notifications if " - << "ServerNotifierThread is used"; + NOTREACHED() << "Shouldn't send notifications if ServerNotifierThread is " + "used"; } void ServerNotifierThread::OnInvalidate(syncable::ModelType model_type) { @@ -70,11 +70,8 @@ void ServerNotifierThread::OnInvalidate(syncable::ModelType model_type) { // TODO(akalin): This is a hack to make new sync data types work // with server-issued notifications. Remove this when it's not // needed anymore. - if (model_type == syncable::UNSPECIFIED) { - LOG(INFO) << "OnInvalidate: UNKNOWN"; - } else { - LOG(INFO) << "OnInvalidate: " << syncable::ModelTypeToString(model_type); - } + VLOG(1) << "OnInvalidate: " << ((model_type == syncable::UNSPECIFIED) ? + "UNKNOWN" : syncable::ModelTypeToString(model_type)); // TODO(akalin): Signal notification only for the invalidated types. // TODO(akalin): Fill this in with something meaningful. IncomingNotificationData notification_data; @@ -83,7 +80,7 @@ void ServerNotifierThread::OnInvalidate(syncable::ModelType model_type) { void ServerNotifierThread::OnInvalidateAll() { DCHECK_EQ(MessageLoop::current(), worker_message_loop()); - LOG(INFO) << "OnInvalidateAll"; + VLOG(1) << "OnInvalidateAll"; // TODO(akalin): Fill this in with something meaningful. IncomingNotificationData notification_data; observers_->Notify(&Observer::OnIncomingNotification, notification_data); |